{"action":"created","issue":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/1005","labels_url":"https://api.github.com/repos/hashicorp/terraform/issues/1005/labels{/name}","comments_url":"https://api.github.com/repos/hashicorp/terraform/issues/1005/comments","events_url":"https://api.github.com/repos/hashicorp/terraform/issues/1005/events","html_url":"https://github.com/hashicorp/terraform/pull/1005","id":58150746,"number":1005,"title":"Revert \"Merge pull request #804 from PeoplePerHour/f-aws-elb-subnet\"","user":{"login":"phinze","id":37534,"avatar_url":"https://avatars.githubusercontent.com/u/37534?v=3","gravatar_id":"","url":"https://api.github.com/users/phinze","html_url":"https://github.com/phinze","followers_url":"https://api.github.com/users/phinze/followers","following_url":"https://api.github.com/users/phinze/following{/other_user}","gists_url":"https://api.github.com/users/phinze/gists{/gist_id}","starred_url":"https://api.github.com/users/phinze/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phinze/subscriptions","organizations_url":"https://api.github.com/users/phinze/orgs","repos_url":"https://api.github.com/users/phinze/repos","events_url":"https://api.github.com/users/phinze/events{/privacy}","received_events_url":"https://api.github.com/users/phinze/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-02-19T00:20:11Z","updated_at":"2015-02-19T17:34:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hashicorp/terraform/pulls/1005","html_url":"https://github.com/hashicorp/terraform/pull/1005","diff_url":"https://github.com/hashicorp/terraform/pull/1005.diff","patch_url":"https://github.com/hashicorp/terraform/pull/1005.patch"},"body":"Unfortunately, the acceptance tests here were improperly passing, and\r\nallowing Subnet updates on ELBs is not as straightfoward as simply\r\nremoving `ForceNew`.\r\n\r\nSubnets on ELBs need to be managed by two explicit API calls:\r\n\r\n * `AttachLoadBalancerToSubnets` - http://bit.ly/elbattachsubnet\r\n * `DetachLoadBalanceFromSubnets` - http://bit.ly/elbdetachsubnet\r\n\r\nWe'll need to circle back and use these APIs to explicitly add support.\r\n\r\nThis fixes the failure of `TestAccAWSELB_AddSubnet` by removing the\r\ntest.\r\n\r\nThis reverts commit 61e91017befb8b61e394776c53abbb675ac09f14, reversing\r\nchanges made to 49b3afe45201a56bd154517cf1a9a4c753156659."},"comment":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/comments/75097641","html_url":"https://github.com/hashicorp/terraform/pull/1005#issuecomment-75097641","issue_url":"https://api.github.com/repos/hashicorp/terraform/issues/1005","id":75097641,"user":{"login":"mitchellh","id":1299,"avatar_url":"https://avatars.githubusercontent.com/u/1299?v=3","gravatar_id":"","url":"https://api.github.com/users/mitchellh","html_url":"https://github.com/mitchellh","followers_url":"https://api.github.com/users/mitchellh/followers","following_url":"https://api.github.com/users/mitchellh/following{/other_user}","gists_url":"https://api.github.com/users/mitchellh/gists{/gist_id}","starred_url":"https://api.github.com/users/mitchellh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mitchellh/subscriptions","organizations_url":"https://api.github.com/users/mitchellh/orgs","repos_url":"https://api.github.com/users/mitchellh/repos","events_url":"https://api.github.com/users/mitchellh/events{/privacy}","received_events_url":"https://api.github.com/users/mitchellh/received_events","type":"User","site_admin":false},"created_at":"2015-02-19T17:34:23Z","updated_at":"2015-02-19T17:34:23Z","body":"@pmoust It won't because the subnet won't change. If the subnet changes it will break it, yes. But that didn't work to begin with. The acceptance test was testing that behavior and was definitely not passing. "}}