)]}'
{"id":"openstack%2Fovsdbapp~726478","triplet_id":"openstack%2Fovsdbapp~master~Ie8ae8f5efae2afb958ed2010053ff04372cc1c51","project":"openstack/ovsdbapp","branch":"master","topic":"ovn/port_forwarding","hashtags":[],"change_id":"Ie8ae8f5efae2afb958ed2010053ff04372cc1c51","subject":"OVNNB backend: Fix LbDelCommand when using vip param","status":"MERGED","created":"2020-05-08 21:53:46.000000000","updated":"2020-08-20 19:30:56.000000000","submitted":"2020-05-15 22:02:10.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"726478-1589580130425-d6aca499","meta_rev_id":"a485c388bc8f86d41e4a073a39b4882e62765a13","_number":726478,"virtual_id_number":726478,"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":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-05-15 22:02:10.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"}],"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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"all":[{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-05-14 13:55:29.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":2,"date":"2020-05-14 12:24:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"}],"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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"all":[{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2020-05-15 19:57:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-05-14 12:24:43.000000000","updated_by":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"reviewer":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"state":"REVIEWER"},{"updated":"2020-05-15 19:57:01.000000000","updated_by":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"reviewer":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"state":"REVIEWER"},{"updated":"2020-05-15 22:02:10.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":"5715015da194d0ec63bd5ff7fa342723dd05325f","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-08 21:53:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e59271071d09139dcff0b7f39fb566716be27632","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-08 21:59:52.000000000","message":"Topic set to ovn/port_forwarding","accounts_in_message":[],"_revision_number":1},{"id":"5bd419f00ef14e955f73c0bebbdbdb0d1bddd41c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-09 00:29:07.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f0075d207a624398a031f42efee48775 : SUCCESS in 3m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4fb64475689c447a8ca33fb4040b76b9 : SUCCESS in 3m 41s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/6caeaf8058b24afc937cbdb13a89283d : SUCCESS in 3m 13s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/d80fae2fcc4e445a982c5bd7170ea11c : SUCCESS in 4m 49s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/d9df690faae84c4b9b1e04c22449d23e : SUCCESS in 4m 17s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/18003ba6f04241c7a65e5976670468ef : SUCCESS in 5m 35s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/04286cb77a124b859c4d56ed41eba224 : SUCCESS in 5m 34s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/c6512d1f9e44420384b05b90aa83c3a1 : SUCCESS in 2h 33m 34s","accounts_in_message":[],"_revision_number":1},{"id":"5816f2754ff724f96bad2c931eb3e3b145d7ad77","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-11 14:54:37.000000000","message":"Patch Set 1: Workflow-1\n\n(2 comments)\n\nlooks good, nice catch! where\u0027d you run into it? I don\u0027t see any code that passes in vip. :D\n\n-1 is just to handle the empty vips case mentioned in the comment to make it consistent with ovn-nbctl lb-del behavior. Nice catch! Thanks!","accounts_in_message":[],"_revision_number":1},{"id":"f2389342a9c1bc9a77ed036792e01461ef907347","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-11 15:12:33.000000000","message":"Removed Workflow-1 by Terry Wilson \u003ctwilson@redhat.com\u003e\n","accounts_in_message":[],"_revision_number":1},{"id":"7262bbd61f1f7344bcc2b0d805bf21a81e8e0342","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-11 15:12:43.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"9a14b91089676893dc66c299d5c4014b0ea4d7b8","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-11 16:09:31.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"80967c4e16710cbc12fb1f22947b859790267e9d","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-11 16:10:28.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"d08e6d1868fcc28d6e18d1573cceb8199a3d158d","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-11 16:13:11.000000000","message":"Patch Set 2:\n\n\u003e where\u0027d you run into it? I don\u0027t see any code that passes in vip. :D\n\nheh, that is code that I\u0027m about to add, in order to support the port forwarding rfe. ;)","accounts_in_message":[],"_revision_number":2},{"id":"2f5a26c140a36d1152980fa007aefd37773166b0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-11 18:15:00.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/1ad78fa2cf0647f3806d83e01dc5ac65 : SUCCESS in 5m 51s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5d028078938d4fa9890529986c3d8754 : SUCCESS in 5m 52s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4c9b3aca02e54ff09653e50071bca6b3 : SUCCESS in 5m 43s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/863d281a0a724f4693457fe8714dff1a : SUCCESS in 5m 47s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/606f7c9db03d41b8b3553fa194232f41 : SUCCESS in 5m 03s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f10f32680cea4fa1a5bbf3f54fcc7ef9 : SUCCESS in 7m 58s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/139409182fc24b639beeef2afc586adf : SUCCESS in 7m 55s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/4281895718304ea8aab811a2e86333db : SUCCESS in 1h 59m 08s","accounts_in_message":[],"_revision_number":2},{"id":"6d869b6d0d980f75bfa22c1b1bb7fb6ae3beed48","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-11 19:10:24.000000000","message":"Patch Set 2: Code-Review+2\n\nNice catch! Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"367b6800734c5e4d7540a59ab26560e1871c3edf","author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"date":"2020-05-14 08:56:52.000000000","message":"Patch Set 2:\n\nWe need to update the docs - https://github.com/openstack/ovsdbapp/blob/master/ovsdbapp/schema/ovn_northbound/api.py#L690\n\nIt\u0027s not clear to which resource the if-exists parameter ties to","accounts_in_message":[],"_revision_number":2},{"id":"89d1762f950dbf7f6b7da49378ce8100b356a63f","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-14 09:53:16.000000000","message":"Patch Set 2: Workflow-1\n\n\u003e We need to update the docs - https://github.com/openstack/ovsdbapp/blob/master/ovsdbapp/schema/ovn_northbound/api.py#L690\n \u003e \n \u003e It\u0027s not clear to which resource the if-exists parameter ties to\n\nValid point! Let me make the docs clearer about the behavior of lb_del api","accounts_in_message":[],"_revision_number":2},{"id":"0ffde1aa4e678b842684233de707bafcd0e535b9","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-05-14 12:21:53.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"5157b72da23be0d01c763dd1d0ccca43a5e360bf","author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"date":"2020-05-14 12:24:43.000000000","message":"Patch Set 3: Code-Review+2\n\nThanks for the docstring change Flavio!","accounts_in_message":[],"_revision_number":3},{"id":"18024d3db2a1097976302fd27ba23c3d91b56d87","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-14 13:55:29.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"91d375dab44389fdefd04f87f409f1f84174957a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-14 14:41:45.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/48d60d18ad744d48a332d2fbb2a26e55 : SUCCESS in 4m 21s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/62e5bb0704f04c17a7c3254f371faa75 : SUCCESS in 6m 42s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/13a21fdc5f424036b972b8b752100312 : SUCCESS in 4m 24s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/b4ba4bd23560413a8e8a8d58a10e6f72 : SUCCESS in 6m 52s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/f6c32c36b06f4c2784a73f19e47ce570 : SUCCESS in 3m 44s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/61e8c04c1f044b9fbf824ff023c21668 : SUCCESS in 6m 46s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/03e9d4505c474e9782344a075ad4315d : SUCCESS in 5m 45s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/54ed4c532c91481d96d9885fb0d7d375 : SUCCESS in 2h 14m 11s","accounts_in_message":[],"_revision_number":3},{"id":"2a93024b11eecc8686f34a0990818d145fa730d8","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-05-15 19:57:01.000000000","message":"Patch Set 3: Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"d7b66c02c039ed24449db191faf5bae7969c5ca4","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-15 19:57:14.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"3d73e50fa89f99f6af2985ea5f2e3be0f39c4cad","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-15 22:02:10.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"78490ac1c3e96633507b44c6f237fd755d56b7d7","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-15 22:02:10.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/75de4f180f264170bf6aa5984cc21f19 : SUCCESS in 3m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/bb383573d94f45e4bdc2c279841ff933 : SUCCESS in 4m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/04f3f50e08124221bf38cf9e869eb674 : SUCCESS in 5m 57s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a742a79e432c48cba7c88b536d90020c : SUCCESS in 4m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/64ea7523770a426e96f4a6a5b87558d6 : SUCCESS in 6m 12s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/17805c525f464b3d942ab3594c64cc8a : SUCCESS in 5m 52s\n- ovsdbapp-neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/97af2bd04ac4421d99b75a26ce57ffa1 : SUCCESS in 1h 47m 57s","accounts_in_message":[],"_revision_number":3},{"id":"a485c388bc8f86d41e4a073a39b4882e62765a13","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2020-08-20 19:30:56.000000000","message":"Patch Set 3: Cherry Picked\n\nThis patchset was cherry picked to branch stable/ussuri as commit e8e64da0ea7f2e3fa6bb2641729f0b0ae0b64f44","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"8cb3b56fa9022254d60a15125f86f6e8a7e16041","revisions":{"6baba06029f2aefe1d10ea472e74f4082a3fe181":{"kind":"REWORK","_number":1,"created":"2020-05-08 21:53:46.000000000","uploader":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"ref":"refs/changes/78/726478/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/78/726478/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/1"}}},"commit":{"parents":[{"commit":"f8235d80225b7fcbb539679cb3d6fe2c585029ae","subject":"Merge \"Add \"inactivity_probe\" parameter to \"enable_connection_uri\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/f8235d80225b7fcbb539679cb3d6fe2c585029ae"}]}],"author":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-08 21:40:45.000000000","tz":-240},"committer":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-08 21:51:36.000000000","tz":-240},"subject":"OVNNB backend: Fix LbDelCommand when using vip param","message":"OVNNB backend: Fix LbDelCommand when using vip param\n\nFix issue in ovn_northbound LbDelCommand where logic\nfor removing vip while using if_exists\u003dTrue was not\ndoing the right thing.\n\nCloses-Bug: #1877673\nChange-Id: Ie8ae8f5efae2afb958ed2010053ff04372cc1c51\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/6baba06029f2aefe1d10ea472e74f4082a3fe181"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/6baba06029f2aefe1d10ea472e74f4082a3fe181"}]},"branch":"refs/heads/master"},"044ec7afee5dc920cd15458cd2c0bf1d50ff3174":{"kind":"REWORK","_number":2,"created":"2020-05-11 16:09:31.000000000","uploader":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"ref":"refs/changes/78/726478/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/78/726478/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/2"}}},"commit":{"parents":[{"commit":"f8235d80225b7fcbb539679cb3d6fe2c585029ae","subject":"Merge \"Add \"inactivity_probe\" parameter to \"enable_connection_uri\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/f8235d80225b7fcbb539679cb3d6fe2c585029ae"}]}],"author":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-08 21:40:45.000000000","tz":-240},"committer":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-11 16:03:56.000000000","tz":-240},"subject":"OVNNB backend: Fix LbDelCommand when using vip param","message":"OVNNB backend: Fix LbDelCommand when using vip param\n\nFix issue in ovn_northbound LbDelCommand where logic\nfor removing vip while using if_exists\u003dTrue was not\ndoing the right thing.\n\nIn order to be consistent with as ovn-nbctl [1], this\nchange also introduces a modified behavior. When the\nlast vip is removed, it will also remove the load\nbalancer instance.\n\n[1]: https://github.com/ovn-org/ovn/blob/1e2f4aaab67090935bba6347358ad9c56b0868bf/utilities/ovn-nbctl.c#L2870\n\nCloses-Bug: #1877673\nChange-Id: Ie8ae8f5efae2afb958ed2010053ff04372cc1c51\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/044ec7afee5dc920cd15458cd2c0bf1d50ff3174"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/044ec7afee5dc920cd15458cd2c0bf1d50ff3174"}]},"branch":"refs/heads/master"},"8cb3b56fa9022254d60a15125f86f6e8a7e16041":{"kind":"REWORK","_number":3,"created":"2020-05-14 12:21:53.000000000","uploader":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"ref":"refs/changes/78/726478/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/78/726478/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/78/726478/3"}}},"commit":{"parents":[{"commit":"f8235d80225b7fcbb539679cb3d6fe2c585029ae","subject":"Merge \"Add \"inactivity_probe\" parameter to \"enable_connection_uri\"\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/f8235d80225b7fcbb539679cb3d6fe2c585029ae"}]}],"author":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-08 21:40:45.000000000","tz":-240},"committer":{"name":"Flavio Fernandes","email":"flaviof@redhat.com","date":"2020-05-14 12:21:50.000000000","tz":-240},"subject":"OVNNB backend: Fix LbDelCommand when using vip param","message":"OVNNB backend: Fix LbDelCommand when using vip param\n\nFix issue in ovn_northbound LbDelCommand where logic\nfor removing vip while using if_exists\u003dTrue was not\ndoing the right thing.\n\nIn order to be consistent with as ovn-nbctl [1], this\nchange also introduces a modified behavior. When the\nlast vip is removed, it will also remove the load\nbalancer instance.\n\n[1]: https://github.com/ovn-org/ovn/blob/1e2f4aaab67090935bba6347358ad9c56b0868bf/utilities/ovn-nbctl.c#L2870\n\nCloses-Bug: #1877673\nChange-Id: Ie8ae8f5efae2afb958ed2010053ff04372cc1c51\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/8cb3b56fa9022254d60a15125f86f6e8a7e16041"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/8cb3b56fa9022254d60a15125f86f6e8a7e16041"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}