)]}'
{"id":"openstack%2Fneutron~697021","triplet_id":"openstack%2Fneutron~stable%2Fqueens~I06036ce5fe15d91aa440dc340a70dd27ae078c53","project":"openstack/neutron","branch":"stable/queens","topic":"bug/1850557-stable/queens","hashtags":[],"change_id":"I06036ce5fe15d91aa440dc340a70dd27ae078c53","subject":"[OVS FW] Clean port rules if port not found in ovsdb","status":"MERGED","created":"2019-12-03 08:21:36.000000000","updated":"2019-12-04 01:58:42.000000000","submitted":"2019-12-04 01:58:42.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":"697021-1575424722843-19300d8e","meta_rev_id":"5254ffbd77c6df0194d4d094b5407e57fadd9cf5","_number":697021,"virtual_id_number":697021,"owner":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:jenkins-gerrit-trigger","value":0,"date":"2019-12-03 09:31:41.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-12-04 01:58:42.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2019-12-03 09:22:51.000000000","_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"all":[{"value":0,"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":2,"date":"2019-12-03 16:32:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":2,"date":"2019-12-04 00:27:28.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"all":[{"value":0,"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":1,"date":"2019-12-04 00:27:28.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-12-03 09:22:51.000000000","updated_by":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"reviewer":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"state":"REVIEWER"},{"updated":"2019-12-03 09:31:41.000000000","updated_by":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-12-03 16:32:11.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-12-04 00:27:28.000000000","updated_by":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"reviewer":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"state":"REVIEWER"},{"updated":"2019-12-04 01:58:42.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":"85050234772eace6f62b36b379f57275fa78e505","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2019-12-03 08:21:36.000000000","message":"Patch Set 1: Cherry Picked from branch stable/rocky.","accounts_in_message":[],"_revision_number":1},{"id":"f84b339fa392d8a0175e25957ede24a492dd5f66","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2019-12-03 09:22:51.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Neutron Hyper-V CI, add a review comment with run-Cloudbase Neutron Hyper-V CI\n\n- neutron http://cloudbase-ci.com/neutron/697021/1 : FAILURE in 1h 00m 43s","accounts_in_message":[],"_revision_number":1},{"id":"7988cde6ba1ad9f585ae0d9134881ab44e5aa8e1","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2019-12-03 09:31:41.000000000","message":"Patch Set 1:\n\nBuild Failed \n\n* Neutron-MACVTAP-ML2-Sriov http://13.74.249.42/refs/changes/21/697021/1/Neutron-MACVTAP-ML2-Sriov : FAILURE\n\n* Neutron-ML2-Sriov http://13.74.249.42/refs/changes/21/697021/1/Neutron-ML2-Sriov : FAILURE","accounts_in_message":[],"_revision_number":1},{"id":"48f574904b1bcec84e2472c6c347c37833e2fd18","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-03 10:11:11.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api-queens https://zuul.opendev.org/t/openstack/build/0928878a8faa4b3e90709c30384b6d7f : SUCCESS in 1h 12m 01s\n- neutron-tempest-plugin-designate-scenario-queens https://zuul.opendev.org/t/openstack/build/139b73ee616f46c98831141189b20b55 : SUCCESS in 46m 55s\n- neutron-tempest-plugin-dvr-multinode-scenario-queens https://zuul.opendev.org/t/openstack/build/f7b7d212ee0848429f8c491a9c071e6b : SUCCESS in 1h 47m 03s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge-queens https://zuul.opendev.org/t/openstack/build/31c45605ac77491e9094c3e47fc7da34 : SUCCESS in 1h 06m 37s\n- neutron-tempest-plugin-scenario-openvswitch-queens https://zuul.opendev.org/t/openstack/build/5a5159cb5e0e4698be9238c5dcf0cddb : SUCCESS in 52m 34s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/bd3941f045bb4905867d488639163317 : SUCCESS in 12m 42s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/de24139f5af74357a9356cd348c26618 : SUCCESS in 27m 31s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/a92f606fbe5242e294c44e40fc38e317 : SUCCESS in 22m 13s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/5c35e6a2d62d455abf2efbe5cfbc7796 : SUCCESS in 6m 19s\n- tempest-full https://zuul.opendev.org/t/openstack/build/056253264902412cb35858701ebbfb1c : SUCCESS in 1h 13m 12s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/327b9a7731e847e480428d6822702097 : SUCCESS in 54m 20s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/71f8d4b8f9924e0986ce052ccbf152a1 : SUCCESS in 1h 15m 56s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/124315c39e9145b6b38b87c6550d7935 : SUCCESS in 38m 16s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/6eda96a1725242fab56b3168c34880bf : FAILURE in 28m 15s (non-voting)\n- neutron-rally-neutron https://zuul.opendev.org/t/openstack/build/6d63eb381f2341638c0383681d12a1c5 : SUCCESS in 1h 30m 02s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/f350c9cad1df4d1686657bda98cac142 : SUCCESS in 1h 23m 58s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/e6ac322a72984daaaa9565bbe8cb37e8 : SUCCESS in 1h 18m 33s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/5fe7a3691c764040bf92c27858b20e6e : SUCCESS in 58m 46s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/4b452692ca3c400088ca63f308b5453f : SUCCESS in 59m 44s\n- neutron-tempest-multinode-full https://zuul.opendev.org/t/openstack/build/5965905db3ec47eaa92efe41578ca532 : SUCCESS in 1h 28m 09s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/fb14a3db3a64402daa6e69a175045690 : SUCCESS in 1h 22m 36s (non-voting)\n- neutron-tempest-ovsfw https://zuul.opendev.org/t/openstack/build/a6bee43d26db4e758b05e4a26cc561e4 : FAILURE in 1h 10m 00s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/dd0e602eb7714ecb8947824705ce572e : SUCCESS in 28m 21s","accounts_in_message":[],"_revision_number":1},{"id":"55145b2c71946bfb19b66eea461e400691b94ec7","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2019-12-03 16:32:11.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"33f517bce9cc6667b138e172d1b1c343692489ea","author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"date":"2019-12-04 00:27:28.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"0e8eb65b2ea0f349f5bfeb1c9e18528ef5c41dbc","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-04 00:27:48.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"fa68349fb494e4ff4b18d19224397de50a721390","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-04 01:58:42.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- neutron-tempest-plugin-api-queens https://zuul.opendev.org/t/openstack/build/1a6dc73c910b418dbe33367b56952ece : SUCCESS in 39m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8223b6a082bc4861ac8b576877772aba : SUCCESS in 10m 37s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/0b2b5def12814209b05006af1965637a : SUCCESS in 22m 19s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/d9f87a6c5eff40fc96f7d8f281e1a101 : SUCCESS in 21m 28s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/16623af413dc40d9aa83d4578a4b0619 : SUCCESS in 5m 03s\n- tempest-full https://zuul.opendev.org/t/openstack/build/f8c068b1046246a6a42b2a77e43418c0 : SUCCESS in 1h 19m 30s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/c0ce10a85ff74bea8521eba211c40e6a : SUCCESS in 56m 09s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/3ad1560425134f92830dfab265302447 : SUCCESS in 1h 24m 31s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/18655ee82d0842d98073788e724945bb : SUCCESS in 36m 57s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/370b3107a3d74cbf9962a5b33385efef : SUCCESS in 1h 23m 53s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/5d0386ee2e4e4fcdbfc0f6e608d00f62 : SUCCESS in 1h 16m 40s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/fdd2d41d4f324aaa9d7addc9ff2754ee : SUCCESS in 1h 07m 21s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/08454f5a121b4102905df8dbf6f28f55 : SUCCESS in 1h 02m 17s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/f21f20c6d5e64631bd75bab9dabacdb8 : SUCCESS in 31m 22s","accounts_in_message":[],"_revision_number":1},{"id":"77f392abcb63db378ff34b7f2bbcaef100477139","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-04 01:58:42.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ef689b284c079fba497723fb779603aa99d8afff","revisions":{"ef689b284c079fba497723fb779603aa99d8afff":{"kind":"REWORK","_number":1,"created":"2019-12-03 08:21:36.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/21/697021/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/21/697021/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/697021/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/697021/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/21/697021/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/21/697021/1"}}},"commit":{"parents":[{"commit":"a4633151d360aae769fbf75a0081054e2d348992","subject":"Merge \"Run revision bump operations en masse\" into stable/queens","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/a4633151d360aae769fbf75a0081054e2d348992"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2019-11-27 09:44:19.000000000","tz":60},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2019-12-03 08:21:36.000000000","tz":0},"subject":"[OVS FW] Clean port rules if port not found in ovsdb","message":"[OVS FW] Clean port rules if port not found in ovsdb\n\nDuring e.g. migration or shelve of VM it may happend that\nport update event will be send to the ovs agent and in the almost\nthe same time, port will be removed from br-int.\nIn such case during update_port_filter method openvswitch firewall\ndriver will not find port in br-int, and it will do nothing with it.\nThat will lead to leftover rules for this port in br-int.\n\nSo this patch adds calling remove_port_filter() method if port was\nnot found in br-int. Just to be sure that there is no any leftovers\nfrom the port in br-int anymore.\n\nConflicts:\n    neutron/agent/linux/openvswitch_firewall/firewall.py\n\nChange-Id: I06036ce5fe15d91aa440dc340a70dd27ae078c53\nCloses-Bug: #1850557\n(cherry picked from commit b01e0c2aa98866df9c25e20a66c02fccccdc7885)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/ef689b284c079fba497723fb779603aa99d8afff"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/ef689b284c079fba497723fb779603aa99d8afff"}]},"branch":"refs/heads/stable/queens"}},"requirements":[],"submit_records":[],"submit_requirements":[]}