{"action":"created","issue":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/1618","labels_url":"https://api.github.com/repos/hashicorp/terraform/issues/1618/labels{/name}","comments_url":"https://api.github.com/repos/hashicorp/terraform/issues/1618/comments","events_url":"https://api.github.com/repos/hashicorp/terraform/issues/1618/events","html_url":"https://github.com/hashicorp/terraform/pull/1618","id":69964758,"number":1618,"title":"provider/aws: Fix issue with updating VPC Security Group IDs for an Instance","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},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-04-21T22:12:04Z","updated_at":"2015-04-28T18:43:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hashicorp/terraform/pulls/1618","html_url":"https://github.com/hashicorp/terraform/pull/1618","diff_url":"https://github.com/hashicorp/terraform/pull/1618.diff","patch_url":"https://github.com/hashicorp/terraform/pull/1618.patch"},"body":"Currently, we aren't correctly setting the ids, and are setting both\r\n`security_groups` and `vpc_security_group_ids` in the state. As a result, we really only use\r\nthe former, but the latter never goes away in the state file :frowning: \r\n\r\nWe also don't actually update the remote security groups in the `update` method.\r\n\r\nThis PR fixes both issues, correctly reading `security_groups` vs.\r\n`vpc_security_group_ids` and allows users to update the latter without\r\ndestroying the Instance when in a VPC.\r\n\r\nShould fix #1611"},"comment":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/comments/97166577","html_url":"https://github.com/hashicorp/terraform/pull/1618#issuecomment-97166577","issue_url":"https://api.github.com/repos/hashicorp/terraform/issues/1618","id":97166577,"user":{"login":"mzupan","id":205373,"avatar_url":"https://avatars.githubusercontent.com/u/205373?v=3","gravatar_id":"","url":"https://api.github.com/users/mzupan","html_url":"https://github.com/mzupan","followers_url":"https://api.github.com/users/mzupan/followers","following_url":"https://api.github.com/users/mzupan/following{/other_user}","gists_url":"https://api.github.com/users/mzupan/gists{/gist_id}","starred_url":"https://api.github.com/users/mzupan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mzupan/subscriptions","organizations_url":"https://api.github.com/users/mzupan/orgs","repos_url":"https://api.github.com/users/mzupan/repos","events_url":"https://api.github.com/users/mzupan/events{/privacy}","received_events_url":"https://api.github.com/users/mzupan/received_events","type":"User","site_admin":false},"created_at":"2015-04-28T18:43:02Z","updated_at":"2015-04-28T18:43:02Z","body":"works great for me :+1: "}}