post release bugfixes

broadinstitute · Nov 27, 2023 · 358ddd2 · 358ddd2
1 parent d1d601f
commit 358ddd2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/charts/hail-search/templates/deployment.yaml b/charts/hail-search/templates/deployment.yaml
@@ -13,6 +13,7 @@ spec:
     metadata:
       annotations:
         checksum/datasetVersions: {{ print $.Values.datasetVersions | sha256sum }}
+        checksum/config: {{ include (print $.Template.BasePath "/environment.yaml") . | sha256sum }}
       {{- with .Values.podAnnotations }}
         {{- toYaml . | nindent 8 }}
       {{- end }}

diff --git a/charts/seqr/templates/check-for-new-samples-from-pipeline.yaml b/charts/seqr/templates/check-for-new-samples-from-pipeline.yaml
@@ -9,7 +9,7 @@ metadata:
   annotations:
     "helm.sh/hook": post-install, post-upgrade
     "helm.sh/hook-weight": "-1"
-    "helm.sh/hook-delete-policy": before-hook-creation,hook-succeeded
+    "helm.sh/hook-delete-policy": before-hook-creation
 spec:
   template:
     metadata:

diff --git a/unit_test/hail_search/test_hail_search_chart.py b/unit_test/hail_search/test_hail_search_chart.py
@@ -21,6 +21,8 @@ def test_values(self):
         p.check_returncode()
         self.assertIn('serviceAccountName: test-hail-search', p.stdout)
         self.assertIn('name: sync-datasets-grch38-snv-indel', p.stdout)
+        self.assertIn('checksum/datasetVersions', p.stdout)  
+        self.assertIn('checksum/config', p.stdout)
 
 
 if __name__ == '__main__':