{"action":"created","issue":{"url":"https://api.github.com/repos/twitter/heron/issues/1420","repository_url":"https://api.github.com/repos/twitter/heron","labels_url":"https://api.github.com/repos/twitter/heron/issues/1420/labels{/name}","comments_url":"https://api.github.com/repos/twitter/heron/issues/1420/comments","events_url":"https://api.github.com/repos/twitter/heron/issues/1420/events","html_url":"https://github.com/twitter/heron/pull/1420","id":178971447,"number":1420,"title":"More configurable Aurora","user":{"login":"chris-pardy","id":5663126,"avatar_url":"https://avatars.githubusercontent.com/u/5663126?v=3","gravatar_id":"","url":"https://api.github.com/users/chris-pardy","html_url":"https://github.com/chris-pardy","followers_url":"https://api.github.com/users/chris-pardy/followers","following_url":"https://api.github.com/users/chris-pardy/following{/other_user}","gists_url":"https://api.github.com/users/chris-pardy/gists{/gist_id}","starred_url":"https://api.github.com/users/chris-pardy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chris-pardy/subscriptions","organizations_url":"https://api.github.com/users/chris-pardy/orgs","repos_url":"https://api.github.com/users/chris-pardy/repos","events_url":"https://api.github.com/users/chris-pardy/events{/privacy}","received_events_url":"https://api.github.com/users/chris-pardy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2016-09-23T20:55:14Z","updated_at":"2016-10-04T21:57:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/twitter/heron/pulls/1420","html_url":"https://github.com/twitter/heron/pull/1420","diff_url":"https://github.com/twitter/heron/pull/1420.diff","patch_url":"https://github.com/twitter/heron/pull/1420.patch"},"body":"We'd like to keep using the builtin Aurora scheduler but we'd like to.\r\n1) Move the default file to a different location, and not have it be named heron.aurora (we're using json for other aurora jobs)\r\n2) Sometimes specify a different file for different topologies.\r\n\r\nThis PR is intended to enable this workflow by allowing configs to override the location and/or name of the default aurora template. Additionally it looks for a System Property which if set will take precedence. Because this method falls back to the old behavior if the properties aren't set this should be a non-breaking change however there is a very small possibility that someone is running the Heron CLI with the environment property HERON_SCHEDULER_JOB_TEMPLATE set to some garbage. I think the risk is minimal enough that it's not required to require a config be turned on to check the environment property."},"comment":{"url":"https://api.github.com/repos/twitter/heron/issues/comments/251525887","html_url":"https://github.com/twitter/heron/pull/1420#issuecomment-251525887","issue_url":"https://api.github.com/repos/twitter/heron/issues/1420","id":251525887,"user":{"login":"kramasamy","id":367684,"avatar_url":"https://avatars.githubusercontent.com/u/367684?v=3","gravatar_id":"","url":"https://api.github.com/users/kramasamy","html_url":"https://github.com/kramasamy","followers_url":"https://api.github.com/users/kramasamy/followers","following_url":"https://api.github.com/users/kramasamy/following{/other_user}","gists_url":"https://api.github.com/users/kramasamy/gists{/gist_id}","starred_url":"https://api.github.com/users/kramasamy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kramasamy/subscriptions","organizations_url":"https://api.github.com/users/kramasamy/orgs","repos_url":"https://api.github.com/users/kramasamy/repos","events_url":"https://api.github.com/users/kramasamy/events{/privacy}","received_events_url":"https://api.github.com/users/kramasamy/received_events","type":"User","site_admin":false},"created_at":"2016-10-04T21:57:34Z","updated_at":"2016-10-04T21:57:34Z","body":"@chris-pardy - looks good. there were some flaky integration tests. we are taking a look at them."}}
{"actor":{"display_login":"kramasamy"}}