{"action":"created","issue":{"url":"https://api.github.com/repos/prometheus/prometheus/issues/1976","repository_url":"https://api.github.com/repos/prometheus/prometheus","labels_url":"https://api.github.com/repos/prometheus/prometheus/issues/1976/labels{/name}","comments_url":"https://api.github.com/repos/prometheus/prometheus/issues/1976/comments","events_url":"https://api.github.com/repos/prometheus/prometheus/issues/1976/events","html_url":"https://github.com/prometheus/prometheus/issues/1976","id":176208052,"number":1976,"title":"consider making cmd/prometheus config defaults as package-level vars","user":{"login":"mischief","id":1487996,"avatar_url":"https://avatars.githubusercontent.com/u/1487996?v=3","gravatar_id":"","url":"https://api.github.com/users/mischief","html_url":"https://github.com/mischief","followers_url":"https://api.github.com/users/mischief/followers","following_url":"https://api.github.com/users/mischief/following{/other_user}","gists_url":"https://api.github.com/users/mischief/gists{/gist_id}","starred_url":"https://api.github.com/users/mischief/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mischief/subscriptions","organizations_url":"https://api.github.com/users/mischief/orgs","repos_url":"https://api.github.com/users/mischief/repos","events_url":"https://api.github.com/users/mischief/events{/privacy}","received_events_url":"https://api.github.com/users/mischief/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-09-10T22:47:15Z","updated_at":"2016-09-22T10:52:05Z","closed_at":null,"body":"in ./cmd/prometheus/config.go, the default directories for assets such as consoles are inline in the cli flag declarations.\r\n\r\nif these defaults were moved out into package level string vars, they could be set with go's -ldflags -X, which would allow a packaging system to set the defaults appropriate to where these files will be installed."},"comment":{"url":"https://api.github.com/repos/prometheus/prometheus/issues/comments/248868931","html_url":"https://github.com/prometheus/prometheus/issues/1976#issuecomment-248868931","issue_url":"https://api.github.com/repos/prometheus/prometheus/issues/1976","id":248868931,"user":{"login":"brian-brazil","id":7115638,"avatar_url":"https://avatars.githubusercontent.com/u/7115638?v=3","gravatar_id":"","url":"https://api.github.com/users/brian-brazil","html_url":"https://github.com/brian-brazil","followers_url":"https://api.github.com/users/brian-brazil/followers","following_url":"https://api.github.com/users/brian-brazil/following{/other_user}","gists_url":"https://api.github.com/users/brian-brazil/gists{/gist_id}","starred_url":"https://api.github.com/users/brian-brazil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brian-brazil/subscriptions","organizations_url":"https://api.github.com/users/brian-brazil/orgs","repos_url":"https://api.github.com/users/brian-brazil/repos","events_url":"https://api.github.com/users/brian-brazil/events{/privacy}","received_events_url":"https://api.github.com/users/brian-brazil/received_events","type":"User","site_admin":false},"created_at":"2016-09-22T10:52:05Z","updated_at":"2016-09-22T10:52:05Z","body":"That brings many of the same challenges as the others: it's an additional way to do things, and it's no longer sufficient to look at the command line to know what is being passed which will make debugging and support harder (imagine someone getting one of these binaries and being confused when we tell them e.g. the default retention is 15 days if you haven't changed anything).\r\n\r\nThis use case could be handled by a shell script wrapper, or more likely as this would be for use in a packaging system via a /etc/defaults file."}}
{"actor":{"display_login":"brian-brazil"}}