{"comment":{"url":"https://api.github.com/repos/kgaillot/pacemaker/comments/11634151","html_url":"https://github.com/kgaillot/pacemaker/commit/5bd8c7d10532a23ea3acfcad2f086ccbafe89a5b#commitcomment-11634151","id":11634151,"user":{"login":"kgaillot","id":8730570,"avatar_url":"https://avatars.githubusercontent.com/u/8730570?v=3","gravatar_id":"","url":"https://api.github.com/users/kgaillot","html_url":"https://github.com/kgaillot","followers_url":"https://api.github.com/users/kgaillot/followers","following_url":"https://api.github.com/users/kgaillot/following{/other_user}","gists_url":"https://api.github.com/users/kgaillot/gists{/gist_id}","starred_url":"https://api.github.com/users/kgaillot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kgaillot/subscriptions","organizations_url":"https://api.github.com/users/kgaillot/orgs","repos_url":"https://api.github.com/users/kgaillot/repos","events_url":"https://api.github.com/users/kgaillot/events{/privacy}","received_events_url":"https://api.github.com/users/kgaillot/received_events","type":"User","site_admin":false},"position":24,"line":801,"path":"lib/common/ipc.c","commit_id":"5bd8c7d10532a23ea3acfcad2f086ccbafe89a5b","created_at":"2015-06-11T13:47:12Z","updated_at":"2015-06-11T13:47:12Z","body":"Yes, the responsibility for error reporting was moved to the callers. When pacemaker starts up, it calls crm_ipc_connect() to verify that an instance isn't already running. So before, this would result in messages that looked like errors in the syslog, which can make sysadmins nervous. Also, with the messages here, they were always logged with info severity, when sometimes they were important errors and sometimes not. Now each caller gives a more appropriate message."}}