{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/6319","repository_url":"https://api.github.com/repos/openshift/openshift-ansible","labels_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/6319/labels{/name}","comments_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/6319/comments","events_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/6319/events","html_url":"https://github.com/openshift/openshift-ansible/pull/6319","id":278066536,"number":6319,"title":"Kuryr var generation in OSt dynamic inventory","user":{"login":"celebdor","id":619423,"avatar_url":"https://avatars0.githubusercontent.com/u/619423?v=4","gravatar_id":"","url":"https://api.github.com/users/celebdor","html_url":"https://github.com/celebdor","followers_url":"https://api.github.com/users/celebdor/followers","following_url":"https://api.github.com/users/celebdor/following{/other_user}","gists_url":"https://api.github.com/users/celebdor/gists{/gist_id}","starred_url":"https://api.github.com/users/celebdor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/celebdor/subscriptions","organizations_url":"https://api.github.com/users/celebdor/orgs","repos_url":"https://api.github.com/users/celebdor/repos","events_url":"https://api.github.com/users/celebdor/events{/privacy}","received_events_url":"https://api.github.com/users/celebdor/received_events","type":"User","site_admin":false},"labels":[{"id":681679137,"url":"https://api.github.com/repos/openshift/openshift-ansible/labels/size/L","name":"size/L","color":"ee9900","default":false}],"state":"open","locked":false,"assignee":{"login":"tomassedovic","id":104339,"avatar_url":"https://avatars1.githubusercontent.com/u/104339?v=4","gravatar_id":"","url":"https://api.github.com/users/tomassedovic","html_url":"https://github.com/tomassedovic","followers_url":"https://api.github.com/users/tomassedovic/followers","following_url":"https://api.github.com/users/tomassedovic/following{/other_user}","gists_url":"https://api.github.com/users/tomassedovic/gists{/gist_id}","starred_url":"https://api.github.com/users/tomassedovic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomassedovic/subscriptions","organizations_url":"https://api.github.com/users/tomassedovic/orgs","repos_url":"https://api.github.com/users/tomassedovic/repos","events_url":"https://api.github.com/users/tomassedovic/events{/privacy}","received_events_url":"https://api.github.com/users/tomassedovic/received_events","type":"User","site_admin":false},"assignees":[{"login":"tomassedovic","id":104339,"avatar_url":"https://avatars1.githubusercontent.com/u/104339?v=4","gravatar_id":"","url":"https://api.github.com/users/tomassedovic","html_url":"https://github.com/tomassedovic","followers_url":"https://api.github.com/users/tomassedovic/followers","following_url":"https://api.github.com/users/tomassedovic/following{/other_user}","gists_url":"https://api.github.com/users/tomassedovic/gists{/gist_id}","starred_url":"https://api.github.com/users/tomassedovic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomassedovic/subscriptions","organizations_url":"https://api.github.com/users/tomassedovic/orgs","repos_url":"https://api.github.com/users/tomassedovic/repos","events_url":"https://api.github.com/users/tomassedovic/events{/privacy}","received_events_url":"https://api.github.com/users/tomassedovic/received_events","type":"User","site_admin":false}],"milestone":null,"comments":10,"created_at":"2017-11-30T10:19:23Z","updated_at":"2018-02-22T16:29:09Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/openshift/openshift-ansible/pulls/6319","html_url":"https://github.com/openshift/openshift-ansible/pull/6319","diff_url":"https://github.com/openshift/openshift-ansible/pull/6319.diff","patch_url":"https://github.com/openshift/openshift-ansible/pull/6319.patch"},"body":"This PR adds the necessary network resources for Kuryr in the heat\r\nstack, including putting a l4 load balancer in front of the API.\r\n\r\nIt then automatically uses the resources to generate the OSEv3 variables\r\nneeded for the already existing kuryr role.\r\n\r\nChange-Id: I6fc4c6bc9835217334db1289987daf358dcf287b\r\nSigned-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>"},"comment":{"url":"https://api.github.com/repos/openshift/openshift-ansible/issues/comments/367737506","html_url":"https://github.com/openshift/openshift-ansible/pull/6319#issuecomment-367737506","issue_url":"https://api.github.com/repos/openshift/openshift-ansible/issues/6319","id":367737506,"user":{"login":"tomassedovic","id":104339,"avatar_url":"https://avatars1.githubusercontent.com/u/104339?v=4","gravatar_id":"","url":"https://api.github.com/users/tomassedovic","html_url":"https://github.com/tomassedovic","followers_url":"https://api.github.com/users/tomassedovic/followers","following_url":"https://api.github.com/users/tomassedovic/following{/other_user}","gists_url":"https://api.github.com/users/tomassedovic/gists{/gist_id}","starred_url":"https://api.github.com/users/tomassedovic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomassedovic/subscriptions","organizations_url":"https://api.github.com/users/tomassedovic/orgs","repos_url":"https://api.github.com/users/tomassedovic/repos","events_url":"https://api.github.com/users/tomassedovic/events{/privacy}","received_events_url":"https://api.github.com/users/tomassedovic/received_events","type":"User","site_admin":false},"created_at":"2018-02-22T16:29:09Z","updated_at":"2018-02-22T16:29:09Z","author_association":"CONTRIBUTOR","body":"@celebdor with the fixes below it now seems to work!\r\n\r\nhttps://github.com/tomassedovic/openshift-ansible/commit/341b7263df007076d4e54c920fcb656264200340\r\n\r\nIn addition, I had to set this in my inventory:\r\n\r\n    openshift_openstack_kuryr_pod_subnet_cidr: \"172.16.0.0/16\"\r\n\r\nBecause the default value conflicts with my network's range.\r\n\r\nAnd allso open DNS ports on each node's firewall by setting:\r\n```\r\nopenshift_master_open_ports:\r\n- service: dns tcp\r\n  port: 53/tcp\r\n- service: dns udp\r\n  port: 53/udp\r\nopenshift_node_open_ports:\r\n- service: dns tcp\r\n  port: 53/tcp\r\n- service: dns udp\r\n  port: 53/udp\r\n```\r\n\r\nI wonder if we can just disable the firewall entirely or add these rules in the Kuryr roles. Anyway, with the changes above, I think we can actually merge this and then work on improving it.\r\n\r\nFor one, it needs serious docs."}}
{"actor":{"display_login":"tomassedovic"}}