bugfix: this feature flag isnt used by airflow, prefer checking a hail env var

bpblanken · bpblanken · commit fdaef5cf2554 · 2025-01-15T19:03:00.000-05:00
diff --git a/v03_pipeline/lib/model/constants.py b/v03_pipeline/lib/model/constants.py
@@ -1,4 +1,4 @@
-from v03_pipeline.lib.model.feature_flag import FeatureFlag
+import os
 
 LOCAL_DISK_MOUNT_PATH = '/var/seqr'
 PROJECTS_EXCLUDED_FROM_LOOKUP = {
@@ -15,11 +15,11 @@
 }
 GRCH37_TO_GRCH38_LIFTOVER_REF_PATH = (
     'gs://hail-common/references/grch37_to_grch38.over.chain.gz'
-    if FeatureFlag.RUN_PIPELINE_ON_DATAPROC
+    if os.environ.get('HAIL_DATAPROC') == '1'
     else 'v03_pipeline/var/liftover/grch37_to_grch38.over.chain.gz'
 )
 GRCH38_TO_GRCH37_LIFTOVER_REF_PATH = (
     'gs://hail-common/references/grch38_to_grch37.over.chain.gz'
-    if FeatureFlag.RUN_PIPELINE_ON_DATAPROC
+    if os.environ.get('HAIL_DATAPROC') == '1'
     else 'v03_pipeline/var/liftover/grch38_to_grch37.over.chain.gz'
 )

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-from v03_pipeline.lib.model.feature_flag import FeatureFlag`
	`1`	`+import os`
`2`	`2`
`3`	`3`	`LOCAL_DISK_MOUNT_PATH = '/var/seqr'`
`4`	`4`	`PROJECTS_EXCLUDED_FROM_LOOKUP = {`
`@@ -15,11 +15,11 @@`
`15`	`15`	`}`
`16`	`16`	`GRCH37_TO_GRCH38_LIFTOVER_REF_PATH = (`
`17`	`17`	`'gs://hail-common/references/grch37_to_grch38.over.chain.gz'`
`18`		`- if FeatureFlag.RUN_PIPELINE_ON_DATAPROC`
	`18`	`+ if os.environ.get('HAIL_DATAPROC') == '1'`
`19`	`19`	`else 'v03_pipeline/var/liftover/grch37_to_grch38.over.chain.gz'`
`20`	`20`	`)`
`21`	`21`	`GRCH38_TO_GRCH37_LIFTOVER_REF_PATH = (`
`22`	`22`	`'gs://hail-common/references/grch38_to_grch37.over.chain.gz'`
`23`		`- if FeatureFlag.RUN_PIPELINE_ON_DATAPROC`
	`23`	`+ if os.environ.get('HAIL_DATAPROC') == '1'`
`24`	`24`	`else 'v03_pipeline/var/liftover/grch38_to_grch37.over.chain.gz'`
`25`	`25`	`)`