{"action":"created","issue":{"url":"https://api.github.com/repos/theforeman/foreman/issues/2376","labels_url":"https://api.github.com/repos/theforeman/foreman/issues/2376/labels{/name}","comments_url":"https://api.github.com/repos/theforeman/foreman/issues/2376/comments","events_url":"https://api.github.com/repos/theforeman/foreman/issues/2376/events","html_url":"https://github.com/theforeman/foreman/pull/2376","id":75699776,"number":2376,"title":"Fixes #10487: Add custom logging support.","user":{"login":"ehelms","id":293363,"avatar_url":"https://avatars.githubusercontent.com/u/293363?v=3","gravatar_id":"","url":"https://api.github.com/users/ehelms","html_url":"https://github.com/ehelms","followers_url":"https://api.github.com/users/ehelms/followers","following_url":"https://api.github.com/users/ehelms/following{/other_user}","gists_url":"https://api.github.com/users/ehelms/gists{/gist_id}","starred_url":"https://api.github.com/users/ehelms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ehelms/subscriptions","organizations_url":"https://api.github.com/users/ehelms/orgs","repos_url":"https://api.github.com/users/ehelms/repos","events_url":"https://api.github.com/users/ehelms/events{/privacy}","received_events_url":"https://api.github.com/users/ehelms/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/theforeman/foreman/labels/Needs%20re-review","name":"Needs re-review","color":"0052cc"},{"url":"https://api.github.com/repos/theforeman/foreman/labels/Needs%20testing","name":"Needs testing","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":18,"created_at":"2015-05-12T18:46:13Z","updated_at":"2015-05-22T15:35:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/theforeman/foreman/pulls/2376","html_url":"https://github.com/theforeman/foreman/pull/2376","diff_url":"https://github.com/theforeman/foreman/pull/2376.diff","patch_url":"https://github.com/theforeman/foreman/pull/2376.patch"},"body":"This change moves away from the Rails logger to the 'logging' gem\r\nto support more configurability of logging. Further, this allows for\r\nthe creation of custom loggers that plugins can then register through\r\nsettings.\r\n\r\nFor example, the plugin katello could define within its settings file\r\n(settings.plugins.d/katello.yaml) the following:\r\n\r\n```\r\n:katello:\r\n  :loggers:\r\n    :pulp_rest:\r\n      :enabled: false\r\n    :cp_rest:\r\n      :enabled: true\r\n```\r\n\r\nWhich would define 2 loggers, pulp_rest and cp_rest where the latter\r\nis turned on be default, but the former can be enabled such that\r\nthe log file isn't continuously filled with messages from that particular\r\nlayer."},"comment":{"url":"https://api.github.com/repos/theforeman/foreman/issues/comments/104693206","html_url":"https://github.com/theforeman/foreman/pull/2376#issuecomment-104693206","issue_url":"https://api.github.com/repos/theforeman/foreman/issues/2376","id":104693206,"user":{"login":"domcleal","id":482089,"avatar_url":"https://avatars.githubusercontent.com/u/482089?v=3","gravatar_id":"","url":"https://api.github.com/users/domcleal","html_url":"https://github.com/domcleal","followers_url":"https://api.github.com/users/domcleal/followers","following_url":"https://api.github.com/users/domcleal/following{/other_user}","gists_url":"https://api.github.com/users/domcleal/gists{/gist_id}","starred_url":"https://api.github.com/users/domcleal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/domcleal/subscriptions","organizations_url":"https://api.github.com/users/domcleal/orgs","repos_url":"https://api.github.com/users/domcleal/repos","events_url":"https://api.github.com/users/domcleal/events{/privacy}","received_events_url":"https://api.github.com/users/domcleal/received_events","type":"User","site_admin":false},"created_at":"2015-05-22T15:35:40Z","updated_at":"2015-05-22T15:35:40Z","body":"> 1) Production environment set to 'syslog' by default\r\n\r\nSorry, I guess I misunderstood earlier, I thought this was just to enable it as an option rather than to change the default.\r\n\r\nI'm not sure with the amount we log and because it's multi-line that we should be doing this - we might be best keeping the file as default, then later on integrating to journald as an option.  Sorry for the confusion."}}