{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/13877","repository_url":"https://api.github.com/repos/openshift/origin","labels_url":"https://api.github.com/repos/openshift/origin/issues/13877/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/13877/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/13877/events","html_url":"https://github.com/openshift/origin/pull/13877","id":223870036,"number":13877,"title":"Fix NATting of external traffic with ovs-networkpolicy","user":{"login":"danwinship","id":96040,"avatar_url":"https://avatars3.githubusercontent.com/u/96040?v=3","gravatar_id":"","url":"https://api.github.com/users/danwinship","html_url":"https://github.com/danwinship","followers_url":"https://api.github.com/users/danwinship/followers","following_url":"https://api.github.com/users/danwinship/following{/other_user}","gists_url":"https://api.github.com/users/danwinship/gists{/gist_id}","starred_url":"https://api.github.com/users/danwinship/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danwinship/subscriptions","organizations_url":"https://api.github.com/users/danwinship/orgs","repos_url":"https://api.github.com/users/danwinship/repos","events_url":"https://api.github.com/users/danwinship/events{/privacy}","received_events_url":"https://api.github.com/users/danwinship/received_events","type":"User","site_admin":false},"labels":[{"id":210156114,"url":"https://api.github.com/repos/openshift/origin/labels/component/networking","name":"component/networking","color":"0052cc","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-04-24T16:03:18Z","updated_at":"2017-04-24T18:42:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/13877","html_url":"https://github.com/openshift/origin/pull/13877","diff_url":"https://github.com/openshift/origin/pull/13877.diff","patch_url":"https://github.com/openshift/origin/pull/13877.patch"},"body":"Traffic to external sites with ovs-networkpolicy was broken because calling `ct(commit)` for our own purposes blocked iptables from acting on the packets, so they wouldn't get NATted.\r\n\r\n(This is part of the same breakage that caused #12837, but this part of the bug was never caught because we didn't put \"external connectivity\" in the acceptance criteria so it wasn't part of the QE tests. And we don't test ovs-networkpolicy at all in jenkins because of #13049.)\r\n\r\nThis patch just assumes that ct zone \"1\" is unused. Maybe we should use a randomer number? Making it configurable seems like overkill...\r\n\r\nFixes https://bugzilla.redhat.com/show_bug.cgi?id=1443765\r\n\r\n@openshift/networking PTAL"},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/296785689","html_url":"https://github.com/openshift/origin/pull/13877#issuecomment-296785689","issue_url":"https://api.github.com/repos/openshift/origin/issues/13877","id":296785689,"user":{"login":"rajatchopra","id":982221,"avatar_url":"https://avatars1.githubusercontent.com/u/982221?v=3","gravatar_id":"","url":"https://api.github.com/users/rajatchopra","html_url":"https://github.com/rajatchopra","followers_url":"https://api.github.com/users/rajatchopra/followers","following_url":"https://api.github.com/users/rajatchopra/following{/other_user}","gists_url":"https://api.github.com/users/rajatchopra/gists{/gist_id}","starred_url":"https://api.github.com/users/rajatchopra/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rajatchopra/subscriptions","organizations_url":"https://api.github.com/users/rajatchopra/orgs","repos_url":"https://api.github.com/users/rajatchopra/repos","events_url":"https://api.github.com/users/rajatchopra/events{/privacy}","received_events_url":"https://api.github.com/users/rajatchopra/received_events","type":"User","site_admin":false},"created_at":"2017-04-24T18:42:16Z","updated_at":"2017-04-24T18:42:16Z","body":"> How about I change it from 1 to 13877 (in honor of this PR)\r\n\r\nRight logic, yeah. +1 :). Put a comment though that this number is not special."}}
{"actor":{"display_login":"rajatchopra"}}