Merge pull request #1287 from captainkerk/autoscaling-update-action-fix

update_auto_scaling_group method incorrectly supports LoadBalancerNames
This commit is contained in:
Jack Danger 2017-10-23 09:42:19 -07:00 committed by GitHub
commit 8bf0ed697a
2 changed files with 4 additions and 5 deletions

View File

@ -229,7 +229,7 @@ class FakeAutoScalingGroup(BaseModel):
def update(self, availability_zones, desired_capacity, max_size, min_size,
launch_config_name, vpc_zone_identifier, default_cooldown,
health_check_period, health_check_type, load_balancers,
health_check_period, health_check_type,
placement_group, termination_policies):
if availability_zones:
self.availability_zones = availability_zones
@ -406,13 +406,13 @@ class AutoScalingBackend(BaseBackend):
desired_capacity, max_size, min_size,
launch_config_name, vpc_zone_identifier,
default_cooldown, health_check_period,
health_check_type, load_balancers,
placement_group, termination_policies):
health_check_type, placement_group,
termination_policies):
group = self.autoscaling_groups[name]
group.update(availability_zones, desired_capacity, max_size,
min_size, launch_config_name, vpc_zone_identifier,
default_cooldown, health_check_period, health_check_type,
load_balancers, placement_group, termination_policies)
placement_group, termination_policies)
return group
def describe_autoscaling_groups(self, names):

View File

@ -189,7 +189,6 @@ class AutoScalingResponse(BaseResponse):
default_cooldown=self._get_int_param('DefaultCooldown'),
health_check_period=self._get_int_param('HealthCheckGracePeriod'),
health_check_type=self._get_param('HealthCheckType'),
load_balancers=self._get_multi_param('LoadBalancerNames.member'),
placement_group=self._get_param('PlacementGroup'),
termination_policies=self._get_multi_param(
'TerminationPolicies.member'),