Merge pull request #276 from NASA-IMPACT/debug_bucket_option

Make debug_bucket argument optional for state machines.
NASA-IMPACT · May 7, 2024 · e090dc0 · e090dc0
2 parents 1b63b27 + d6a484a
commit e090dc0
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 8 deletions.
diff --git a/stack/hlsconstructs/landsat_mgrs_step_function.py b/stack/hlsconstructs/landsat_mgrs_step_function.py
@@ -21,7 +21,7 @@ def __init__(
         mgrs_logger: Lambda,
         get_random_wait: Lambda,
         gibs_outputbucket: str,
-        debug_bucket: Union[bool, str],
+        debug_bucket: Union[bool, str] = False,
         **kwargs,
     ) -> None:
         super().__init__(scope, id, **kwargs)

diff --git a/stack/hlsconstructs/landsat_step_function.py b/stack/hlsconstructs/landsat_step_function.py
@@ -25,7 +25,7 @@ def __init__(
         get_random_wait: Lambda,
         replace_existing: bool,
         landsat_mgrs_step_function_arn: str,
-        debug_bucket: Union[bool, str],
+        debug_bucket: Union[bool, str] = False,
         **kwargs,
     ) -> None:
         super().__init__(scope, id, **kwargs)

diff --git a/stack/hlsconstructs/sentinel_errors_step_function.py b/stack/hlsconstructs/sentinel_errors_step_function.py
@@ -20,7 +20,7 @@ def __init__(
         update_sentinel_failure: Lambda,
         get_random_wait: Lambda,
         gibs_outputbucket: str,
-        debug_bucket: Union[bool, str],
+        debug_bucket: Union[bool, str] = False,
         **kwargs,
     ) -> None:
         super().__init__(scope, id, **kwargs)

diff --git a/stack/hlsconstructs/sentinel_step_function.py b/stack/hlsconstructs/sentinel_step_function.py
@@ -23,7 +23,7 @@ def __init__(
         check_exit_code: Lambda,
         replace_existing: bool,
         gibs_outputbucket: str,
-        debug_bucket: Union[bool, str],
+        debug_bucket: Union[bool, str] = False,
         **kwargs,
     ) -> None:
         super().__init__(scope, id, **kwargs)

diff --git a/stack/stack.py b/stack/stack.py
@@ -595,6 +595,7 @@ def __init__(self, scope: core.Construct, id: str, **kwargs) -> None:
             outputbucket_role_arn=OUTPUT_BUCKET_ROLE_ARN,
             gibs_outputbucket=GIBS_OUTPUT_BUCKET,
             get_random_wait=self.get_random_wait,
+            debug_bucket=DEBUG_BUCKET,
         )
 
         self.sentinel_errors_step_function_historic = SentinelErrorsStepFunction(
@@ -638,8 +639,6 @@ def __init__(self, scope: core.Construct, id: str, **kwargs) -> None:
             pr2mgrs=self.pr2mgrs_lambda,
             mgrs_logger=self.mgrs_logger,
             get_random_wait=self.get_random_wait,
-            gibs_outputbucket=GIBS_OUTPUT_BUCKET,
-            debug_bucket=DEBUG_BUCKET,
         )
 
         self.landsat_mgrs_step_function_historic = LandsatMGRSStepFunction(
@@ -654,7 +653,6 @@ def __init__(self, scope: core.Construct, id: str, **kwargs) -> None:
             pr2mgrs=self.pr2mgrs_lambda,
             mgrs_logger=self.mgrs_logger,
             get_random_wait=self.get_random_wait,
-            gibs_outputbucket=GIBS_OUTPUT_BUCKET_HISTORIC,
         )
 
         self.landsat_mgrs_partials_step_function_historic = (
@@ -670,7 +668,6 @@ def __init__(self, scope: core.Construct, id: str, **kwargs) -> None:
                 pr2mgrs=self.pr2mgrs_lambda,
                 mgrs_logger=self.mgrs_logger,
                 get_random_wait=self.get_random_wait,
-                gibs_outputbucket=GIBS_OUTPUT_BUCKET_HISTORIC,
             )
         )