feat: add envFrom for deployment and statefulset, add githash to char…

…t version
wandb · Oct 24, 2024 · 12044e3 · 12044e3
1 parent 40b4d37
commit 12044e3
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 1 deletion.
diff --git a/.github/workflows/pr-release.yaml b/.github/workflows/pr-release.yaml
@@ -54,8 +54,9 @@ jobs:
         env:
           PR_NUMBER: ${{ github.event.number }}
         run: |
+          git_hash=$(git rev-parse --short "$GITHUB_SHA")
           for chart in $(ct list-changed --config ct.yaml); do
-          helm package --version=$(helm show chart ${chart} | yq .version)-PR${PR_NUMBER} -d .cr-release-packages ${chart}
+          helm package --version=$(helm show chart ${chart} | yq .version)-PR${PR_NUMBER}-${git_hash} -d .cr-release-packages ${chart}
           done
         shell: bash
 

diff --git a/charts/wandb-service/templates/deployment.yaml b/charts/wandb-service/templates/deployment.yaml
@@ -33,6 +33,10 @@ spec:
         {{- toYaml .Values.podSecurityContext | nindent 8 }}
       containers:
         - name: {{ .Chart.Name }}
+          {{ with .Values.envFrom }}
+          envFrom:
+              {{- toYaml . | nindent 12 }}
+          {{- end }}
           {{ with .Values.env }}
           env:
             {{- toYaml . | nindent 12 }}

diff --git a/charts/wandb-service/templates/statefulset.yaml b/charts/wandb-service/templates/statefulset.yaml
@@ -33,6 +33,14 @@ spec:
         {{- toYaml .Values.podSecurityContext | nindent 8 }}
       containers:
         - name: {{ .Chart.Name }}
+          {{ with .Values.envFrom }}
+          envFrom:
+              {{- toYaml . | nindent 12 }}
+          {{- end }}
+          {{ with .Values.env }}
+          env:
+            {{- toYaml . | nindent 12 }}
+          {{- end }}
           securityContext:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"

diff --git a/charts/wandb-service/values.yaml b/charts/wandb-service/values.yaml
@@ -22,6 +22,7 @@ nameOverride: ""
 fullnameOverride: ""
 
 env: []
+envFrom: []
 
 # This section builds out the service account more information can be found here: https://kubernetes.io/docs/concepts/security/service-accounts/
 serviceAccount: