helm(fix): correct analysistemplate invocation

Sefaria · Jul 4, 2023 · 5a08a0c · 5a08a0c
1 parent eb261e5
commit 5a08a0c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/helm-chart/sefaria-project/templates/analysistemplate/rollout-complete.yaml b/helm-chart/sefaria-project/templates/analysistemplate/rollout-complete.yaml
@@ -6,7 +6,9 @@ metadata:
     deployEnv: "{{ .Values.deployEnv }}"
 spec:
   args:
-    - name: healthcheck-hostname
+    - name: revision
+    - name: deployment
+    - name: version
   metrics:
     - name: healthcheck-ready
       provider:

diff --git a/helm-chart/sefaria-project/templates/rollout/web.yaml b/helm-chart/sefaria-project/templates/rollout/web.yaml
@@ -26,11 +26,11 @@ spec:
       {{- if eq .Values.sandbox "false" }} 
       postPromotionAnalysis:
         templates: 
-          - templateName: rollout-complete
+          - templateName: rollout-complete-{{ .Values.deployEnv }}
         args:
           - name: revision
             value: "{{ .Release.Revision }}"
-          - name: deployEnv
+          - name: deployment
             value: {{ .Values.deployEnv }}
           - name: version
             value: {{ .Values.web.containerImage.tag }}