{"action":"created","issue":{"url":"https://api.github.com/repos/freeipa/freeipa/issues/927","repository_url":"https://api.github.com/repos/freeipa/freeipa","labels_url":"https://api.github.com/repos/freeipa/freeipa/issues/927/labels{/name}","comments_url":"https://api.github.com/repos/freeipa/freeipa/issues/927/comments","events_url":"https://api.github.com/repos/freeipa/freeipa/issues/927/events","html_url":"https://github.com/freeipa/freeipa/pull/927","id":244718778,"number":927,"title":"WebUI: Cockpit integration","user":{"login":"pvomacka","id":9040976,"avatar_url":"https://avatars1.githubusercontent.com/u/9040976?v=4","gravatar_id":"","url":"https://api.github.com/users/pvomacka","html_url":"https://github.com/pvomacka","followers_url":"https://api.github.com/users/pvomacka/followers","following_url":"https://api.github.com/users/pvomacka/following{/other_user}","gists_url":"https://api.github.com/users/pvomacka/gists{/gist_id}","starred_url":"https://api.github.com/users/pvomacka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pvomacka/subscriptions","organizations_url":"https://api.github.com/users/pvomacka/orgs","repos_url":"https://api.github.com/users/pvomacka/repos","events_url":"https://api.github.com/users/pvomacka/events{/privacy}","received_events_url":"https://api.github.com/users/pvomacka/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"pvoborni","id":5136843,"avatar_url":"https://avatars0.githubusercontent.com/u/5136843?v=4","gravatar_id":"","url":"https://api.github.com/users/pvoborni","html_url":"https://github.com/pvoborni","followers_url":"https://api.github.com/users/pvoborni/followers","following_url":"https://api.github.com/users/pvoborni/following{/other_user}","gists_url":"https://api.github.com/users/pvoborni/gists{/gist_id}","starred_url":"https://api.github.com/users/pvoborni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pvoborni/subscriptions","organizations_url":"https://api.github.com/users/pvoborni/orgs","repos_url":"https://api.github.com/users/pvoborni/repos","events_url":"https://api.github.com/users/pvoborni/events{/privacy}","received_events_url":"https://api.github.com/users/pvoborni/received_events","type":"User","site_admin":false},"assignees":[{"login":"pvoborni","id":5136843,"avatar_url":"https://avatars0.githubusercontent.com/u/5136843?v=4","gravatar_id":"","url":"https://api.github.com/users/pvoborni","html_url":"https://github.com/pvoborni","followers_url":"https://api.github.com/users/pvoborni/followers","following_url":"https://api.github.com/users/pvoborni/following{/other_user}","gists_url":"https://api.github.com/users/pvoborni/gists{/gist_id}","starred_url":"https://api.github.com/users/pvoborni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pvoborni/subscriptions","organizations_url":"https://api.github.com/users/pvoborni/orgs","repos_url":"https://api.github.com/users/pvoborni/repos","events_url":"https://api.github.com/users/pvoborni/events{/privacy}","received_events_url":"https://api.github.com/users/pvoborni/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2017-07-21T16:21:51Z","updated_at":"2017-08-31T13:12:28Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/freeipa/freeipa/pulls/927","html_url":"https://github.com/freeipa/freeipa/pull/927","diff_url":"https://github.com/freeipa/freeipa/pull/927.diff","patch_url":"https://github.com/freeipa/freeipa/pull/927.patch"},"body":"Link to the cockpit is placed to each host details page in case\r\nthe Cockpit is installed on the server.\r\n\r\nShowing or hiding cockpit link is possible because of Cockpit's API\r\nwhich provides public URL for check whether Cockpit is the software\r\nwhich listen on given port.\r\n\r\nhttps://pagure.io/freeipa/issue/4891"},"comment":{"url":"https://api.github.com/repos/freeipa/freeipa/issues/comments/326290931","html_url":"https://github.com/freeipa/freeipa/pull/927#issuecomment-326290931","issue_url":"https://api.github.com/repos/freeipa/freeipa/issues/927","id":326290931,"user":{"login":"pvoborni","id":5136843,"avatar_url":"https://avatars0.githubusercontent.com/u/5136843?v=4","gravatar_id":"","url":"https://api.github.com/users/pvoborni","html_url":"https://github.com/pvoborni","followers_url":"https://api.github.com/users/pvoborni/followers","following_url":"https://api.github.com/users/pvoborni/following{/other_user}","gists_url":"https://api.github.com/users/pvoborni/gists{/gist_id}","starred_url":"https://api.github.com/users/pvoborni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pvoborni/subscriptions","organizations_url":"https://api.github.com/users/pvoborni/orgs","repos_url":"https://api.github.com/users/pvoborni/repos","events_url":"https://api.github.com/users/pvoborni/events{/privacy}","received_events_url":"https://api.github.com/users/pvoborni/received_events","type":"User","site_admin":false},"created_at":"2017-08-31T13:12:28Z","updated_at":"2017-08-31T13:12:28Z","author_association":"OWNER","body":"1.  I'd change message `Cockpit has not been detected on host.example.test host` into just:  `Cockpit has not been detected on host.example.test`\r\n\r\n2. When cockpit is detected the message should match it: e.g. MD equivalent: `Cockpit detected on host.  [Open Cockpit](https://host.example.test:9090)`\r\n\r\n3. `link_widget` has `on_link_clicked` method. If you click on cockpit link this method is executed and fails with exception which is logged in console. Practically it won't be noticed much because the page is then switched to cockpit interface.  But even though I think it should be fixed. It doesn't happen if cockpit UI is opened in new tab."}}
{"actor":{"display_login":"pvoborni"}}