{"action":"created","issue":{"url":"https://api.github.com/repos/sosreport/sos/issues/1163","repository_url":"https://api.github.com/repos/sosreport/sos","labels_url":"https://api.github.com/repos/sosreport/sos/issues/1163/labels{/name}","comments_url":"https://api.github.com/repos/sosreport/sos/issues/1163/comments","events_url":"https://api.github.com/repos/sosreport/sos/issues/1163/events","html_url":"https://github.com/sosreport/sos/pull/1163","id":280959707,"number":1163,"title":" [ipa] use correct PKI directories for tomcat version","user":{"login":"tscherf","id":1714713,"avatar_url":"https://avatars0.githubusercontent.com/u/1714713?v=4","gravatar_id":"","url":"https://api.github.com/users/tscherf","html_url":"https://github.com/tscherf","followers_url":"https://api.github.com/users/tscherf/followers","following_url":"https://api.github.com/users/tscherf/following{/other_user}","gists_url":"https://api.github.com/users/tscherf/gists{/gist_id}","starred_url":"https://api.github.com/users/tscherf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tscherf/subscriptions","organizations_url":"https://api.github.com/users/tscherf/orgs","repos_url":"https://api.github.com/users/tscherf/repos","events_url":"https://api.github.com/users/tscherf/events{/privacy}","received_events_url":"https://api.github.com/users/tscherf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-12-11T10:10:10Z","updated_at":"2017-12-21T13:58:00Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/sosreport/sos/pulls/1163","html_url":"https://github.com/sosreport/sos/pull/1163","diff_url":"https://github.com/sosreport/sos/pull/1163.diff","patch_url":"https://github.com/sosreport/sos/pull/1163.patch"},"body":"The PKI subsystem uses different folders in IPA v3 and v4 for the NSS DB.\r\nTherefore, different paths strings need to be passed to certutil, based on\r\nthe installed IPA version.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/sosreport/sos/issues/comments/353357112","html_url":"https://github.com/sosreport/sos/pull/1163#issuecomment-353357112","issue_url":"https://api.github.com/repos/sosreport/sos/issues/1163","id":353357112,"user":{"login":"pmoravec","id":4340368,"avatar_url":"https://avatars3.githubusercontent.com/u/4340368?v=4","gravatar_id":"","url":"https://api.github.com/users/pmoravec","html_url":"https://github.com/pmoravec","followers_url":"https://api.github.com/users/pmoravec/followers","following_url":"https://api.github.com/users/pmoravec/following{/other_user}","gists_url":"https://api.github.com/users/pmoravec/gists{/gist_id}","starred_url":"https://api.github.com/users/pmoravec/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pmoravec/subscriptions","organizations_url":"https://api.github.com/users/pmoravec/orgs","repos_url":"https://api.github.com/users/pmoravec/repos","events_url":"https://api.github.com/users/pmoravec/events{/privacy}","received_events_url":"https://api.github.com/users/pmoravec/received_events","type":"User","site_admin":false},"created_at":"2017-12-21T13:58:00Z","updated_at":"2017-12-21T13:58:00Z","author_association":"CONTRIBUTOR","body":"I think any approach is OK wrt. more commits or incrementally. IMHO the current commit should be separate as it changes one logical/coherent area, and other changes should be done in further commits on top of this one.\r\n\r\nThere are few pep8 issues (see https://travis-ci.org/sosreport/sos/jobs/314767272 lines 1499-1501), apart of that +1 to the commit / PR."}}
{"actor":{"display_login":"pmoravec"}}