)]}'
{"id":"openstack%2Fnetworking-ovn~688002","triplet_id":"openstack%2Fnetworking-ovn~stable%2Frocky~I3e9d62ffc5eab396e983c0769e53d2d1f35644f6","project":"openstack/networking-ovn","branch":"stable/rocky","topic":"bug/1843485","hashtags":[],"change_id":"I3e9d62ffc5eab396e983c0769e53d2d1f35644f6","subject":"Do not lose router logical port upon subnet add on the external net","status":"MERGED","created":"2019-10-10 22:56:54.000000000","updated":"2019-10-15 10:32:48.000000000","submitted":"2019-10-15 10:32:48.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"688002-1571135568314-bad3269c","meta_rev_id":"90aa858a2f486941a9d16235535fcb6d2ef4d103","_number":688002,"virtual_id_number":688002,"owner":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-10-15 10:32:48.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"all":[{"value":2,"date":"2019-10-15 08:25:22.000000000","_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":2,"date":"2019-10-14 14:47:39.000000000","_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"all":[{"value":1,"date":"2019-10-15 08:25:22.000000000","_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"date":"2019-10-14 10:55:55.000000000","_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-10-14 10:55:55.000000000","updated_by":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"reviewer":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"state":"REVIEWER"},{"updated":"2019-10-14 14:47:39.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"REVIEWER"},{"updated":"2019-10-15 08:25:22.000000000","updated_by":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"reviewer":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"state":"REVIEWER"},{"updated":"2019-10-15 10:32:48.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"dd87ada93d0c1b7eb131e9ebacc167c92117161a","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2019-10-10 22:56:54.000000000","message":"Patch Set 1: Cherry Picked from branch stable/stein.","accounts_in_message":[],"_revision_number":1},{"id":"f22a50865575b41df20e4e559bc53bafb7ee3019","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2019-10-10 22:57:33.000000000","message":"Patch Set 1: Cherry Picked\n\nThis patchset was cherry picked to branch stable/queens as commit 919eaabd5d94ed4838ba9c471b5a2e31fd5f08e1","accounts_in_message":[],"_revision_number":1},{"id":"41cdcbd73bbacfa6ddd62a2bc002626e4fa2fb62","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-11 01:40:01.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/48cec1257ee1473e9556804aac561aef : SUCCESS in 4m 31s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/2cffd9389f294f0bbf0b33401dd812f9 : SUCCESS in 8m 32s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/342fe7c167c5499e9ef9b38e391c0475 : SUCCESS in 3m 50s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/402ffe1a6757462fa2046d8758f67328 : SUCCESS in 9m 34s\n- networking-ovn-tempest-dsvm-ovs-release https://zuul.opendev.org/t/openstack/build/4305b6fa24f645aba402ae27274e5f25 : SUCCESS in 1h 29m 34s\n- networking-ovn-tempest-dsvm-ovs-release-python3 https://zuul.opendev.org/t/openstack/build/ba6a9cb685ec461f81599b29a21db1d8 : SUCCESS in 1h 33m 58s\n- networking-ovn-rally-task https://zuul.opendev.org/t/openstack/build/b3068252c163404abc8452331795fcd0 : SUCCESS in 1h 36m 55s\n- networking-ovn-dsvm-functional-py27 https://zuul.opendev.org/t/openstack/build/5f0556da19764c6d8df88c977cfb180f : SUCCESS in 22m 38s\n- networking-ovn-dsvm-functional https://zuul.opendev.org/t/openstack/build/ff3fd8b199be4ac6aaaa9020ab6399b2 : SUCCESS in 22m 49s\n- tripleo-ci-centos-7-scenario007-multinode-oooq-container https://zuul.opendev.org/t/openstack/build/3b4565147d9c4074a56e8ddadb884e6a : SUCCESS in 2h 38m 11s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/6b8621efd29343918a4538fd638fc8dd : SUCCESS in 10m 26s\n- networking-ovn-dsvm-grenade https://zuul.opendev.org/t/openstack/build/3494afd9c8cf46e0a7c7ab1492b49de3 : SUCCESS in 1h 48m 08s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/0c62edf2025e43e5bbf4476606147ff7 : SUCCESS in 11m 24s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f9cba4522062815b8fb869672e503b18b2c61522","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2019-10-14 10:50:08.000000000","message":"Topic set to bug/1843485","accounts_in_message":[],"_revision_number":1},{"id":"a10bfa088e042763df5799807be52ed37af7bf71","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2019-10-14 14:47:39.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"57ec7829187ad1e28a7b12d8354c296ef36d0645","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2019-10-15 08:25:22.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"5a6d8fd338b45df139a562be6d46c7cf3003303e","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-15 08:25:34.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"6d049d28482fa0392f3c984044f5a331a2c80bbd","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-15 10:32:48.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"90aa858a2f486941a9d16235535fcb6d2ef4d103","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-15 10:32:48.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/97c750b4a6944bf1bed16ba3da3a2400 : SUCCESS in 4m 37s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/9fa90bda8ab24a47934884935ca3c585 : SUCCESS in 8m 59s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/d131585937ea4b7a9c50aafde167ef79 : SUCCESS in 5m 24s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/7f27be5846f5440f97014f164efba57d : SUCCESS in 11m 45s\n- networking-ovn-tempest-dsvm-ovs-release https://zuul.opendev.org/t/openstack/build/54cd500595ed4efe94d74431359c05a8 : SUCCESS in 2h 03m 15s\n- networking-ovn-tempest-dsvm-ovs-release-python3 https://zuul.opendev.org/t/openstack/build/e3a856a8c2da443a8fefdcec30f7836d : SUCCESS in 1h 35m 22s\n- networking-ovn-rally-task https://zuul.opendev.org/t/openstack/build/c42a57f5d97a45a5ba0495b7b9c15383 : SUCCESS in 1h 42m 23s\n- networking-ovn-dsvm-functional-py27 https://zuul.opendev.org/t/openstack/build/7cac7892e84e451b8c3853c403e60242 : SUCCESS in 23m 28s\n- networking-ovn-dsvm-functional https://zuul.opendev.org/t/openstack/build/583c56d16f0241bf91c93ff215cc11f4 : SUCCESS in 18m 58s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/44ac4bbda7384060ad8212aa040fbcf4 : SUCCESS in 8m 28s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"bac75284f4600ebcc81e3a8decf9d21e35fa78f3","revisions":{"bac75284f4600ebcc81e3a8decf9d21e35fa78f3":{"kind":"REWORK","_number":1,"created":"2019-10-10 22:56:54.000000000","uploader":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"ref":"refs/changes/02/688002/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-ovn","ref":"refs/changes/02/688002/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-ovn refs/changes/02/688002/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-ovn refs/changes/02/688002/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-ovn refs/changes/02/688002/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-ovn refs/changes/02/688002/1"}}},"commit":{"parents":[{"commit":"964205b2a927e460ea5fa76ed4c6a052c979e741","subject":"Fix evacuation when host dies uncleanly","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-ovn/commit/964205b2a927e460ea5fa76ed4c6a052c979e741"}]}],"author":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2019-09-12 22:32:34.000000000","tz":-240},"committer":{"name":"Flavio Fernandes","email":"flavio@flaviof.com","date":"2019-10-10 22:56:54.000000000","tz":0},"subject":"Do not lose router logical port upon subnet add on the external net","message":"Do not lose router logical port upon subnet add on the external net\n\nPrior to this patch, logical port used by the router to the external\nnetwork was lost if subnet was added to that network. That was\ncaused by missing explicit delete call in DelLRouterPortCommand.\n\nAlso added functional test to exercise this particular code path:\n  test_router::test_gateway_chassis_with_subnet_changes()\n\nCloses-Bug: #1843485\nCo-authored-by: Daniel Alvarez \u003cdalvarez@redhat.com\u003e\nCo-authored-by: Terry Wilson \u003ctwilson@redhat.com\u003e\nSigned-off-by: Flavio Fernandes \u003cflaviof@redhat.com\u003e\n\nChange-Id: I3e9d62ffc5eab396e983c0769e53d2d1f35644f6\n(cherry picked from commit 92142f3165806527e33a4544162ad4eb301e8436)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-ovn/commit/bac75284f4600ebcc81e3a8decf9d21e35fa78f3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-ovn/commit/bac75284f4600ebcc81e3a8decf9d21e35fa78f3"}]},"branch":"refs/heads/stable/rocky"}},"requirements":[],"submit_records":[],"submit_requirements":[]}