{"action":"created","issue":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/9515","repository_url":"https://api.github.com/repos/hashicorp/terraform","labels_url":"https://api.github.com/repos/hashicorp/terraform/issues/9515/labels{/name}","comments_url":"https://api.github.com/repos/hashicorp/terraform/issues/9515/comments","events_url":"https://api.github.com/repos/hashicorp/terraform/issues/9515/events","html_url":"https://github.com/hashicorp/terraform/pull/9515","id":184533426,"number":9515,"title":"provider/aws: aws_redshift_subnet_group allows description to be modified","user":{"login":"dennis-bsi","id":16431200,"avatar_url":"https://avatars.githubusercontent.com/u/16431200?v=3","gravatar_id":"","url":"https://api.github.com/users/dennis-bsi","html_url":"https://github.com/dennis-bsi","followers_url":"https://api.github.com/users/dennis-bsi/followers","following_url":"https://api.github.com/users/dennis-bsi/following{/other_user}","gists_url":"https://api.github.com/users/dennis-bsi/gists{/gist_id}","starred_url":"https://api.github.com/users/dennis-bsi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dennis-bsi/subscriptions","organizations_url":"https://api.github.com/users/dennis-bsi/orgs","repos_url":"https://api.github.com/users/dennis-bsi/repos","events_url":"https://api.github.com/users/dennis-bsi/events{/privacy}","received_events_url":"https://api.github.com/users/dennis-bsi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-21T17:17:11Z","updated_at":"2016-10-21T19:32:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hashicorp/terraform/pulls/9515","html_url":"https://github.com/hashicorp/terraform/pull/9515","diff_url":"https://github.com/hashicorp/terraform/pull/9515.diff","patch_url":"https://github.com/hashicorp/terraform/pull/9515.patch"},"body":"Updating the description of aws_redshift_subnet_group would show in a plan, and update tfstate, but never update the value on AWS.\r\n```\r\n/tmp/tfissue $ cat rs_sg.tf\r\nprovider \"aws\" {\r\n    region = \"us-west-2\"\r\n}\r\n\r\nresource \"aws_redshift_subnet_group\" \"test\" {\r\n    name = \"redshift-test\"\r\n    description = \"My description\"\r\n    subnet_ids  = [ \"REDACTED\" ]\r\n}\r\n\r\n/tmp/tfissue $ ./terraform apply\r\naws_redshift_subnet_group.test: Creating...\r\n  description:           \"\" => \"My description\"\r\n  name:                  \"\" => \"redshift-test\"\r\n  subnet_ids.#:          \"\" => \"1\"\r\n  subnet_ids.3988262476: \"\" => \"REDACTED\"\r\naws_redshift_subnet_group.test: Creation complete\r\n\r\nApply complete! Resources: 1 added, 0 changed, 0 destroyed.\r\n\r\n/tmp/tfissue $ aws redshift describe-cluster-subnet-groups\r\n{\r\n    \"ClusterSubnetGroups\": [\r\n        {\r\nprovider \"aws\" {\r\n            \"Description\": \"My description\",\r\n            \"ClusterSubnetGroupName\": \"redshift-test\"\r\n        }\r\n    ]\r\n}\r\n\r\n/tmp/tfissue $ vi rs_sg.tf\r\n...changing description in file...\r\n\r\n/tmp/tfissue $ ./terraform plan\r\naws_redshift_subnet_group.test: Refreshing state... (ID: redshift-test)\r\n\r\n~ aws_redshift_subnet_group.test\r\n    description: \"My description\" => \"Totally different\"\r\n\r\nPlan: 0 to add, 1 to change, 0 to destroy.\r\n\r\n/tmp/tfissue $ ./terraform apply\r\naws_redshift_subnet_group.test: Refreshing state... (ID: redshift-test)\r\naws_redshift_subnet_group.test: Modifying...\r\n  description: \"My description\" => \"Totally different\"\r\naws_redshift_subnet_group.test: Modifications complete\r\n\r\nApply complete! Resources: 0 added, 1 changed, 0 destroyed.\r\n\r\n/tmp/tfissue $ aws redshift describe-cluster-subnet-groups\r\n{\r\n    \"ClusterSubnetGroups\": [\r\n        {\r\n            \"Description\": \"My description\",\r\n            \"ClusterSubnetGroupName\": \"redshift-test\"\r\n        }\r\n    ]\r\n}\r\n```\r\n\r\nAcceptance tests\r\n```\r\nmake testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSRedshiftSubnetGroup_updateDescription'\r\n==> Checking that code complies with gofmt requirements...\r\ngo generate $(go list ./... | grep -v /terraform/vendor/)\r\n2016/10/21 12:12:09 Generated command/internal_plugin_list.go\r\nTF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSRedshiftSubnetGroup_updateDescription -timeout 120m\r\n=== RUN   TestAccAWSRedshiftSubnetGroup_updateDescription\r\n--- PASS: TestAccAWSRedshiftSubnetGroup_updateDescription (50.29s)\r\nPASS\r\nok  \tgithub.com/hashicorp/terraform/builtin/providers/aws\t50.330s\r\n```"},"comment":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/comments/255447278","html_url":"https://github.com/hashicorp/terraform/pull/9515#issuecomment-255447278","issue_url":"https://api.github.com/repos/hashicorp/terraform/issues/9515","id":255447278,"user":{"login":"catsby","id":61721,"avatar_url":"https://avatars.githubusercontent.com/u/61721?v=3","gravatar_id":"","url":"https://api.github.com/users/catsby","html_url":"https://github.com/catsby","followers_url":"https://api.github.com/users/catsby/followers","following_url":"https://api.github.com/users/catsby/following{/other_user}","gists_url":"https://api.github.com/users/catsby/gists{/gist_id}","starred_url":"https://api.github.com/users/catsby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/catsby/subscriptions","organizations_url":"https://api.github.com/users/catsby/orgs","repos_url":"https://api.github.com/users/catsby/repos","events_url":"https://api.github.com/users/catsby/events{/privacy}","received_events_url":"https://api.github.com/users/catsby/received_events","type":"User","site_admin":false},"created_at":"2016-10-21T19:32:32Z","updated_at":"2016-10-21T19:32:32Z","body":"LGTM, thanks!\r\n\r\n```\r\nTF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSRedshiftSubnetGroup_updateDescription -timeout 120m\r\n=== RUN   TestAccAWSRedshiftSubnetGroup_updateDescription\r\n--- PASS: TestAccAWSRedshiftSubnetGroup_updateDescription (27.42s)\r\nPASS\r\nok  \tgithub.com/hashicorp/terraform/builtin/providers/aws\t27.460s\r\nTest:\r\n```"}}
{"actor":{"display_login":"catsby"}}