{"action":"created","issue":{"url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/265","repository_url":"https://api.github.com/repos/redhat-cop/casl-ansible","labels_url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/265/labels{/name}","comments_url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/265/comments","events_url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/265/events","html_url":"https://github.com/redhat-cop/casl-ansible/pull/265","id":336373375,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk3ODQyMzQ3","number":265,"title":"Add required port for Prometheus to sample inventories","user":{"login":"makentenza","id":10531016,"node_id":"MDQ6VXNlcjEwNTMxMDE2","avatar_url":"https://avatars1.githubusercontent.com/u/10531016?v=4","gravatar_id":"","url":"https://api.github.com/users/makentenza","html_url":"https://github.com/makentenza","followers_url":"https://api.github.com/users/makentenza/followers","following_url":"https://api.github.com/users/makentenza/following{/other_user}","gists_url":"https://api.github.com/users/makentenza/gists{/gist_id}","starred_url":"https://api.github.com/users/makentenza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/makentenza/subscriptions","organizations_url":"https://api.github.com/users/makentenza/orgs","repos_url":"https://api.github.com/users/makentenza/repos","events_url":"https://api.github.com/users/makentenza/events{/privacy}","received_events_url":"https://api.github.com/users/makentenza/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-06-27T20:21:11Z","updated_at":"2018-07-20T08:04:56Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/redhat-cop/casl-ansible/pulls/265","html_url":"https://github.com/redhat-cop/casl-ansible/pull/265","diff_url":"https://github.com/redhat-cop/casl-ansible/pull/265.diff","patch_url":"https://github.com/redhat-cop/casl-ansible/pull/265.patch"},"body":"#### What does this PR do?\r\nAdd the required ports that need to be opened on the OS side to support Prometheus monitoring using Node Exporter\r\n\r\n#### How should this be manually tested?\r\nFollow instructions from README, deploy Prometheus and Node Exporter and check Node Exporter targets and HAProxy are working\r\n\r\n#### Is there a relevant Issue open for this?\r\nResolves #263 \r\n#### Who would you like to review this?\r\ncc: @redhat-cop/casl\r\n"},"comment":{"url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/comments/406521644","html_url":"https://github.com/redhat-cop/casl-ansible/pull/265#issuecomment-406521644","issue_url":"https://api.github.com/repos/redhat-cop/casl-ansible/issues/265","id":406521644,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNjUyMTY0NA==","user":{"login":"makentenza","id":10531016,"node_id":"MDQ6VXNlcjEwNTMxMDE2","avatar_url":"https://avatars1.githubusercontent.com/u/10531016?v=4","gravatar_id":"","url":"https://api.github.com/users/makentenza","html_url":"https://github.com/makentenza","followers_url":"https://api.github.com/users/makentenza/followers","following_url":"https://api.github.com/users/makentenza/following{/other_user}","gists_url":"https://api.github.com/users/makentenza/gists{/gist_id}","starred_url":"https://api.github.com/users/makentenza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/makentenza/subscriptions","organizations_url":"https://api.github.com/users/makentenza/orgs","repos_url":"https://api.github.com/users/makentenza/repos","events_url":"https://api.github.com/users/makentenza/events{/privacy}","received_events_url":"https://api.github.com/users/makentenza/received_events","type":"User","site_admin":false},"created_at":"2018-07-20T08:04:56Z","updated_at":"2018-07-20T08:04:56Z","author_association":"CONTRIBUTOR","body":"@oybed sure, I though I didn't add that to that inventory sample as there was already an entry for that, but just checked the entry was only for the haproxy stats on the Infra nodes. I've added the node-exporter one to the Nodes definition now."}}
{"actor":{"display_login":"makentenza"}}