{"action":"created","issue":{"url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/6321","repository_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws","labels_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/6321/labels{/name}","comments_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/6321/comments","events_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/6321/events","html_url":"https://github.com/terraform-providers/terraform-provider-aws/pull/6321","id":376481631,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI3Njg5MzI0","number":6321,"title":"service/ec2: Replace custom sorting implementation with sort.Slice()","user":{"login":"bflad","id":189114,"node_id":"MDQ6VXNlcjE4OTExNA==","avatar_url":"https://avatars3.githubusercontent.com/u/189114?v=4","gravatar_id":"","url":"https://api.github.com/users/bflad","html_url":"https://github.com/bflad","followers_url":"https://api.github.com/users/bflad/followers","following_url":"https://api.github.com/users/bflad/following{/other_user}","gists_url":"https://api.github.com/users/bflad/gists{/gist_id}","starred_url":"https://api.github.com/users/bflad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bflad/subscriptions","organizations_url":"https://api.github.com/users/bflad/orgs","repos_url":"https://api.github.com/users/bflad/repos","events_url":"https://api.github.com/users/bflad/events{/privacy}","received_events_url":"https://api.github.com/users/bflad/received_events","type":"User","site_admin":false},"labels":[{"id":794043766,"node_id":"MDU6TGFiZWw3OTQwNDM3NjY=","url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/labels/service/ec2","name":"service/ec2","color":"bfd4f2","default":false},{"id":737832193,"node_id":"MDU6TGFiZWw3Mzc4MzIxOTM=","url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/labels/size/M","name":"size/M","color":"ffffff","default":false},{"id":810203048,"node_id":"MDU6TGFiZWw4MTAyMDMwNDg=","url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/labels/technical-debt","name":"technical-debt","color":"1d76db","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/milestones/47","html_url":"https://github.com/terraform-providers/terraform-provider-aws/milestone/47","labels_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/milestones/47/labels","id":3786815,"node_id":"MDk6TWlsZXN0b25lMzc4NjgxNQ==","number":47,"title":"v1.43.0","description":"","creator":{"login":"bflad","id":189114,"node_id":"MDQ6VXNlcjE4OTExNA==","avatar_url":"https://avatars3.githubusercontent.com/u/189114?v=4","gravatar_id":"","url":"https://api.github.com/users/bflad","html_url":"https://github.com/bflad","followers_url":"https://api.github.com/users/bflad/followers","following_url":"https://api.github.com/users/bflad/following{/other_user}","gists_url":"https://api.github.com/users/bflad/gists{/gist_id}","starred_url":"https://api.github.com/users/bflad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bflad/subscriptions","organizations_url":"https://api.github.com/users/bflad/orgs","repos_url":"https://api.github.com/users/bflad/repos","events_url":"https://api.github.com/users/bflad/events{/privacy}","received_events_url":"https://api.github.com/users/bflad/received_events","type":"User","site_admin":false},"open_issues":0,"closed_issues":45,"state":"closed","created_at":"2018-11-01T00:31:00Z","updated_at":"2018-11-07T23:02:07Z","due_on":null,"closed_at":"2018-11-07T23:02:07Z"},"comments":0,"created_at":"2018-11-01T17:03:27Z","updated_at":"2018-11-09T19:32:15Z","closed_at":"2018-11-01T18:55:36Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/pulls/6321","html_url":"https://github.com/terraform-providers/terraform-provider-aws/pull/6321","diff_url":"https://github.com/terraform-providers/terraform-provider-aws/pull/6321.diff","patch_url":"https://github.com/terraform-providers/terraform-provider-aws/pull/6321.patch"},"body":"Closes #6318 \r\n\r\nOutput from acceptance testing:\r\n\r\n```\r\n--- PASS: TestAccDataSourceAwsEbsSnapshotIds_empty (10.51s)\r\n--- PASS: TestAccDataSourceAwsAmiIds_basic (11.66s)\r\n--- PASS: TestAccAWSAmiDataSource_instanceStore (14.34s)\r\n--- PASS: TestAccAWSAmiDataSource_localNameFilter (14.76s)\r\n--- PASS: TestAccAWSAmiDataSource_ownersEmpty (14.99s)\r\n--- PASS: TestAccDataSourceAwsAmiIds_empty (15.45s)\r\n--- PASS: TestAccAWSAmiDataSource_natInstance (15.62s)\r\n--- PASS: TestAccDataSourceAwsAmiIds_sorted (17.92s)\r\n--- PASS: TestAccAWSAmiDataSource_windowsInstance (18.81s)\r\n--- PASS: TestAccAWSEbsSnapshotDataSource_basic (27.87s)\r\n--- PASS: TestAccAWSEbsSnapshotDataSource_multipleFilters (28.60s)\r\n--- PASS: TestAccDataSourceAwsEbsSnapshotIds_basic (41.48s)\r\n--- PASS: TestAccDataSourceAwsEbsSnapshotIds_sorted (49.30s)\r\n--- PASS: TestAccAWSAmiDataSource_owners (59.73s)\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/comments/437470133","html_url":"https://github.com/terraform-providers/terraform-provider-aws/pull/6321#issuecomment-437470133","issue_url":"https://api.github.com/repos/terraform-providers/terraform-provider-aws/issues/6321","id":437470133,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNzQ3MDEzMw==","user":{"login":"danlamanna","id":608229,"node_id":"MDQ6VXNlcjYwODIyOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/608229?v=4","gravatar_id":"","url":"https://api.github.com/users/danlamanna","html_url":"https://github.com/danlamanna","followers_url":"https://api.github.com/users/danlamanna/followers","following_url":"https://api.github.com/users/danlamanna/following{/other_user}","gists_url":"https://api.github.com/users/danlamanna/gists{/gist_id}","starred_url":"https://api.github.com/users/danlamanna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danlamanna/subscriptions","organizations_url":"https://api.github.com/users/danlamanna/orgs","repos_url":"https://api.github.com/users/danlamanna/repos","events_url":"https://api.github.com/users/danlamanna/events{/privacy}","received_events_url":"https://api.github.com/users/danlamanna/received_events","type":"User","site_admin":false},"created_at":"2018-11-09T19:32:15Z","updated_at":"2018-11-09T19:32:15Z","author_association":"NONE","body":"I believe this presents a regression in version 1.43.0.\r\n\r\nIn an AWS account with more than 1 EBS snapshot, this can be reproduced:\r\n1) Create a terraform file with this content\r\n```\r\nprovider \"aws\" {\r\n  region = \"us-east-1\"\r\n  version = \"<1.43.0\"\r\n}\r\n\r\ndata \"aws_ebs_snapshot\" \"example\" {\r\n  most_recent = true\r\n  owners      = [\"self\"]\r\n}\r\n\r\noutput \"foo\" { value = \"${data.aws_ebs_snapshot.example.id}\" }\r\n```\r\n2) terraform init and terraform apply\r\n3) Note the snapshot ID output\r\n4) Remove local state, and the version specifier\r\n5) terraform init and terraform apply\r\n6) Note a different snapshot ID"}}
{"actor":{"display_login":"danlamanna"}}