Skip to content

Commit

Permalink
Merge pull request #3895 from br0ch0n/rds_pub_access_param
Browse files Browse the repository at this point in the history
Issue #3894 RDS publicly_accessible param shouldn't force new resource
  • Loading branch information
jen20 committed Nov 25, 2015
2 parents 183fc5c + 6875e9a commit c262bc7
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion builtin/providers/aws/resource_aws_db_instance.go
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ func resourceAwsDbInstance() *schema.Resource {
"publicly_accessible": &schema.Schema{
Type: schema.TypeBool,
Optional: true,
ForceNew: true,
Computed: true,
},

"vpc_security_group_ids": &schema.Schema{
Expand Down Expand Up @@ -705,6 +705,11 @@ func resourceAwsDbInstanceUpdate(d *schema.ResourceData, meta interface{}) error
req.MultiAZ = aws.Bool(d.Get("multi_az").(bool))
requestUpdate = true
}
if d.HasChange("publicly_accessible") {
d.SetPartial("publicly_accessible")
req.PubliclyAccessible = aws.Bool(d.Get("publicly_accessible").(bool))
requestUpdate = true
}
if d.HasChange("storage_type") {
d.SetPartial("storage_type")
req.StorageType = aws.String(d.Get("storage_type").(string))
Expand Down

0 comments on commit c262bc7

Please sign in to comment.