{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/322","repository_url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack","labels_url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/322/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/322/comments","events_url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/322/events","html_url":"https://github.com/kubernetes/cloud-provider-openstack/pull/322","id":369120378,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIyMTM3MjE5","number":322,"title":"[Ingress] Do not recreate octavia resources when the controller restarts","user":{"login":"lingxiankong","id":4410121,"node_id":"MDQ6VXNlcjQ0MTAxMjE=","avatar_url":"https://avatars3.githubusercontent.com/u/4410121?v=4","gravatar_id":"","url":"https://api.github.com/users/lingxiankong","html_url":"https://github.com/lingxiankong","followers_url":"https://api.github.com/users/lingxiankong/followers","following_url":"https://api.github.com/users/lingxiankong/following{/other_user}","gists_url":"https://api.github.com/users/lingxiankong/gists{/gist_id}","starred_url":"https://api.github.com/users/lingxiankong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lingxiankong/subscriptions","organizations_url":"https://api.github.com/users/lingxiankong/orgs","repos_url":"https://api.github.com/users/lingxiankong/repos","events_url":"https://api.github.com/users/lingxiankong/events{/privacy}","received_events_url":"https://api.github.com/users/lingxiankong/received_events","type":"User","site_admin":false},"labels":[{"id":875191555,"node_id":"MDU6TGFiZWw4NzUxOTE1NTU=","url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/labels/cncf-cla:%20yes","name":"cncf-cla: yes","color":"bfe5bf","default":false},{"id":875378880,"node_id":"MDU6TGFiZWw4NzUzNzg4ODA=","url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/labels/needs-ok-to-test","name":"needs-ok-to-test","color":"b60205","default":false},{"id":875375867,"node_id":"MDU6TGFiZWw4NzUzNzU4Njc=","url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/labels/size/L","name":"size/L","color":"ee9900","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":17,"created_at":"2018-10-11T13:25:50Z","updated_at":"2018-10-11T15:29:51Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/pulls/322","html_url":"https://github.com/kubernetes/cloud-provider-openstack/pull/322","diff_url":"https://github.com/kubernetes/cloud-provider-openstack/pull/322.diff","patch_url":"https://github.com/kubernetes/cloud-provider-openstack/pull/322.patch"},"body":"<!-- Thanks for sending a pull request! -->\r\n\r\n**What this PR does / why we need it**:\r\nIf the ingress resource doesn't change, nothing needs to do when the\r\ningress controller restarts. In order to do that we need to store\r\nthe ingress resource version information in the load balancer's\r\ndescription field.\r\n\r\n**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #311\r\n\r\n**Special notes for your reviewer**:\r\nWith this PR, when the ingress controller restarts, for existing ingress that not changed, you can see the log below:\r\n```\r\nINFO[0000] ingress created or updated, will create or update octavia resources  ingress=default/test-octavia-ingress\r\nDEBU[0000] loadbalancer exists                           name=k8s-default-test-octavia-ingress-lb\r\nINFO[0000] loadbalancer active                           id=d15108bd-1f4a-49d5-8e8a-5c5fb80720d9 name=k8s-default-test-octavia-ingress-lb\r\nINFO[0000] ingress not change                            ingress=test-octavia-ingress\r\n```\r\n\r\n**Release note**:\r\n<!--  Steps to write your release note:\r\n1. Use the release-note-* labels to set the release note state (if you have access)\r\n2. Enter your extended release note in the below block; leaving it blank means using the PR title as the release note. If no release note is required, just write `NONE`.\r\n-->\r\n```release-note\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/comments/429000235","html_url":"https://github.com/kubernetes/cloud-provider-openstack/pull/322#issuecomment-429000235","issue_url":"https://api.github.com/repos/kubernetes/cloud-provider-openstack/issues/322","id":429000235,"node_id":"MDEyOklzc3VlQ29tbWVudDQyOTAwMDIzNQ==","user":{"login":"theopenlab-ci[bot]","id":33714816,"node_id":"MDM6Qm90MzM3MTQ4MTY=","avatar_url":"https://avatars0.githubusercontent.com/in/6778?v=4","gravatar_id":"","url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D","html_url":"https://github.com/apps/theopenlab-ci","followers_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/followers","following_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/repos","events_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/theopenlab-ci%5Bbot%5D/received_events","type":"Bot","site_admin":false},"created_at":"2018-10-11T15:29:51Z","updated_at":"2018-10-11T15:29:51Z","author_association":"NONE","body":"Build succeeded.\n\n- [cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12 ](http://logs.openlabtesting.org/logs/22/322/ae79a6fcd1d2fbfbf8ed9f0b6c5d8764158f7b7d/cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12/cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12/97903ab/) : SUCCESS in 2h 02m 54s\n"}}
{"actor":{"display_login":"theopenlab-ci"}}