)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ef6696140c115f5955ebbe14f05a03ba0c524c91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bac6c18d_549bc52e","updated":"2026-02-04 19:08:48.000000000","message":"These modified tests are skipped in tempest gate jobs, can you let me know which job these are enabled and run testing change with depends-on with this change? so that we can see the result.","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"e4b69eb467097b2eccc4edb3ca9f98ab8e91ddc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31ec9187_5f539154","updated":"2026-02-12 07:37:59.000000000","message":"recheck docs gate timed out","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"f816705cd9bb0ce228b714ee726b6da8ab4f6c0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a6261493_8d904fdd","updated":"2026-02-12 09:35:53.000000000","message":"recheck last one didn\u0027t trigger anything https://zuul.opendev.org/t/openstack/status/change/974752,2","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"da7ceeeda1b8f9d04e7bbdefc04e9a42d14b032a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"355a9731_8e3276a6","updated":"2026-02-12 03:53:49.000000000","message":"tempest job is green now https://review.opendev.org/c/openstack/tempest/+/976501","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f1495f8c89ed59b6d817a56fd6d536c6c8803a08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"be6b05a2_0f1acfa9","in_reply_to":"0d7f8945_205f41f4","updated":"2026-02-09 21:27:05.000000000","message":"perfect thanks. I will +w it once gate if unblocked otehrwise lgtm.","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f7e69c4623d629fc28031a4066a30ba0e13b9e00","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cbaf9621_a5e2f241","in_reply_to":"155e5e97_bfa9622c","updated":"2026-02-09 20:55:11.000000000","message":"well, it is not the issue that these test exist in tempest or neutron-tempest-plugin, it is matter of where we can run these tests in upstream.\n\n@mblue@redhat.com Anyways, if currently we do not have any upstream job configured for this, it is ok but can you run and provide the downstream testing result and we can merge it.","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5998728ad3f27cb354e0e439624677a944245e3d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"155e5e97_bfa9622c","in_reply_to":"76bdca3d_19d68c32","updated":"2026-02-09 14:16:28.000000000","message":"Right now I\u0027m not sure where these tests are running. That\u0027s something to be investigated. In any case, we can also move them to neutron-tempest-plugin, in order to have more control over them (Neutron team).","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"2352a6e6dba6dd5006fb8a86266dec464c96e1e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d380d2d6_7b9b584e","in_reply_to":"bac6c18d_549bc52e","updated":"2026-02-05 09:19:56.000000000","message":"Well we have some downstream CI jobs running with pre defined external provider network that is vlan, I see some upstream CI definitions that are with such flat network named `public` ahead of time.\nWhat do you think?\nShould we add such CI run with vlan equivalent for `neutron-tempest-plugin` or `tempest`?","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"e4b69eb467097b2eccc4edb3ca9f98ab8e91ddc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a40a19c4_901d993d","in_reply_to":"be6b05a2_0f1acfa9","updated":"2026-02-12 07:37:59.000000000","message":"thanks!","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"294896081db6ec68a3bd278690b51449fea33858","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d7f8945_205f41f4","in_reply_to":"cbaf9621_a5e2f241","updated":"2026-02-09 21:18:02.000000000","message":"Tests pass too for CI jobs with pre made public network.\n\nWhen configuring tempest conf test class related values (`qos_placement_physnet`, `provider_net_base_segmentation_id`), ovn-cms-options to have `resource_provider_*` values on computes, and adding `placement` to `service_plugins` in neutron configuration, the tests pass as seen in the following output:\n```\nsh-5.1$ stestr run tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_empty_update [22.057948s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_migrate_with_qos_min_bw_allocation [48.537687s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_basic [25.763259s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_update_policy [23.964148s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_update_policy_direction_change [18.105985s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_update_policy_from_zero [16.614603s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_update_policy_to_zero [18.834825s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_qos_min_bw_allocation_update_with_multiple_ports [25.025085s] ... ok\n{0} tempest.scenario.test_network_qos_placement.MinBwAllocationPlacementTest.test_resize_with_qos_min_bw_allocation [47.413598s] ... ok\n\n\u003d\u003d\u003d\u003d\u003d\u003d\nTotals\n\u003d\u003d\u003d\u003d\u003d\u003d\nRan: 9 tests in 246.3594 sec.\n - Passed: 9\n - Skipped: 0\n - Expected Fail: 0\n - Unexpected Success: 0\n - Failed: 0\nSum of execute time for each test: 246.3171 sec.\n```","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8fba8d9ef83fcd69836ae3638d8041e744b59641","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76bdca3d_19d68c32","in_reply_to":"d380d2d6_7b9b584e","updated":"2026-02-06 01:57:57.000000000","message":"yeah, it will be good to have that config in upstream or @ralonsoh@redhat.com or @katonalala@gmail.com knows if there is any existing jobs?","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"}],"tempest/scenario/test_network_qos_placement.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"94ede505cd8bdeccd32d41c326373d2b35464923","unresolved":true,"context_lines":[{"line_number":161,"context_line":"        if vlan_ext_nets:"},{"line_number":162,"context_line":"            self.prov_network \u003d vlan_ext_nets[0]"},{"line_number":163,"context_line":"            if not self.prov_network[\u0027shared\u0027]:"},{"line_number":164,"context_line":"                self.prov_network \u003d self.networks_client.update_network("},{"line_number":165,"context_line":"                    self.prov_network[\u0027id\u0027], shared\u003dTrue)[\u0027network\u0027]"},{"line_number":166,"context_line":"                self.addClassResourceCleanup("},{"line_number":167,"context_line":"                    self.networks_client.update_network,"}],"source_content_type":"text/x-python","patch_set":2,"id":"56310ae8_470447cd","line":164,"range":{"start_line":164,"start_character":41,"end_line":164,"end_character":56},"updated":"2026-01-27 13:23:07.000000000","message":"+1, to have prepared network, which is accessible for tempest","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"05828c2dc35660d957561f1894214274956a3c9e","unresolved":true,"context_lines":[{"line_number":161,"context_line":"        if vlan_ext_nets:"},{"line_number":162,"context_line":"            self.prov_network \u003d vlan_ext_nets[0]"},{"line_number":163,"context_line":"            if not self.prov_network[\u0027shared\u0027]:"},{"line_number":164,"context_line":"                self.prov_network \u003d self.networks_client.update_network("},{"line_number":165,"context_line":"                    self.prov_network[\u0027id\u0027], shared\u003dTrue)[\u0027network\u0027]"},{"line_number":166,"context_line":"                self.addClassResourceCleanup("},{"line_number":167,"context_line":"                    self.networks_client.update_network,"}],"source_content_type":"text/x-python","patch_set":2,"id":"89c1bf27_864f8ad6","line":164,"range":{"start_line":164,"start_character":41,"end_line":164,"end_character":56},"in_reply_to":"56310ae8_470447cd","updated":"2026-01-27 16:12:59.000000000","message":"Thank you for the review.\nAlso by the way, the reason I don\u0027t assume `shared\u003dTrue` is set prior to test run within CI is that we had some issues running together with vlan transparency tests in the past.\nTherefore changing it only for the test class run time (in case not set already).","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ba5cd8db3dcf287539416dbd86be1257652ce451","unresolved":true,"context_lines":[{"line_number":166,"context_line":"                self.addClassResourceCleanup("},{"line_number":167,"context_line":"                    self.networks_client.update_network,"},{"line_number":168,"context_line":"                    self.prov_network[\u0027id\u0027],"},{"line_number":169,"context_line":"                    shared\u003dFalse)"},{"line_number":170,"context_line":"        else:"},{"line_number":171,"context_line":"            physnet_name \u003d CONF.network_feature_enabled.qos_placement_physnet"},{"line_number":172,"context_line":"            base_segm \u003d \\"}],"source_content_type":"text/x-python","patch_set":2,"id":"af5447ff_ffd14c01","line":169,"range":{"start_line":169,"start_character":20,"end_line":169,"end_character":32},"updated":"2026-01-27 13:52:26.000000000","message":"If several tests inheriting from this class are being executed in parallel, the pre-existing networks will change the `shared` property in the middle of the test.","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"dcedc2b352e52b5506a79eb0518edd047b82ba9c","unresolved":false,"context_lines":[{"line_number":166,"context_line":"                self.addClassResourceCleanup("},{"line_number":167,"context_line":"                    self.networks_client.update_network,"},{"line_number":168,"context_line":"                    self.prov_network[\u0027id\u0027],"},{"line_number":169,"context_line":"                    shared\u003dFalse)"},{"line_number":170,"context_line":"        else:"},{"line_number":171,"context_line":"            physnet_name \u003d CONF.network_feature_enabled.qos_placement_physnet"},{"line_number":172,"context_line":"            base_segm \u003d \\"}],"source_content_type":"text/x-python","patch_set":2,"id":"33f4d17c_b8500b7f","line":169,"range":{"start_line":169,"start_character":20,"end_line":169,"end_character":32},"in_reply_to":"4e4bde46_6cc3adcf","updated":"2026-02-09 14:17:09.000000000","message":"Done","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"05828c2dc35660d957561f1894214274956a3c9e","unresolved":true,"context_lines":[{"line_number":166,"context_line":"                self.addClassResourceCleanup("},{"line_number":167,"context_line":"                    self.networks_client.update_network,"},{"line_number":168,"context_line":"                    self.prov_network[\u0027id\u0027],"},{"line_number":169,"context_line":"                    shared\u003dFalse)"},{"line_number":170,"context_line":"        else:"},{"line_number":171,"context_line":"            physnet_name \u003d CONF.network_feature_enabled.qos_placement_physnet"},{"line_number":172,"context_line":"            base_segm \u003d \\"}],"source_content_type":"text/x-python","patch_set":2,"id":"4e4bde46_6cc3adcf","line":169,"range":{"start_line":169,"start_character":20,"end_line":169,"end_character":32},"in_reply_to":"af5447ff_ffd14c01","updated":"2026-01-27 16:12:59.000000000","message":"Thank you for the review.\nThe `addClassResourceCleanup` should execute when all test methods finished executing, while tempest class teardown, so should be set `shared\u003dTrue` by whichever test executed first (in case race condition happens that few somehow try in parallel, tested that with update share\u003dTrue through CLI seems to not cause issues so far), then after all executed in class, the `shared\u003dFalse` will be set.","commit_id":"357169c4fb5abc8daa3df72f27c019c4e2eabea2"}]}