{"action":"closed","issue":{"url":"https://api.github.com/repos/rhos-infra/InfraRed/issues/59","repository_url":"https://api.github.com/repos/rhos-infra/InfraRed","labels_url":"https://api.github.com/repos/rhos-infra/InfraRed/issues/59/labels{/name}","comments_url":"https://api.github.com/repos/rhos-infra/InfraRed/issues/59/comments","events_url":"https://api.github.com/repos/rhos-infra/InfraRed/issues/59/events","html_url":"https://github.com/rhos-infra/InfraRed/issues/59","id":137236498,"number":59,"title":"Print caller information to the log","user":{"login":"obaranov","id":15234398,"avatar_url":"https://avatars.githubusercontent.com/u/15234398?v=3","gravatar_id":"","url":"https://api.github.com/users/obaranov","html_url":"https://github.com/obaranov","followers_url":"https://api.github.com/users/obaranov/followers","following_url":"https://api.github.com/users/obaranov/following{/other_user}","gists_url":"https://api.github.com/users/obaranov/gists{/gist_id}","starred_url":"https://api.github.com/users/obaranov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/obaranov/subscriptions","organizations_url":"https://api.github.com/users/obaranov/orgs","repos_url":"https://api.github.com/users/obaranov/repos","events_url":"https://api.github.com/users/obaranov/events{/privacy}","received_events_url":"https://api.github.com/users/obaranov/received_events","type":"User","site_admin":false},"labels":[{"id":324784573,"url":"https://api.github.com/repos/rhos-infra/InfraRed/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-02-29T12:06:11Z","updated_at":"2016-12-22T10:28:45Z","closed_at":"2016-12-22T10:28:45Z","body":"From current log it's very hard to get any useful information from what module/method/class the log line was printed. I'd suggest to add that information to simplify debug process.\n\nWe can do next: \n1) Add %(module)s and %(funcName)s to the logger message format. But this will now work good for classes and objects.\n2) Do not use one name for all the logger. Each class/module need to create own logger with: \n\n```\nLOG = logging.getLogger(self.__class__.__name__) \n```\n\nAnd after that add logger name (%(name)s) to the logger message format. \n\nMy vote is for option 2). \n"}}
{"actor":{"display_login":"tkammer"}}