ResiLien
/
terraform
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Cleanup termination policy update logic.

This commit is contained in:
Trevor Pounds 2016-02-11 08:45:11 -08:00
parent 44b00782c3
commit b39cc55f37
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
builtin/providers/aws/resource_aws_autoscaling_group.go
View File

@ -321,18 +321,11 @@ func resourceAwsAutoscalingGroupUpdate(d *schema.ResourceData, meta interface{})
if d.HasChange("termination_policies") { if d.HasChange("termination_policies") {
// If the termination policy is set to null, we need to explicitly set // If the termination policy is set to null, we need to explicitly set
// it back to "Default", or the API won't reset it for us. // it back to "Default", or the API won't reset it for us.
// This means GetOk() will fail us on the zero check. if v, ok := d.GetOk("termination_policies"); ok && len(v.([]interface{})) > 0 {
v := d.Get("termination_policies")
if len(v.([]interface{})) > 0 {
opts.TerminationPolicies = expandStringList(v.([]interface{})) opts.TerminationPolicies = expandStringList(v.([]interface{}))
} else { } else {
// Policies is a slice of string pointers, so build one.
// Maybe there's a better idiom for this?
log.Printf("[DEBUG] Explictly setting null termination policy to 'Default'") log.Printf("[DEBUG] Explictly setting null termination policy to 'Default'")
pol := "Default" opts.TerminationPolicies = aws.StringSlice([]string{"Default"})
s := make([]*string, 1, 1)
s[0] = &pol
opts.TerminationPolicies = s
} }
} }