{"push_id":1090193606,"size":7,"distinct_size":4,"ref":"refs/heads/master","head":"b74905f11b55826b9028d8627cb33be2d3dbbee7","before":"77d924914843f630f236b3d9c7314cf3eba8a536","commits":[{"sha":"f339d0d1811c8d1ebae2eb15be2a80ace44356ea","author":{"email":"824a461fb8802d7c67c8546b57082d830e462940@wooh.hu","name":"Adam Papai"},"message":"Fix tf/tfstate generation for security groups\n\nAdjusted the tests and both the tf/tfstate exports according to the statements\nbelow. The tests should be improved too, for now they're ok.\n\nNote that if it's a VPC security group, the group name is nil in the permission\nset.\n\n EC2-Classic security groups can be:\n\n  - security groups from the same account\n  - security groups from another account\n  - CIDR\n\nIf it is an EC2-Classic security group from the same account, the rule should\ncontain only the security group name not the group id.\n\nExample tf export snippet: `security_groups = [\"sg-1234efgh\"]`\n\nIf it's an EC2-Classic security group from another account, the role should\ncontain the account id and the security group name, not the id.\n\nExample tf export snippet: `security_groups = [\"098765432109/sg-1234efgh\"]`\n\nVPC security groups can be:\n  - security groups from the same account + same VPC\n  - CIDR\n\nA rule inside of a VPC security group can't be SGs from a different VPC or from\na differert account. Only CIDRs and SGs from the same VPC.\n\nA rule should contain only the security group ID or a CIDR.\n\nExample tf export snippet: `security_groups = [\"sg-1234efgh\"]`\n\nThe\n[documentation](https://www.terraform.io/docs/providers/aws/r/security_group.html) says:\n\n```\nsecurity_groups - (Optional)\nList of security group Group Names if using EC2-Classic or the default VPC, or\nGroup IDs if using a non-default VPC.\n```","distinct":false,"url":"https://api.github.com/repos/woohgit/terraforming/commits/f339d0d1811c8d1ebae2eb15be2a80ace44356ea"},{"sha":"ddf9907d116716bf67f456b8c4780eba5bddab4f","author":{"email":"824a461fb8802d7c67c8546b57082d830e462940@wooh.hu","name":"Adam Papai"},"message":"Remove space fixes","distinct":false,"url":"https://api.github.com/repos/woohgit/terraforming/commits/ddf9907d116716bf67f456b8c4780eba5bddab4f"},{"sha":"116cc34dd5a4f44bc5ad4b48f7edecb5ca325ad7","author":{"email":"824a461fb8802d7c67c8546b57082d830e462940@wooh.hu","name":"Adam Papai"},"message":"Fix and/next","distinct":false,"url":"https://api.github.com/repos/woohgit/terraforming/commits/116cc34dd5a4f44bc5ad4b48f7edecb5ca325ad7"},{"sha":"ace7e5031e7fe3e20954c619c06a67741a83fe24","author":{"email":"ffe67e04cad0f509a3c518d430ab39c6b335877c@gmail.com","name":"Jason Miller"},"message":"Include description field for IAM Policies\n\nWithout this, terraform will attempt to destroy and recreate the\npolicies, which will fail if they're attached to anything already.","distinct":true,"url":"https://api.github.com/repos/woohgit/terraforming/commits/ace7e5031e7fe3e20954c619c06a67741a83fe24"},{"sha":"c4d9b5b1ee6303d6c3b17a3604b3339b3518fc64","author":{"email":"ffe67e04cad0f509a3c518d430ab39c6b335877c@gmail.com","name":"Jason Miller"},"message":"Update iam_policy_spec to include description field","distinct":true,"url":"https://api.github.com/repos/woohgit/terraforming/commits/c4d9b5b1ee6303d6c3b17a3604b3339b3518fc64"},{"sha":"62c1bb25d88053871cf70843f9905219ad328f1d","author":{"email":"a82155263ff6c87af20b403c031cb8d122ade2c2@gmail.com","name":"Daisuke Fujita"},"message":"Merge pull request #200 from woohgit/security_groups_generate_valid_output\n\nFix tf/tfstate output for security groups","distinct":true,"url":"https://api.github.com/repos/woohgit/terraforming/commits/62c1bb25d88053871cf70843f9905219ad328f1d"},{"sha":"b74905f11b55826b9028d8627cb33be2d3dbbee7","author":{"email":"a82155263ff6c87af20b403c031cb8d122ade2c2@gmail.com","name":"Daisuke Fujita"},"message":"Merge pull request #203 from stormbeta/iam_improvements\n\nInclude description field for IAM Policies","distinct":true,"url":"https://api.github.com/repos/woohgit/terraforming/commits/b74905f11b55826b9028d8627cb33be2d3dbbee7"}]}