{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/beat-generator/issues/18","repository_url":"https://api.github.com/repos/elastic/beat-generator","labels_url":"https://api.github.com/repos/elastic/beat-generator/issues/18/labels{/name}","comments_url":"https://api.github.com/repos/elastic/beat-generator/issues/18/comments","events_url":"https://api.github.com/repos/elastic/beat-generator/issues/18/events","html_url":"https://github.com/elastic/beat-generator/pull/18","id":139895794,"number":18,"title":"Configuration initializer and renaming","user":{"login":"buehler","id":1067213,"avatar_url":"https://avatars.githubusercontent.com/u/1067213?v=3","gravatar_id":"","url":"https://api.github.com/users/buehler","html_url":"https://github.com/buehler","followers_url":"https://api.github.com/users/buehler/followers","following_url":"https://api.github.com/users/buehler/following{/other_user}","gists_url":"https://api.github.com/users/buehler/gists{/gist_id}","starred_url":"https://api.github.com/users/buehler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/buehler/subscriptions","organizations_url":"https://api.github.com/users/buehler/orgs","repos_url":"https://api.github.com/users/buehler/repos","events_url":"https://api.github.com/users/buehler/events{/privacy}","received_events_url":"https://api.github.com/users/buehler/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-03-10T14:05:22Z","updated_at":"2016-03-14T08:13:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/elastic/beat-generator/pulls/18","html_url":"https://github.com/elastic/beat-generator/pull/18","diff_url":"https://github.com/elastic/beat-generator/pull/18.diff","patch_url":"https://github.com/elastic/beat-generator/pull/18.patch"},"body":"Based on the discussion of #17, here is the config PR with the following commits:\r\n\r\n- Rename beat.yml to {{cookiecutter.beat}}.yml\r\n- Add the config initializer to get a pointer to the config struct (instead of init the config and set defaults in the beat code)\r\n- Rename the `Configuration` to `beatConfig` and thus make it private by default\r\n\r\nI'm happy for more input :smiley: "},"comment":{"url":"https://api.github.com/repos/elastic/beat-generator/issues/comments/196193851","html_url":"https://github.com/elastic/beat-generator/pull/18#issuecomment-196193851","issue_url":"https://api.github.com/repos/elastic/beat-generator/issues/18","id":196193851,"user":{"login":"ruflin","id":244900,"avatar_url":"https://avatars.githubusercontent.com/u/244900?v=3","gravatar_id":"","url":"https://api.github.com/users/ruflin","html_url":"https://github.com/ruflin","followers_url":"https://api.github.com/users/ruflin/followers","following_url":"https://api.github.com/users/ruflin/following{/other_user}","gists_url":"https://api.github.com/users/ruflin/gists{/gist_id}","starred_url":"https://api.github.com/users/ruflin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ruflin/subscriptions","organizations_url":"https://api.github.com/users/ruflin/orgs","repos_url":"https://api.github.com/users/ruflin/repos","events_url":"https://api.github.com/users/ruflin/events{/privacy}","received_events_url":"https://api.github.com/users/ruflin/received_events","type":"User","site_admin":false},"created_at":"2016-03-14T08:13:46Z","updated_at":"2016-03-14T08:13:46Z","body":"* +1 on beatConfig renaming\r\n* +1 on let initi die in favor of ucfg\r\n* For the PR's: Lets make sure we don't mix things and have one by one :-)\r\n* Thanks for checking. Yeoman is based on npm and js and I prefer not to introduce one more language dependency. Python is already needed for parts of the development. My preferred option would be to have something golang based, so no additional tool is needed to fetch the template.\r\n\r\nSome background info: In the long run we probably have multiple generators. One for a full beat, one for a module in metricbeat, one for a protocol in packetbeat etc. It would be nice if we could have these all under a similar path. Not sure yet how we are going to organise these.\r\n\r\nThe reason I'm quite picky on these PR's is because everything that gets merged will show up in each someone creates. Perhaps in the long run we need a \"release\" version."}}