From d7a48dd65bd325708f0b4647a65551c1a7d0c3ac Mon Sep 17 00:00:00 2001
From: GraemeKnights <email@graemeknights.com>
Date: Fri, 19 Mar 2021 16:22:00 +0800
Subject: [PATCH] Use EnvVar values for updating stacksets

Alter the update_stack_set function to use self.failed_tolerance_percent and self.max_concurrent_percent (which are both sourced from OS EnvVars) like the update_stack_instances and delete_stack_set functions do
---
 source/aws/services/cloudformation.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/source/aws/services/cloudformation.py b/source/aws/services/cloudformation.py
index d7852b8..9116db2 100644
--- a/source/aws/services/cloudformation.py
+++ b/source/aws/services/cloudformation.py
@@ -253,8 +253,7 @@ def update_stack_instances(self, stack_set_name, account_list, region_list,
                 raise
 
     def update_stack_set(self, stack_set_name, parameter, template_url,
-                         capabilities, failed_tolerance_percent=0,
-                         max_concurrent_percent=100):
+                         capabilities):
         try:
             parameters = []
             param_dict = {}
@@ -280,8 +279,8 @@ def update_stack_set(self, stack_set_name, parameter, template_url,
                     'ADMINISTRATION_ROLE_ARN'),
                 ExecutionRoleName=os.environ.get('EXECUTION_ROLE_NAME'),
                 OperationPreferences={
-                    'FailureTolerancePercentage': failed_tolerance_percent,
-                    'MaxConcurrentPercentage': max_concurrent_percent
+                    'FailureTolerancePercentage': self.failed_tolerance_percent,
+                    'MaxConcurrentPercentage': self.max_concurrent_percent
                 }
             )
             return response