{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/37272","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/37272/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/37272/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/37272/events","html_url":"https://github.com/saltstack/salt/pull/37272","id":185627599,"number":37272,"title":"Get default logging level and log file from default opts dict","user":{"login":"vutny","id":16338056,"avatar_url":"https://avatars.githubusercontent.com/u/16338056?v=3","gravatar_id":"","url":"https://api.github.com/users/vutny","html_url":"https://github.com/vutny","followers_url":"https://api.github.com/users/vutny/followers","following_url":"https://api.github.com/users/vutny/following{/other_user}","gists_url":"https://api.github.com/users/vutny/gists{/gist_id}","starred_url":"https://api.github.com/users/vutny/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vutny/subscriptions","organizations_url":"https://api.github.com/users/vutny/orgs","repos_url":"https://api.github.com/users/vutny/repos","events_url":"https://api.github.com/users/vutny/events{/privacy}","received_events_url":"https://api.github.com/users/vutny/received_events","type":"User","site_admin":false},"labels":[{"id":77737132,"url":"https://api.github.com/repos/saltstack/salt/labels/Pending%20Discussion","name":"Pending Discussion","color":"c7def8","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-10-27T10:14:46Z","updated_at":"2016-11-10T23:45:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/37272","html_url":"https://github.com/saltstack/salt/pull/37272","diff_url":"https://github.com/saltstack/salt/pull/37272.diff","patch_url":"https://github.com/saltstack/salt/pull/37272.patch"},"body":"### What does this PR do?\n\nIt establishes a single source of truth for default logging level and log file path setting for Salt components. In other words, the motivation is to get rid of hard-coded values in `salt/utils/parsers.py` which are not taken into account in any case, but this could give some side-effects like described below.\n### What issues does this PR fix or reference?\n\nFor example, the `salt-call` and `salt-cloud` commands are displaying wrong information about default log-level in their online help.\n\nI have completely empty `/etc/salt/minion` file and nothing in `/etc/salt/minion.d`.\nHelp says that default log level is `info`:\n\n```\n# salt-call --help\nUsage: salt-call [options] <function> [arguments]\n\nSalt call is used to execute module functions locally on a minion\n\n...\n  Logging Options:\n    Logging options which override any settings defined on the\n    configuration files.\n\n    -l LOG_LEVEL, --log-level=LOG_LEVEL\n                        Console logging log level. One of 'all', 'garbage',\n                        'trace', 'debug', 'profile', 'info', 'warning',\n                        'error', 'critical', 'quiet'. Default: 'info'.\n    --log-file=LOG_FILE\n                        Log file path. Default: '/var/log/salt/minion'.\n    --log-file-level=LOG_LEVEL_LOGFILE\n                        Logfile logging log level. One of 'all', 'garbage',\n                        'trace', 'debug', 'profile', 'info', 'warning',\n                        'error', 'critical', 'quiet'. Default: 'info'.\n```\n\nBut actually it is not:\n\n``` yaml\n# salt-call config.get log_level\nlocal:\n    warning\n```\n### Tests written?\n\nNo\n"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/259841603","html_url":"https://github.com/saltstack/salt/pull/37272#issuecomment-259841603","issue_url":"https://api.github.com/repos/saltstack/salt/issues/37272","id":259841603,"user":{"login":"cachedout","id":111616,"avatar_url":"https://avatars.githubusercontent.com/u/111616?v=3","gravatar_id":"","url":"https://api.github.com/users/cachedout","html_url":"https://github.com/cachedout","followers_url":"https://api.github.com/users/cachedout/followers","following_url":"https://api.github.com/users/cachedout/following{/other_user}","gists_url":"https://api.github.com/users/cachedout/gists{/gist_id}","starred_url":"https://api.github.com/users/cachedout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cachedout/subscriptions","organizations_url":"https://api.github.com/users/cachedout/orgs","repos_url":"https://api.github.com/users/cachedout/repos","events_url":"https://api.github.com/users/cachedout/events{/privacy}","received_events_url":"https://api.github.com/users/cachedout/received_events","type":"User","site_admin":false},"created_at":"2016-11-10T23:45:20Z","updated_at":"2016-11-10T23:45:20Z","body":"Hi @vutny \r\n\r\nNo hurry here but just checking with you on the test progress. Thanks!"}}
{"actor":{"display_login":"cachedout"}}