diff --git a/charts/clearml-agent/Chart.yaml b/charts/clearml-agent/Chart.yaml index 9844f41..def6d6c 100644 --- a/charts/clearml-agent/Chart.yaml +++ b/charts/clearml-agent/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: clearml-agent description: MLOps platform Task running agent type: application -version: "3.3.1" +version: "3.3.2" appVersion: "1.24" kubeVersion: ">= 1.21.0-0 < 1.27.0-0" home: https://clear.ml @@ -21,4 +21,4 @@ keywords: annotations: artifacthub.io/changes: | - kind: fixed - description: typo on existingSecret references + description: clearml agent internal helper variable name diff --git a/charts/clearml-agent/README.md b/charts/clearml-agent/README.md index 3c5ae9d..991608d 100644 --- a/charts/clearml-agent/README.md +++ b/charts/clearml-agent/README.md @@ -1,6 +1,6 @@ # ClearML Kubernetes Agent -![Version: 3.3.1](https://img.shields.io/badge/Version-3.3.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.24](https://img.shields.io/badge/AppVersion-1.24-informational?style=flat-square) +![Version: 3.3.2](https://img.shields.io/badge/Version-3.3.2-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.24](https://img.shields.io/badge/AppVersion-1.24-informational?style=flat-square) MLOps platform Task running agent diff --git a/charts/clearml-agent/templates/_helpers.tpl b/charts/clearml-agent/templates/_helpers.tpl index 3239bae..fbed08c 100644 --- a/charts/clearml-agent/templates/_helpers.tpl +++ b/charts/clearml-agent/templates/_helpers.tpl @@ -1,23 +1,23 @@ {{/* Expand the name of the chart. */}} -{{- define "clearml.name" -}} +{{- define "clearmlAgent.name" -}} {{- .Release.Name | trunc 59 | trimSuffix "-" }} {{- end }} {{/* Create chart name and version as used by the chart label. */}} -{{- define "clearml.chart" -}} +{{- define "clearmlAgent.chart" -}} {{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 59 | trimSuffix "-" }} {{- end }} {{/* Common labels */}} -{{- define "clearml.labels" -}} -helm.sh/chart: {{ include "clearml.chart" . }} -{{ include "clearml.selectorLabels" . }} +{{- define "clearmlAgent.labels" -}} +helm.sh/chart: {{ include "clearmlAgent.chart" . }} +{{ include "clearmlAgent.selectorLabels" . }} {{- if .Chart.AppVersion }} app.kubernetes.io/version: {{ .Chart.AppVersion | quote }} {{- end }} @@ -30,7 +30,7 @@ app.kubernetes.io/managed-by: {{ .Release.Service }} {{/* Common annotations */}} -{{- define "clearml.annotations" -}} +{{- define "clearmlAgent.annotations" -}} {{- if $.Values.agentk8sglue.annotations }} {{ toYaml $.Values.agentk8sglue.annotations }} {{- end }} @@ -39,8 +39,8 @@ Common annotations {{/* Selector labels */}} -{{- define "clearml.selectorLabels" -}} -app.kubernetes.io/name: {{ include "clearml.name" . }} +{{- define "clearmlAgent.selectorLabels" -}} +app.kubernetes.io/name: {{ include "clearmlAgent.name" . }} app.kubernetes.io/instance: {{ .Release.Name }} {{- end }} @@ -48,18 +48,18 @@ app.kubernetes.io/instance: {{ .Release.Name }} Selector labels (agentk8sglue) */}} {{- define "agentk8sglue.selectorLabels" -}} -app.kubernetes.io/name: {{ include "clearml.name" . }} -app.kubernetes.io/instance: {{ include "clearml.name" . }} +app.kubernetes.io/name: {{ include "clearmlAgent.name" . }} +app.kubernetes.io/instance: {{ include "clearmlAgent.name" . }} {{- end }} {{/* Create the name of the service account to use */}} -{{- define "clearml.serviceAccountName" -}} +{{- define "clearmlAgent.serviceAccountName" -}} {{- if .Values.agentk8sglue.serviceExistingAccountName }} {{- .Values.agentk8sglue.serviceExistingAccountName }} {{- else }} -{{- include "clearml.name" . }}-sa +{{- include "clearmlAgent.name" . }}-sa {{- end }} {{- end }} diff --git a/charts/clearml-agent/templates/agentk8sglue-configmap.yaml b/charts/clearml-agent/templates/agentk8sglue-configmap.yaml index aecde2e..df064d0 100644 --- a/charts/clearml-agent/templates/agentk8sglue-configmap.yaml +++ b/charts/clearml-agent/templates/agentk8sglue-configmap.yaml @@ -1,7 +1,7 @@ apiVersion: v1 kind: ConfigMap metadata: - name: {{ include "clearml.name" . }}-pt + name: {{ include "clearmlAgent.name" . }}-pt data: {{- if .Values.enterpriseFeatures.enabled }} template.yaml: | @@ -30,7 +30,7 @@ data: {{- if $.Values.imageCredentials.existingSecret }} - name: {{ $.Values.imageCredentials.existingSecret }} {{- else }} - - name: {{ include "clearml.name" $ }}-ark + - name: {{ include "clearmlAgent.name" $ }}-ark {{- end }} {{- end }} {{- if $value.templateOverrides.schedulerName }} @@ -66,14 +66,14 @@ data: {{- if $value.templateOverrides.fileMounts }} - name: filemounts secret: - secretName: {{ include "clearml.name" $ }}-{{ $key }}-fm + secretName: {{ include "clearmlAgent.name" $ }}-{{ $key }}-fm {{- else if $.Values.agentk8sglue.basePodTemplate.fileMounts }} - name: filemounts secret: - secretName: {{ include "clearml.name" $ }}-fm + secretName: {{ include "clearmlAgent.name" $ }}-fm {{- end }} {{- if not $.Values.enterpriseFeatures.serviceAccountClusterAccess }} - serviceAccountName: {{ include "clearml.serviceAccountName" $ }} + serviceAccountName: {{ include "clearmlAgent.serviceAccountName" $ }} {{- end }} {{- if $value.templateOverrides.initContainers }} initContainers: @@ -126,7 +126,7 @@ data: {{- if $.Values.clearml.existingAgentk8sglueSecret }} name: {{ $.Values.clearml.existingAgentk8sglueSecret }} {{- else }} - name: {{ include "clearml.name" $ }}-ac + name: {{ include "clearmlAgent.name" $ }}-ac {{- end }} key: agentk8sglue_key - name: CLEARML_API_SECRET_KEY @@ -135,7 +135,7 @@ data: {{- if $.Values.clearml.existingAgentk8sglueSecret }} name: {{ $.Values.clearml.existingAgentk8sglueSecret }} {{- else }} - name: {{ include "clearml.name" $ }}-ac + name: {{ include "clearmlAgent.name" $ }}-ac {{- end }} key: agentk8sglue_secret {{- end }} @@ -188,9 +188,9 @@ data: {{- range $key, $value := $.Values.enterpriseFeatures.queues }} {{ $key }}: {{- if $value.templateOverrides.fileMounts }} - - {{ include "clearml.name" $ }}-{{ $key }}-fm + - {{ include "clearmlAgent.name" $ }}-{{ $key }}-fm {{- else if $.Values.agentk8sglue.basePodTemplate.fileMounts }} - - {{ include "clearml.name" $ }}-fm + - {{ include "clearmlAgent.name" $ }}-fm {{- end }} {{- end }} {{- else }} @@ -208,14 +208,14 @@ data: {{- if .Values.imageCredentials.existingSecret }} - name: {{ .Values.imageCredentials.existingSecret }} {{- else }} - - name: name: {{ include "clearml.name" $ }}-ark + - name: name: {{ include "clearmlAgent.name" $ }}-ark {{- end }} {{- end }} {{- with .Values.agentk8sglue.basePodTemplate.volumes }} volumes: {{- toYaml . | nindent 8 }} {{- end }} - serviceAccountName: {{ include "clearml.serviceAccountName" $ }} + serviceAccountName: {{ include "clearmlAgent.serviceAccountName" $ }} containers: - resources: {{- toYaml .Values.agentk8sglue.basePodTemplate.resources | nindent 10 }} @@ -238,7 +238,7 @@ data: {{- if .Values.clearml.existingAgentk8sglueSecret }} name: {{ .Values.clearml.existingAgentk8sglueSecret }} {{- else }} - name: {{ include "clearml.name" . }}-ac + name: {{ include "clearmlAgent.name" . }}-ac {{- end }} key: agentk8sglue_key - name: CLEARML_API_SECRET_KEY @@ -247,7 +247,7 @@ data: {{- if .Values.clearml.existingAgentk8sglueSecret }} name: {{ .Values.clearml.existingAgentk8sglueSecret }} {{- else }} - name: {{ include "clearml.name" . }}-ac + name: {{ include "clearmlAgent.name" . }}-ac {{- end }} key: agentk8sglue_secret {{- if .Values.agentk8sglue.basePodTemplate.env }} @@ -274,7 +274,7 @@ data: metadata: name: clearml-session-{{ . }} labels: - {{- include "clearml.labels" $ | nindent 8 }} + {{- include "clearmlAgent.labels" $ | nindent 8 }} {{- with $.Values.sessions.svcAnnotations }} annotations: {{- toYaml . | nindent 8 }} diff --git a/charts/clearml-agent/templates/agentk8sglue-deployment.yaml b/charts/clearml-agent/templates/agentk8sglue-deployment.yaml index a3f9f8d..f54b487 100644 --- a/charts/clearml-agent/templates/agentk8sglue-deployment.yaml +++ b/charts/clearml-agent/templates/agentk8sglue-deployment.yaml @@ -1,11 +1,11 @@ apiVersion: apps/v1 kind: Deployment metadata: - name: {{ include "clearml.name" . }} + name: {{ include "clearmlAgent.name" . }} labels: - {{- include "clearml.labels" . | nindent 4 }} + {{- include "clearmlAgent.labels" . | nindent 4 }} annotations: - {{- include "clearml.annotations" . | nindent 4 }} + {{- include "clearmlAgent.annotations" . | nindent 4 }} spec: replicas: {{ .Values.agentk8sglue.replicaCount }} selector: @@ -15,19 +15,19 @@ spec: metadata: annotations: checksum/config: {{ printf "%s%s" .Values.clearml .Values.agentk8sglue | sha256sum }} - {{- include "clearml.annotations" . | nindent 8 }} + {{- include "clearmlAgent.annotations" . | nindent 8 }} labels: - {{- include "clearml.labels" . | nindent 8 }} + {{- include "clearmlAgent.labels" . | nindent 8 }} spec: {{- if .Values.imageCredentials.enabled }} imagePullSecrets: {{- if .Values.imageCredentials.existingSecret }} - name: {{ .Values.imageCredentials.existingSecret }} {{- else }} - - name: {{ include "clearml.name" . }}-ark + - name: {{ include "clearmlAgent.name" . }}-ark {{- end }} {{- end }} - serviceAccountName: {{ include "clearml.serviceAccountName" . }} + serviceAccountName: {{ include "clearmlAgent.serviceAccountName" . }} securityContext: {{ toYaml .Values.agentk8sglue.securityContext | nindent 8 }} initContainers: - name: init-k8s-glue @@ -68,7 +68,7 @@ spec: export PATH=$PATH:$HOME/bin; source /root/.bashrc && /root/entrypoint.sh volumeMounts: - - name: {{ include "clearml.name" . }}-pt + - name: {{ include "clearmlAgent.name" . }}-pt mountPath: /root/template {{ if .Values.clearml.clearmlConfig }} - name: k8sagent-clearml-conf-volume @@ -122,15 +122,15 @@ spec: - name: CLEARML_API_ACCESS_KEY valueFrom: secretKeyRef: - name: {{ include "clearml.name" . }}-ac + name: {{ include "clearmlAgent.name" . }}-ac key: agentk8sglue_key - name: CLEARML_API_SECRET_KEY valueFrom: secretKeyRef: - name: {{ include "clearml.name" . }}-ac + name: {{ include "clearmlAgent.name" . }}-ac key: agentk8sglue_secret - name: CLEARML_WORKER_ID - value: {{ include "clearml.name" . }} + value: {{ include "clearmlAgent.name" . }} - name: CLEARML_AGENT_UPDATE_REPO value: "" - name: FORCE_CLEARML_AGENT_REPO @@ -186,13 +186,13 @@ spec: {{- toYaml . | nindent 8 }} {{- end }} volumes: - - name: {{ include "clearml.name" . }}-pt + - name: {{ include "clearmlAgent.name" . }}-pt configMap: - name: {{ include "clearml.name" . }}-pt + name: {{ include "clearmlAgent.name" . }}-pt {{ if .Values.clearml.clearmlConfig }} - name: k8sagent-clearml-conf-volume secret: - secretName: {{ include "clearml.name" . }}-ac + secretName: {{ include "clearmlAgent.name" . }}-ac items: - key: clearml.conf path: clearml.conf @@ -203,5 +203,5 @@ spec: {{ if .Values.agentk8sglue.fileMounts }} - name: filemounts secret: - secretName: {{ include "clearml.name" . }}-afm + secretName: {{ include "clearmlAgent.name" . }}-afm {{- end }} diff --git a/charts/clearml-agent/templates/agentk8sglue-rbac.yaml b/charts/clearml-agent/templates/agentk8sglue-rbac.yaml index 7b19d9c..549ca3a 100644 --- a/charts/clearml-agent/templates/agentk8sglue-rbac.yaml +++ b/charts/clearml-agent/templates/agentk8sglue-rbac.yaml @@ -2,7 +2,7 @@ apiVersion: v1 kind: ServiceAccount metadata: - name: {{ include "clearml.serviceAccountName" . }} + name: {{ include "clearmlAgent.serviceAccountName" . }} namespace: {{ .Release.Namespace }} {{- end }} {{- if .Values.enterpriseFeatures.serviceAccountClusterAccess }} @@ -10,7 +10,7 @@ metadata: apiVersion: rbac.authorization.k8s.io/v1 kind: ClusterRole metadata: - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa rules: - apiGroups: - "" @@ -28,21 +28,21 @@ rules: apiVersion: rbac.authorization.k8s.io/v1 kind: ClusterRoleBinding metadata: - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa subjects: - kind: ServiceAccount - name: {{ include "clearml.serviceAccountName" . }} + name: {{ include "clearmlAgent.serviceAccountName" . }} namespace: {{ .Release.Namespace }} roleRef: apiGroup: rbac.authorization.k8s.io kind: ClusterRole - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa {{- else }} --- apiVersion: rbac.authorization.k8s.io/v1 kind: Role metadata: - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa rules: - apiGroups: - "" @@ -60,13 +60,13 @@ rules: apiVersion: rbac.authorization.k8s.io/v1 kind: RoleBinding metadata: - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa subjects: - kind: ServiceAccount - name: {{ include "clearml.serviceAccountName" . }} + name: {{ include "clearmlAgent.serviceAccountName" . }} namespace: {{ .Release.Namespace }} roleRef: apiGroup: rbac.authorization.k8s.io kind: Role - name: {{ include "clearml.name" . }}-kpa + name: {{ include "clearmlAgent.name" . }}-kpa {{- end }} diff --git a/charts/clearml-agent/templates/clearml-secrets.yaml b/charts/clearml-agent/templates/clearml-secrets.yaml index 718ba59..b4348f8 100644 --- a/charts/clearml-agent/templates/clearml-secrets.yaml +++ b/charts/clearml-agent/templates/clearml-secrets.yaml @@ -1,7 +1,7 @@ apiVersion: v1 kind: Secret metadata: - name: {{ include "clearml.name" . }}-ac + name: {{ include "clearmlAgent.name" . }}-ac data: agentk8sglue_key: {{ .Values.clearml.agentk8sglueKey | b64enc }} agentk8sglue_secret: {{ .Values.clearml.agentk8sglueSecret | b64enc }} @@ -12,7 +12,7 @@ data: apiVersion: v1 kind: Secret metadata: - name: {{ include "clearml.name" . }}-ark + name: {{ include "clearmlAgent.name" . }}-ark type: kubernetes.io/dockerconfigjson data: .dockerconfigjson: {{ template "imagePullSecret" . }} diff --git a/charts/clearml-agent/templates/service-secret.yaml b/charts/clearml-agent/templates/service-secret.yaml index 8434a0c..40bcb36 100644 --- a/charts/clearml-agent/templates/service-secret.yaml +++ b/charts/clearml-agent/templates/service-secret.yaml @@ -2,7 +2,7 @@ apiVersion: v1 kind: Secret metadata: - name: {{ include "clearml.name" . }}-afm + name: {{ include "clearmlAgent.name" . }}-afm data: {{- range .Values.agentk8sglue.fileMounts }} {{ .name }}: {{ .fileContent | b64enc }} @@ -14,7 +14,7 @@ data: apiVersion: v1 kind: Secret metadata: - name: {{ include "clearml.name" . }}-fm + name: {{ include "clearmlAgent.name" . }}-fm data: {{- range .Values.agentk8sglue.basePodTemplate.fileMounts }} {{ .name }}: {{ .fileContent | b64enc }} @@ -26,7 +26,7 @@ data: apiVersion: v1 kind: Secret metadata: - name: {{ include "clearml.name" $ }}-{{ $key }}-fm + name: {{ include "clearmlAgent.name" $ }}-{{ $key }}-fm data: {{- range .templateOverrides.fileMounts }} {{ .name }}: {{ .fileContent | b64enc }} diff --git a/charts/clearml-agent/templates/service-sessions.yaml b/charts/clearml-agent/templates/service-sessions.yaml index 3602291..87c1b29 100644 --- a/charts/clearml-agent/templates/service-sessions.yaml +++ b/charts/clearml-agent/templates/service-sessions.yaml @@ -7,7 +7,7 @@ kind: Service metadata: name: clearml-session-{{ . }} labels: - {{- include "clearml.labels" $ | nindent 4 }} + {{- include "clearmlAgent.labels" $ | nindent 4 }} {{- with $.Values.sessions.svcAnnotations }} annotations: {{- toYaml . | nindent 4 }}