{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/71674","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/71674/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/71674/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/71674/events","html_url":"https://github.com/kubernetes/kubernetes/pull/71674","id":387045492,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM1NjIzNDk3","number":71674,"title":"Change XPN firewall change msg. Should be required by security admin","user":{"login":"grayluck","id":4760200,"node_id":"MDQ6VXNlcjQ3NjAyMDA=","avatar_url":"https://avatars3.githubusercontent.com/u/4760200?v=4","gravatar_id":"","url":"https://api.github.com/users/grayluck","html_url":"https://github.com/grayluck","followers_url":"https://api.github.com/users/grayluck/followers","following_url":"https://api.github.com/users/grayluck/following{/other_user}","gists_url":"https://api.github.com/users/grayluck/gists{/gist_id}","starred_url":"https://api.github.com/users/grayluck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grayluck/subscriptions","organizations_url":"https://api.github.com/users/grayluck/orgs","repos_url":"https://api.github.com/users/grayluck/repos","events_url":"https://api.github.com/users/grayluck/events{/privacy}","received_events_url":"https://api.github.com/users/grayluck/received_events","type":"User","site_admin":false},"labels":[{"id":477397086,"node_id":"MDU6TGFiZWw0NzczOTcwODY=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/cncf-cla:%20yes","name":"cncf-cla: yes","color":"bfe5bf","default":false},{"id":116801185,"node_id":"MDU6TGFiZWwxMTY4MDExODU=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/kind/documentation","name":"kind/documentation","color":"c7def8","default":false},{"id":573014062,"node_id":"MDU6TGFiZWw1NzMwMTQwNjI=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/needs-ok-to-test","name":"needs-ok-to-test","color":"b60205","default":false},{"id":114528273,"node_id":"MDU6TGFiZWwxMTQ1MjgyNzM=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/priority/backlog","name":"priority/backlog","color":"fbca04","default":false},{"id":349530249,"node_id":"MDU6TGFiZWwzNDk1MzAyNDk=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/release-note-none","name":"release-note-none","color":"c2e0c6","default":false},{"id":958178286,"node_id":"MDU6TGFiZWw5NTgxNzgyODY=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/sig/cloud-provider","name":"sig/cloud-provider","color":"d2b48c","default":false},{"id":116712108,"node_id":"MDU6TGFiZWwxMTY3MTIxMDg=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/sig/network","name":"sig/network","color":"d2b48c","default":false},{"id":253450895,"node_id":"MDU6TGFiZWwyNTM0NTA4OTU=","url":"https://api.github.com/repos/kubernetes/kubernetes/labels/size/S","name":"size/S","color":"77bb00","default":false}],"state":"open","locked":false,"assignee":{"login":"bowei","id":837034,"node_id":"MDQ6VXNlcjgzNzAzNA==","avatar_url":"https://avatars1.githubusercontent.com/u/837034?v=4","gravatar_id":"","url":"https://api.github.com/users/bowei","html_url":"https://github.com/bowei","followers_url":"https://api.github.com/users/bowei/followers","following_url":"https://api.github.com/users/bowei/following{/other_user}","gists_url":"https://api.github.com/users/bowei/gists{/gist_id}","starred_url":"https://api.github.com/users/bowei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bowei/subscriptions","organizations_url":"https://api.github.com/users/bowei/orgs","repos_url":"https://api.github.com/users/bowei/repos","events_url":"https://api.github.com/users/bowei/events{/privacy}","received_events_url":"https://api.github.com/users/bowei/received_events","type":"User","site_admin":false},"assignees":[{"login":"bowei","id":837034,"node_id":"MDQ6VXNlcjgzNzAzNA==","avatar_url":"https://avatars1.githubusercontent.com/u/837034?v=4","gravatar_id":"","url":"https://api.github.com/users/bowei","html_url":"https://github.com/bowei","followers_url":"https://api.github.com/users/bowei/followers","following_url":"https://api.github.com/users/bowei/following{/other_user}","gists_url":"https://api.github.com/users/bowei/gists{/gist_id}","starred_url":"https://api.github.com/users/bowei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bowei/subscriptions","organizations_url":"https://api.github.com/users/bowei/orgs","repos_url":"https://api.github.com/users/bowei/repos","events_url":"https://api.github.com/users/bowei/events{/privacy}","received_events_url":"https://api.github.com/users/bowei/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2018-12-03T23:05:33Z","updated_at":"2018-12-04T21:46:14Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/kubernetes/kubernetes/pulls/71674","html_url":"https://github.com/kubernetes/kubernetes/pull/71674","diff_url":"https://github.com/kubernetes/kubernetes/pull/71674.diff","patch_url":"https://github.com/kubernetes/kubernetes/pull/71674.patch"},"body":"<!--  Thanks for sending a pull request!  Here are some tips for you:\r\n\r\n1. If this is your first time, please read our contributor guidelines: https://git.k8s.io/community/contributors/guide#your-first-contribution and developer guide https://git.k8s.io/community/contributors/devel/development.md#development-guide\r\n2. Please label this pull request according to what type of issue you are addressing, especially if this is a release targeted pull request. For reference on required PR/issue labels, read here:\r\nhttps://git.k8s.io/community/contributors/devel/release.md#issue-kind-label\r\n3. Ensure you have added or ran the appropriate tests for your PR: https://git.k8s.io/community/contributors/devel/testing.md\r\n4. If you want *faster* PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews\r\n5. Follow the instructions for writing a release note: https://git.k8s.io/community/contributors/guide/release-notes.md\r\n6. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests\r\n-->\r\n\r\n**What type of PR is this?**\r\n> Uncomment only one, leave it on its own line:\r\n>\r\n> /kind api-change\r\n> /kind bug\r\n> /kind cleanup\r\n> /kind design\r\n/kind documentation\r\n> /kind failing-test\r\n> /kind feature\r\n> /kind flake\r\n\r\n**What this PR does / why we need it**:\r\nWhen XPN cluster fails to create a firewall rule due to insufficient permission, it will file a event which states that the firewall rule being created by network admin. To keep consistent with IAM, the firewall rule should be created by security admin.\r\n\r\nWill change accordingly in ingress.\r\n\r\n**Special notes for your reviewer**:\r\n/assign @bowei \r\n\r\n**Does this PR introduce a user-facing change?**:\r\n<!--  \r\nIf no, just write \"NONE\".\r\nIf yes, a release note is required:\r\nEnter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string \"action required\".\r\n2. \r\n-->\r\n```release-note\r\nNONE\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/444271009","html_url":"https://github.com/kubernetes/kubernetes/pull/71674#issuecomment-444271009","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/71674","id":444271009,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NDI3MTAwOQ==","user":{"login":"grayluck","id":4760200,"node_id":"MDQ6VXNlcjQ3NjAyMDA=","avatar_url":"https://avatars3.githubusercontent.com/u/4760200?v=4","gravatar_id":"","url":"https://api.github.com/users/grayluck","html_url":"https://github.com/grayluck","followers_url":"https://api.github.com/users/grayluck/followers","following_url":"https://api.github.com/users/grayluck/following{/other_user}","gists_url":"https://api.github.com/users/grayluck/gists{/gist_id}","starred_url":"https://api.github.com/users/grayluck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grayluck/subscriptions","organizations_url":"https://api.github.com/users/grayluck/orgs","repos_url":"https://api.github.com/users/grayluck/repos","events_url":"https://api.github.com/users/grayluck/events{/privacy}","received_events_url":"https://api.github.com/users/grayluck/received_events","type":"User","site_admin":false},"created_at":"2018-12-04T21:46:14Z","updated_at":"2018-12-04T21:46:14Z","author_association":"CONTRIBUTOR","body":"This change will be synced to ingress-gce so that we can reduce the source ranges for ingress health checks."}}
{"actor":{"display_login":"grayluck"}}