{"action":"created","issue":{"url":"https://api.github.com/repos/convox/rack/issues/550","repository_url":"https://api.github.com/repos/convox/rack","labels_url":"https://api.github.com/repos/convox/rack/issues/550/labels{/name}","comments_url":"https://api.github.com/repos/convox/rack/issues/550/comments","events_url":"https://api.github.com/repos/convox/rack/issues/550/events","html_url":"https://github.com/convox/rack/pull/550","id":148801941,"number":550,"title":"amzn-ami-2016.03.a-amazon-ecs-optimized + cloud-init","user":{"login":"bobzoller","id":16850,"avatar_url":"https://avatars.githubusercontent.com/u/16850?v=3","gravatar_id":"","url":"https://api.github.com/users/bobzoller","html_url":"https://github.com/bobzoller","followers_url":"https://api.github.com/users/bobzoller/followers","following_url":"https://api.github.com/users/bobzoller/following{/other_user}","gists_url":"https://api.github.com/users/bobzoller/gists{/gist_id}","starred_url":"https://api.github.com/users/bobzoller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bobzoller/subscriptions","organizations_url":"https://api.github.com/users/bobzoller/orgs","repos_url":"https://api.github.com/users/bobzoller/repos","events_url":"https://api.github.com/users/bobzoller/events{/privacy}","received_events_url":"https://api.github.com/users/bobzoller/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-04-16T01:05:42Z","updated_at":"2016-04-18T17:22:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/convox/rack/pulls/550","html_url":"https://github.com/convox/rack/pull/550","diff_url":"https://github.com/convox/rack/pull/550.diff","patch_url":"https://github.com/convox/rack/pull/550.patch"},"body":"@nzoschke I took a pass at #522 by using cloud-init as you hinted in #517.\r\n\r\ncloud-init's `bootcmd` runs _before_ `write_files`, which made for a much gnarlier patch.  `runcmd` however, runs after `write_files` but before services start, so it seems we can get away without `bootcmd`.\r\n\r\nI chose to keep `InstanceBootCommand` and the `cfn-{init,signal}` in a separate file that runs after services are started, which preserves the existing behavior.\r\n\r\nI could have gone a bit more minimal on the patch by leaving the swap and yum commands as bash, but I chose to leverage cloud-init's features instead.  Happy to reconsider that.\r\n\r\nfixes #522 \r\n\r\n## Release Playbook\r\n- [ ] Rebase against master\r\n- [ ] Release branch ()\r\n- [ ] Pass CI ()\r\n- [ ] Code review\r\n- [ ] Merge into master\r\n- [ ] Release master ()\r\n- [ ] Pass CI ()\r\n- [ ] Update staging rack\r\n- [ ] Edit release record in GitHub\r\n- [ ] Publish release\r\n- [ ] Release CLI"},"comment":{"url":"https://api.github.com/repos/convox/rack/issues/comments/211484026","html_url":"https://github.com/convox/rack/pull/550#issuecomment-211484026","issue_url":"https://api.github.com/repos/convox/rack/issues/550","id":211484026,"user":{"login":"bobzoller","id":16850,"avatar_url":"https://avatars.githubusercontent.com/u/16850?v=3","gravatar_id":"","url":"https://api.github.com/users/bobzoller","html_url":"https://github.com/bobzoller","followers_url":"https://api.github.com/users/bobzoller/followers","following_url":"https://api.github.com/users/bobzoller/following{/other_user}","gists_url":"https://api.github.com/users/bobzoller/gists{/gist_id}","starred_url":"https://api.github.com/users/bobzoller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bobzoller/subscriptions","organizations_url":"https://api.github.com/users/bobzoller/orgs","repos_url":"https://api.github.com/users/bobzoller/repos","events_url":"https://api.github.com/users/bobzoller/events{/privacy}","received_events_url":"https://api.github.com/users/bobzoller/received_events","type":"User","site_admin":false},"created_at":"2016-04-18T17:22:34Z","updated_at":"2016-04-18T17:22:34Z","body":"Thanks for the pointer, @mwarkentin.  It does appear the built-in exclude of `kernel` predated this new ECS base image, so I'll guess adding `kernel*` is still necessary.  Added it back in 727bcd7."}}