)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"97efa4ce1ff476dd8eb903833fe921a78ae54adb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"0b939063_1f5ced5a","updated":"2021-10-26 09:10:01.000000000","message":"recheck deps is not in merge conflict now","commit_id":"25469f8b2fec48cdad12876a667c952a39f81f0f"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"0f92122ff4625339f16fb6e82a8534b64253b0a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"07b98552_6b17e59e","updated":"2022-01-12 08:16:16.000000000","message":"lgtm, thanks","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"bb5035c19b237728bafafbecb7919cbf4169fe8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"87be5dfb_69a332cc","updated":"2021-12-13 08:14:14.000000000","message":"recheck .. will these tests be executed somewhere in gates (e.g. neutron)?","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"bfc0ba54c03333fc4ed945a4996ba68adc748c73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"f9d1ce0d_37772469","updated":"2021-12-10 10:52:59.000000000","message":"recheck just to bump it","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"ab8b6aff818b2bcbf0e3eb12ee07dd02ff2ed1a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"06503671_e65c5e58","updated":"2021-11-29 15:17:43.000000000","message":"recheck lets see if this still works...","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b1034d0ec6c13e2579317f635c4fc838e0783b22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"6e2bf863_397fb3af","updated":"2022-01-12 03:19:59.000000000","message":"thanks lgtm. Leaving +A to Martin in case he has any comment.\n\nTest are passing in nova as well as in neutron job\n- https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_632/811748/8/check/nova-next/632bb75/testr_results.html\n\n- https://614c13582752de139914-134fee3611f99557a1466ec5867d2d52.ssl.cf5.rackcdn.com/811746/5/check/neutron-ovs-tempest-multinode-full/bf06569/testr_results.html","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"62e42e5ca3a35cab5fe93b4ed7acef63cc8e4db2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"2270ec6b_a1f06c5a","in_reply_to":"87be5dfb_69a332cc","updated":"2021-12-13 11:14:29.000000000","message":"Yes, both\n* in nova  https://review.opendev.org/c/openstack/nova/+/811748 and \n* in neutron https://review.opendev.org/c/openstack/neutron/+/811746","commit_id":"f294b0d8dca7d7251d274a44c140bffdf75959ae"}],"tempest/scenario/test_network_qos_placement.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"651a1d02d16b4478253c2089190f2d2c78b53c1e","unresolved":true,"context_lines":[{"line_number":963,"context_line":"        self.interface_client.delete_interface("},{"line_number":964,"context_line":"            server_id\u003dserver[\u0027id\u0027],"},{"line_number":965,"context_line":"            port_id\u003dport2[\u0027id\u0027])"},{"line_number":966,"context_line":"        waiters.wait_for_interface_detach("},{"line_number":967,"context_line":"            self.interface_client, server[\u0027id\u0027], port2[\u0027id\u0027])"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"        self.assert_allocations(server, port2, 0, 0)"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"    @decorators.idempotent_id(\u002736ffdb85-6cc2-4cc9-a426-cad5bac8626b\u0027)"},{"line_number":972,"context_line":"    @testtools.skipUnless("}],"source_content_type":"text/x-python","patch_set":11,"id":"cd6c8950_4cb50f63","line":969,"range":{"start_line":966,"start_character":8,"end_line":969,"end_character":52},"updated":"2021-10-01 15:00:12.000000000","message":"As this run proves[1] the interface removed from the API response before the actual detach finishes so we need to figure out a better waiter to assert that the allocation is removed.\n\n[1]https://zuul.opendev.org/t/openstack/build/a327cbb3287d4e6090baa598d75c4db6/log/compute1/logs/screen-n-cpu.txt#48177-48180","commit_id":"782d7d8e1930a6b19d9273109b6bd0c1a262f4b0"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"ee0066df0024a703ef3e146bac29220ff2e44b6d","unresolved":false,"context_lines":[{"line_number":963,"context_line":"        self.interface_client.delete_interface("},{"line_number":964,"context_line":"            server_id\u003dserver[\u0027id\u0027],"},{"line_number":965,"context_line":"            port_id\u003dport2[\u0027id\u0027])"},{"line_number":966,"context_line":"        waiters.wait_for_interface_detach("},{"line_number":967,"context_line":"            self.interface_client, server[\u0027id\u0027], port2[\u0027id\u0027])"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"        self.assert_allocations(server, port2, 0, 0)"},{"line_number":970,"context_line":""},{"line_number":971,"context_line":"    @decorators.idempotent_id(\u002736ffdb85-6cc2-4cc9-a426-cad5bac8626b\u0027)"},{"line_number":972,"context_line":"    @testtools.skipUnless("}],"source_content_type":"text/x-python","patch_set":11,"id":"c9f10c7c_fcafadba","line":969,"range":{"start_line":966,"start_character":8,"end_line":969,"end_character":52},"in_reply_to":"cd6c8950_4cb50f63","updated":"2021-10-05 09:40:07.000000000","message":"So the nova order of actions:\n1) unplug the device from the VM\n2) unbind the port in neutron\n3) deallocate resources\n\nThe os-interfaces API list plugged interfaces based on the list of ports bound to the instance. So the wait_for_interface_detach could return at 2) but this assert can still see the allocated resources in placement if it run before 3) finishes.\n\nWe need to wait for the os-instance-actions to list that the detach is finished before we can assert. That action is only recorded after 3).\n\n\u003d\u003e Done","commit_id":"782d7d8e1930a6b19d9273109b6bd0c1a262f4b0"}]}