)]}'
{"id":"openstack%2Fnetworking-sfc~926892","triplet_id":"openstack%2Fnetworking-sfc~master~I76099a4cae2b47a748478525c25882538390bb91","project":"openstack/networking-sfc","branch":"master","topic":"bug/2068727","attention_set":{},"removed_from_attention_set":{"1131":{"account":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"last_update":"2024-08-22 16:53:07.000000000","reason":"\u003cGERRIT_ACCOUNT_1131\u003e replied on the change","reason_account":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}},"11975":{"account":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"last_update":"2024-08-23 15:43:44.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I76099a4cae2b47a748478525c25882538390bb91","subject":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id","status":"MERGED","created":"2024-08-22 10:28:43.000000000","updated":"2024-08-23 15:44:52.000000000","submitted":"2024-08-23 15:43:44.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":12,"unresolved_comment_count":3,"has_review_started":true,"submission_id":"926892-bug/2068727","meta_rev_id":"e596e47a58fa419a235f1227e82899c209dc7e59","_number":926892,"virtual_id_number":926892,"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:zuul:gate","value":2,"date":"2024-08-23 15:43:44.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}],"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":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2024-08-23 14:00:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},{"value":2,"date":"2024-08-22 16:53:07.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}],"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":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2024-08-23 14:00:23.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}],"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":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2024-08-22 13:12:02.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"},{"updated":"2024-08-22 13:44:57.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":"CC"},{"updated":"2024-08-22 16:53:07.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":"2024-08-23 14:00:23.000000000","updated_by":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"reviewer":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"state":"REVIEWER"}],"messages":[{"id":"85494644fc020a0dedcc25b2ada5fccc1fb4ba52","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2024-08-22 10:28:43.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4922e4b7f3541d3bbb8dfc5b411815b3eeb84e46","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-22 13:12:02.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/b6b151ce3fcd418e9794e002af66690a\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5ec7a911724e4c4d835825e779dcf36f : SUCCESS in 10m 02s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1a4751b6493c42628e5bc40a529fd409 : SUCCESS in 6m 37s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/f52d04fb02a948a2b3b5b3e1ab7c8bc6 : SUCCESS in 7m 40s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/cda722dc54ac4b68ab3fb5b77725d72c : SUCCESS in 7m 47s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/16606ea3f49341e9a5bb8cf50e7b31f8 : SUCCESS in 8m 47s\n- networking-sfc-functional https://zuul.opendev.org/t/openstack/build/f499e94422464e2abeec6c3cf5166e70 : SUCCESS in 16m 50s\n- networking-sfc-tempest https://zuul.opendev.org/t/openstack/build/fcdd8416c0a5472a9459a3c925da97d2 : FAILURE in 2h 27m 21s\n- networking-sfc-tempest-multinode https://zuul.opendev.org/t/openstack/build/363e76b1d6724051a4fa341779cf4ff6 : FAILURE in 2h 42m 38s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b8378bbf472eb71b4533c4dfec4a6ce604b403f5","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2024-08-22 13:44:57.000000000","message":"Patch Set 1:\n\n(4 comments)","accounts_in_message":[],"_revision_number":1},{"id":"25149754d4e9e5ae0b0e00d48495473d28bed1dd","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2024-08-22 14:04:36.000000000","message":"Patch Set 2: Commit message was updated.\n\nOutdated Votes:\n* Verified-1\n","accounts_in_message":[],"_revision_number":2},{"id":"92aecd74db29c0fd0f0009f3fe1f8a52a7efa369","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-22 14:36:18.000000000","message":"Patch Set 2: Verified-1\n\nThis change depends on a change that failed to merge.\n\nChange https://review.opendev.org/926915 is needed.","accounts_in_message":[],"_revision_number":2},{"id":"978eb782fae307ea1bf1ef7924542b754a0268b2","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2024-08-22 14:43:37.000000000","message":"Patch Set 2:\n\n(3 comments)","accounts_in_message":[],"_revision_number":2},{"id":"d383d8611f7bb9305db4fcfffbca27d13bd63ede","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2024-08-22 14:43:40.000000000","message":"Uploaded patch set 3.\n\nOutdated Votes:\n* Verified-1\n","accounts_in_message":[],"_revision_number":3},{"id":"28ef2c07af9d1c5163829c1aaabb07d33b2cbe2d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-22 16:47:38.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/89c6ed4f978c40e988758d3fd2fd2920\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f24a7f36f5684695bdaebabfba897ed6 : SUCCESS in 12m 55s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/cad6fbbf728b48bb9b59e8a903079006 : SUCCESS in 6m 40s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/d3f0c36c14384466944251f71c516708 : SUCCESS in 7m 45s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/92f0c27faf0d4958bc6a894a2cdebec1 : SUCCESS in 8m 55s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dd14bfc040c64c00a488cff8ec188d42 : SUCCESS in 12m 04s\n- networking-sfc-functional https://zuul.opendev.org/t/openstack/build/df1edbc10cf14f87983145c7acca0872 : SUCCESS in 16m 49s\n- networking-sfc-tempest https://zuul.opendev.org/t/openstack/build/602e91a01a7e41d4b1b2a3fd3664171a : SUCCESS in 1h 45m 49s\n- networking-sfc-tempest-multinode https://zuul.opendev.org/t/openstack/build/4fe52b94a2764df984668a6963d9e340 : SUCCESS in 2h 00m 46s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"893102b35d07581807ccd0d606e03082fb4f312d","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2024-08-22 16:53:07.000000000","message":"Patch Set 3: Code-Review+2\n\n(4 comments)","accounts_in_message":[],"_revision_number":3},{"id":"946ee9e31da37afed519e130a4df15ab402c7ad9","author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"date":"2024-08-23 14:00:23.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"cbc574be09e9aee637e054d86c1609a846d971ec","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-23 14:00:54.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"41dd8ff50e650bf0d9774cb947e0537c192f3007","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-23 15:43:44.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a8f9b3537c8d4b0abe03f64008fd8340\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dd4c237f54cc40ea88b311e11d95016d : SUCCESS in 10m 49s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/fd0806cd25174b50a904d30f43d6edb8 : SUCCESS in 8m 34s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/fc6816755c2f4dc19bfe2ccc3d0d878a : SUCCESS in 7m 47s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b63f98306a0b45d7af1ed60933d31b90 : SUCCESS in 7m 58s\n- networking-sfc-functional https://zuul.opendev.org/t/openstack/build/90dd43d114cb4be3a7f0482b6a28807d : SUCCESS in 15m 41s\n- networking-sfc-tempest https://zuul.opendev.org/t/openstack/build/278ec305dcca432d825675a41168cabf : SUCCESS in 1h 38m 25s","accounts_in_message":[],"_revision_number":3},{"id":"380754fddbaaf8e48ed867894d009efb99848a83","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-23 15:43:44.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":3},{"id":"e596e47a58fa419a235f1227e82899c209dc7e59","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-08-23 15:44:52.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/827c64ed050e449e85df8837d0041b88\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/501df26f42404e93847c60e8f5d12641 : SUCCESS in 48s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"94d303464bfc9dc2eb244004924ad685e78053f8","revisions":{"41508e5c17cfbebea01b168c93fdf6f2345e9775":{"kind":"REWORK","_number":1,"created":"2024-08-22 10:28:43.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/92/926892/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-sfc","ref":"refs/changes/92/926892/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/1"}}},"commit":{"parents":[{"commit":"bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18","subject":"Merge \"Add py312 job to periodic based on testing runtime for 2024.2\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 10:24:08.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 10:28:39.000000000","tz":120},"subject":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id","message":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id\n\nPrivate method _delete_ports_flowrules_by_id was done in the way that it\nwas expecting list of ports given as \"ports_id\" variable but the only\nplace where this method was called is passing ID of the one port only.\nAs it was not list but string, iteraton was done over string elements\nand that was causing plenty of messages send to the neutron server for\nevery char of the each deleted port if. This was overloading\nneutron-ovs-agent.\n\nThis patch changes that private method to be\n_delete_port_flowrules_by_id and to expect only one port id to be passed\nto it as argument as there is no any place in the code where old method\nwould be called with list of the ports.\n\nCloses-bug: #2068727\nChange-Id: I76099a4cae2b47a748478525c25882538390bb91\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/41508e5c17cfbebea01b168c93fdf6f2345e9775"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/41508e5c17cfbebea01b168c93fdf6f2345e9775"}]},"branch":"refs/heads/master"},"78a54b17adafdcaeab2b8488573d3be2a62ed2c8":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2024-08-22 14:04:36.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/92/926892/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-sfc","ref":"refs/changes/92/926892/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/2"}}},"commit":{"parents":[{"commit":"bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18","subject":"Merge \"Add py312 job to periodic based on testing runtime for 2024.2\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 10:24:08.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 14:04:36.000000000","tz":0},"subject":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id","message":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id\n\nPrivate method _delete_ports_flowrules_by_id was done in the way that it\nwas expecting list of ports given as \"ports_id\" variable but the only\nplace where this method was called is passing ID of the one port only.\nAs it was not list but string, iteraton was done over string elements\nand that was causing plenty of messages send to the neutron server for\nevery char of the each deleted port if. This was overloading\nneutron-ovs-agent.\n\nThis patch changes that private method to be\n_delete_port_flowrules_by_id and to expect only one port id to be passed\nto it as argument as there is no any place in the code where old method\nwould be called with list of the ports.\n\nDepends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926915\n\nCloses-bug: #2068727\nChange-Id: I76099a4cae2b47a748478525c25882538390bb91\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/78a54b17adafdcaeab2b8488573d3be2a62ed2c8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/78a54b17adafdcaeab2b8488573d3be2a62ed2c8"}]},"branch":"refs/heads/master","description":"Edit commit message"},"94d303464bfc9dc2eb244004924ad685e78053f8":{"kind":"REWORK","_number":3,"created":"2024-08-22 14:43:40.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/92/926892/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-sfc","ref":"refs/changes/92/926892/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-sfc refs/changes/92/926892/3"}}},"commit":{"parents":[{"commit":"bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18","subject":"Merge \"Add py312 job to periodic based on testing runtime for 2024.2\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/bcc6be3e8b4e89ad6bb6bff66d2c537ee8c77f18"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 10:24:08.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2024-08-22 14:43:23.000000000","tz":120},"subject":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id","message":"Fix _delete_ports_flowrules_by_id so that it expects only one port_id\n\nPrivate method _delete_ports_flowrules_by_id was done in the way that it\nwas expecting list of ports given as \"ports_id\" variable but the only\nplace where this method was called is passing ID of the one port only.\nAs it was not list but string, iteraton was done over string elements\nand that was causing plenty of messages send to the neutron server for\nevery char of the each deleted port if. This was overloading\nneutron-ovs-agent.\n\nThis patch changes that private method to be\n_delete_port_flowrules_by_id and to expect only one port id to be passed\nto it as argument as there is no any place in the code where old method\nwould be called with list of the ports.\n\nDepends-On: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926915\n\nCloses-bug: #2068727\nChange-Id: I76099a4cae2b47a748478525c25882538390bb91\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/94d303464bfc9dc2eb244004924ad685e78053f8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-sfc/commit/94d303464bfc9dc2eb244004924ad685e78053f8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dMAX"],"failing_atoms":["label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dMAX"],"failing_atoms":["label:Workflow\u003dMIN"],"atom_explanations":{}}}]}