- Deployment pipeline no longer initializes resource if it is disabled.

waterbear-cloud · Mar 15, 2021 · df4955e · df4955e
1 parent 5a991de
commit df4955e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/paco/application/reseng_deploymentpipeline.py b/src/paco/application/reseng_deploymentpipeline.py
@@ -514,6 +514,8 @@ def init_stage_action(self, stage, action):
             method(stage, action)
 
     def init_resource(self):
+        if self.pipeline.is_enabled() == False:
+            return
         self.pipeline.resolve_ref_obj = self
         self.pipeline.configuration.resolve_ref_obj = self
         self.pipeline_account_ctx = self.paco_ctx.get_account_context(self.pipeline.configuration.account)

diff --git a/src/paco/cftemplates/asg.py b/src/paco/cftemplates/asg.py
@@ -332,7 +332,6 @@ def __init__(
                     continue
 
                 if ecr_deploy and len(ecr_deploy.release_phase.ecs) > 0:
-                    idx = 0
                     pull_repos = []
                     push_repos = []
                     for repository in ecr_deploy.repositories:
@@ -377,6 +376,7 @@ def __init__(
                         )
                     )
                     iam_cluster_cache = []
+                    idx = 0
                     for command in ecr_deploy.release_phase.ecs:
                         service_obj = get_model_obj_from_ref(command.service, self.paco_ctx.project)
                         ecs_services_obj = get_parent_by_interface(service_obj, schemas.IECSServices)
@@ -457,7 +457,7 @@ def __init__(
                         Statement(
                             Sid='ECSReleasePhaseSSMAutomationExecution',
                             Effect=Allow,
-                            Action=[
+                              Action=[
                                 Action('ssm', 'StartAutomationExecution'),
                                 Action('ssm', 'StopAutomationExecution'),
                                 Action('ssm', 'GetAutomationExecution'),