)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"16b4282a3da23c7ddd2ccaf7ff74f47fabf247c7","unresolved":true,"context_lines":[{"line_number":13,"context_line":"Closes-Bug: #1943724"},{"line_number":14,"context_line":"Partial-Bug: #1922237"},{"line_number":15,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/809168/"},{"line_number":16,"context_line":"Depends-On: https://review.opendev.org/c/openstack/nova/+/805046"},{"line_number":17,"context_line":"See-Also: https://review.opendev.org/785236"},{"line_number":18,"context_line":"Change-Id: I99a49b107b1872ddf83d1d8497a26a8d728feb07"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"ba65f31b_fea454d2","line":16,"updated":"2021-09-29 14:31:27.000000000","message":"I think you can drop this dependency. Functionally what is needed in nova is already landed in Xena.","commit_id":"9ec5e5849ec3fdf9498d67f119557be9aa7be3c2"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"4a3b9c4f65501f112b59b67a8e3c461c7f385f3b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Closes-Bug: #1943724"},{"line_number":14,"context_line":"Partial-Bug: #1922237"},{"line_number":15,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/809168/"},{"line_number":16,"context_line":"Depends-On: https://review.opendev.org/c/openstack/nova/+/805046"},{"line_number":17,"context_line":"See-Also: https://review.opendev.org/785236"},{"line_number":18,"context_line":"Change-Id: I99a49b107b1872ddf83d1d8497a26a8d728feb07"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"aa8d6288_c6c17873","line":16,"in_reply_to":"ba65f31b_fea454d2","updated":"2021-10-11 13:38:00.000000000","message":"Done","commit_id":"9ec5e5849ec3fdf9498d67f119557be9aa7be3c2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"84fa51a85c5007f2f42a030294cccf687d12c167","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"18fcc186_100bc039","updated":"2021-10-18 11:11:58.000000000","message":"The tests in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/806689 passed on it.","commit_id":"3e905d1cdc48e7b3648c1dd5e8bd9c31b16abbfb"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"0887e8cf9c2d799f08f0c5a273f86961c03659c7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":11,"id":"e488dbd2_50da0de0","updated":"2021-10-12 06:58:15.000000000","message":"recheck","commit_id":"3e905d1cdc48e7b3648c1dd5e8bd9c31b16abbfb"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"0f720436d7a0cfde61e09810a72a9787d2c81496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2fd19972_9c783f6e","updated":"2021-10-30 17:53:39.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"ed5596d6976d40cb40029afb1a3ab4ce54c93518","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"31c3e5a3_bb58e2ec","updated":"2021-10-29 14:55:48.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"580ddaa92001fc5135efb4eab3e4d2aa6baa23e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"66bf203c_6a4d3867","updated":"2021-10-30 02:15:01.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ef42db305381e0cb8f2b09f85cabbb6ed50d2dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"67b128cc_c1d7b661","updated":"2021-10-31 05:33:19.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"7904b500d2dcbeae8b9aace4952d5b00becf2c70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"b798900f_5551aac5","updated":"2021-10-30 05:31:10.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"4400eb3ae310a47e10be3887e909e5015a4b3a65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"c9fee7ae_3c0fa6ab","updated":"2021-10-28 14:07:44.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"38c6a55595ceaec843ce7e014d716b0ab7ff4865","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"caadc9cd_382abfca","updated":"2021-10-29 09:03:19.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"de772e394706ef457f5acf5f107d5a004affcde4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"e774bd4a_487fd086","updated":"2021-10-30 13:31:03.000000000","message":"recheck","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"9925c99cb9583c419228aa6c405f2aef72dedd1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"6c1cb2f1_b2370b00","updated":"2021-10-28 09:10:23.000000000","message":"recheck to see if the latest tempest change still works with this","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"4400eb3ae310a47e10be3887e909e5015a4b3a65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"7832daa2_56b177b9","in_reply_to":"6c1cb2f1_b2370b00","updated":"2021-10-28 14:07:44.000000000","message":"qos tempest tests are passing \\o/","commit_id":"8db15cb2f3dec37441103df15ff28c7e72de682c"}],"neutron/exceptions/qos.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"93b19cc57d4195a5790e0dca9d9b78fe1ac3261c","unresolved":true,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from neutron._i18n import _"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class QosPlacementAllocationUpdateConflict(e.Conflict):"},{"line_number":21,"context_line":"    message \u003d _(\"Updating placement allocation with %(alloc_diff)s for \""},{"line_number":22,"context_line":"                \"consumer %(consumer)s failed. The requested resources would \""}],"source_content_type":"text/x-python","patch_set":7,"id":"0f068456_9822ea91","line":19,"updated":"2021-09-23 13:23:59.000000000","message":"Add a note to move this to n-lib","commit_id":"e5e7e1b683fa149bdaacca223d518c49345b4e34"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"f9f303c74e80e3cfbae86e6e86d3a7d09ff27923","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from neutron._i18n import _"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class QosPlacementAllocationUpdateConflict(e.Conflict):"},{"line_number":21,"context_line":"    message \u003d _(\"Updating placement allocation with %(alloc_diff)s for \""},{"line_number":22,"context_line":"                \"consumer %(consumer)s failed. The requested resources would \""}],"source_content_type":"text/x-python","patch_set":7,"id":"24381add_172b21fb","line":19,"in_reply_to":"0f068456_9822ea91","updated":"2021-09-23 14:51:45.000000000","message":"Done","commit_id":"e5e7e1b683fa149bdaacca223d518c49345b4e34"}],"neutron/plugins/ml2/drivers/mech_agent.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"ece77316efcfd02e076599b97e5b70ae585b12f9","unresolved":true,"context_lines":[{"line_number":257,"context_line":"                # not responsible, must be somebody else"},{"line_number":258,"context_line":"                return False"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        return len(reported) \u003d\u003d len(allocation)"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"class SimpleAgentMechanismDriverBase(AgentMechanismDriverBase,"}],"source_content_type":"text/x-python","patch_set":10,"id":"96cfe5e2_53a34161","line":260,"updated":"2021-09-29 12:18:48.000000000","message":"during a QoS policy update on a bound port from a QoS policy to None this code is reporting all agents on the compute to be responsible for the port allocation. This is now due to the fact that both the allocation and the reported dict is empty. \n\nI\u0027m not sure what to do here. I\u0027m not sure ignoring this case is always good. E.g if there is an agent that implements data plane enforcement (I belive sriov agent implements it for bandwidth) and then the a bound port with QoS(min_bw\u003d1000) is update to QoS\u003dNone, then the sriov agent should know that the min_bw request is removed so that the dataplane enforcement can be removed from the port. What I\u0027m not sure about is that the current code path is the one that should notify the sriov agent about such change. \n\n\n\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.drivers.mech_agent [None req-accca6e9-8d93-43d0-997a-07362f30e6b2 tempest-QoSBandwidthAndPacketRateTests-1500987257 tempest-QoSBandwidthAndPacketRateTests-1500987257-project] !!! responsible_for_ports_allocation reported\u003d{} allocation\u003d{}\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers [None req-accca6e9-8d93-43d0-997a-07362f30e6b2 tempest-QoSBandwidthAndPacketRateTests-1500987257 tempest-QoSBandwidthAndPacketRateTests-1500987257-project] Mechanism driver openvswitch failed in bind_port: neutron_lib.exceptions.placement.AmbiguousResponsibilityForResourceProvider: Expected one driver to be responsible for resource provider , but got many: openvswitch,sriovnicswitch\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers Traceback (most recent call last):\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers   File \"/opt/stack/neutron/neutron/plugins/ml2/managers.py\", line 922, in _bind_port_level\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers     drivers\u003d[self._infer_driver_from_allocation(\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers   File \"/opt/stack/neutron/neutron/plugins/ml2/managers.py\", line 977, in _infer_driver_from_allocation\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers     raise place_exc.AmbiguousResponsibilityForResourceProvider(\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers neutron_lib.exceptions.placement.AmbiguousResponsibilityForResourceProvider: Expected one driver to be responsible for resource provider , but got many: openvswitch,sriovnicswitch\nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers \nSep 29 11:55:39 master0 neutron-server[2657365]: ERROR neutron.plugins.ml2.managers [None req-accca6e9-8d93-43d0-997a-07362f30e6b2 tempest-QoSBandwidthAndPacketRateTests-1500987257 tempest-QoSBandwidthAndPacketRateTests-1500987257-project] Failed to bind port edf8f960-3bbf-4c9d-9b90-0e08366761ca on host node0 for vnic_type normal using segments [{\u0027id\u0027: \u00277b248642-9158-45e6-b718-49e4ba4c4427\u0027, \u0027network_type\u0027: \u0027vlan\u0027, \u0027physical_network\u0027: \u0027physnet0\u0027, \u0027segmentation_id\u0027: 3000, \u0027network_id\u0027: \u00271219b2f3-379e-47de-8d83-45057403e157\u0027}]\n\nthis can be reproduced with the tempest test in https://review.opendev.org/c/openstack/tempest/+/806257/7/tempest/scenario/test_network_qos_placement.py#905","commit_id":"9ec5e5849ec3fdf9498d67f119557be9aa7be3c2"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"4a3b9c4f65501f112b59b67a8e3c461c7f385f3b","unresolved":false,"context_lines":[{"line_number":257,"context_line":"                # not responsible, must be somebody else"},{"line_number":258,"context_line":"                return False"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        return len(reported) \u003d\u003d len(allocation)"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"class SimpleAgentMechanismDriverBase(AgentMechanismDriverBase,"}],"source_content_type":"text/x-python","patch_set":10,"id":"2aaf1a95_f0c0cb98","line":260,"in_reply_to":"96cfe5e2_53a34161","updated":"2021-10-11 13:38:00.000000000","message":"Done","commit_id":"9ec5e5849ec3fdf9498d67f119557be9aa7be3c2"}]}