)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"17b0b40abe2eb386f22ae09559c943781d97ee13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3aa1df0f_4189b2f3","updated":"2022-06-17 17:37:27.000000000","message":"this worked fine, let\u0027s remove the dependency on revert and merge this fix directly. revert is failed in check pipeline so that will be delayed to merge","commit_id":"f11bd630aa2899a708e5ae48f58a90c2ea7f6244"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c37d572ca75ffca78e14d025b8b62030725e5cd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b73c1cbe_71f97c67","updated":"2022-06-17 17:51:01.000000000","message":"thanks, re-applying +W on this gate blocker. basically no logical change from previous PS which was on based on revert vs this PS which is based on master.","commit_id":"c852639184f47976562c9ec2ea152719f57e4f41"}],"tempest/api/compute/admin/test_live_migration.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"03eb2a6cbefa48512185db9ea11ccbd46ea22583","unresolved":true,"context_lines":[{"line_number":182,"context_line":"        attach volume. This differs from test_volume_backed_live_migration"},{"line_number":183,"context_line":"        above that tests live-migration with only an attached volume."},{"line_number":184,"context_line":"        \"\"\""},{"line_number":185,"context_line":"        validation_resources \u003d self.get_test_validation_resources("},{"line_number":186,"context_line":"            self.os_primary)"},{"line_number":187,"context_line":"        server \u003d self.create_test_server("},{"line_number":188,"context_line":"            validatable\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"684a6a4f_dd23bfc9","line":185,"range":{"start_line":185,"start_character":36,"end_line":185,"end_character":65},"updated":"2022-06-17 14:46:36.000000000","message":"let\u0027s try to use class level validation resource here as server cleanup is done at class level.\n\nself.get_class_validation_resources(","commit_id":"7a05fe60109722239d3b1ab1df6bfdd3558f31db"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"9be898012498f9bbfe7bc1eb26fdb47f6cbfcce5","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        attach volume. This differs from test_volume_backed_live_migration"},{"line_number":183,"context_line":"        above that tests live-migration with only an attached volume."},{"line_number":184,"context_line":"        \"\"\""},{"line_number":185,"context_line":"        validation_resources \u003d self.get_test_validation_resources("},{"line_number":186,"context_line":"            self.os_primary)"},{"line_number":187,"context_line":"        server \u003d self.create_test_server("},{"line_number":188,"context_line":"            validatable\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"77582968_1a12264d","line":185,"range":{"start_line":185,"start_character":36,"end_line":185,"end_character":65},"in_reply_to":"684a6a4f_dd23bfc9","updated":"2022-06-17 14:54:54.000000000","message":"Done","commit_id":"7a05fe60109722239d3b1ab1df6bfdd3558f31db"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"cdcfed183fa945bb17cba4cbc7381d6823059f80","unresolved":true,"context_lines":[{"line_number":184,"context_line":"        \"\"\""},{"line_number":185,"context_line":"        validation_resources \u003d self.get_test_validation_resources("},{"line_number":186,"context_line":"            self.os_primary)"},{"line_number":187,"context_line":"        server \u003d self.create_test_server("},{"line_number":188,"context_line":"            validatable\u003dTrue,"},{"line_number":189,"context_line":"            validation_resources\u003dvalidation_resources,"},{"line_number":190,"context_line":"            wait_until\u003d\"SSHABLE\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"5cee05bd_9904afed","line":187,"updated":"2022-06-17 12:05:01.000000000","message":"For some reason the cleanup of the validation resource happens before the server and the port is cleaned up causing the validation resource cleanup to fail with:\n\nTraceback (most recent call last):\n  File \"/opt/stack/new/tempest/.tox/tempest/lib/python3.8/site-packages/fixtures/fixture.py\", line 125, in cleanUp\n    return self._cleanups(raise_errors\u003draise_first)\n  File \"/opt/stack/new/tempest/.tox/tempest/lib/python3.8/site-packages/fixtures/callmany.py\", line 89, in __call__\n    reraise(error[0], error[1], error[2])\n  File \"/opt/stack/new/tempest/.tox/tempest/lib/python3.8/site-packages/testtools/compat.py\", line 27, in reraise\n    raise exc_obj.with_traceback(exc_tb)\n  File \"/opt/stack/new/tempest/.tox/tempest/lib/python3.8/site-packages/fixtures/callmany.py\", line 83, in __call__\n    cleanup(*args, **kwargs)\n  File \"/opt/stack/new/tempest/tempest/lib/common/validation_resources.py\", line 346, in clear_validation_resources\n    raise has_exception\n  File \"/opt/stack/new/tempest/tempest/lib/common/validation_resources.py\", line 315, in clear_validation_resources\n    security_group_client.delete_security_group(sec_id)\n  File \"/opt/stack/new/tempest/tempest/lib/services/network/security_groups_client.py\", line 59, in delete_security_group\n    return self.delete_resource(uri)\n  File \"/opt/stack/new/tempest/tempest/lib/services/network/base.py\", line 42, in delete_resource\n    resp, body \u003d self.delete(req_uri)\n  File \"/opt/stack/new/tempest/tempest/lib/common/rest_client.py\", line 330, in delete\n    return self.request(\u0027DELETE\u0027, url, extra_headers, headers, body)\n  File \"/opt/stack/new/tempest/tempest/lib/common/rest_client.py\", line 711, in request\n    self._error_checker(resp, resp_body)\n  File \"/opt/stack/new/tempest/tempest/lib/common/rest_client.py\", line 832, in _error_checker\n    raise exceptions.Conflict(resp_body, resp\u003dresp)\ntempest.lib.exceptions.Conflict: Conflict with state of target resource\nDetails: {\u0027type\u0027: \u0027SecurityGroupInUse\u0027, \u0027message\u0027: \u0027Security Group 76c857c7-59b8-4aa2-9ef0-fab8cab24389 in use.\u0027, \u0027detail\u0027: \u0027\u0027}\n\nSee for example https://zuul.opendev.org/t/openstack/build/c7b5440bdfc34eff944ddcd1cc923b96/logs","commit_id":"7a05fe60109722239d3b1ab1df6bfdd3558f31db"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"9be898012498f9bbfe7bc1eb26fdb47f6cbfcce5","unresolved":false,"context_lines":[{"line_number":184,"context_line":"        \"\"\""},{"line_number":185,"context_line":"        validation_resources \u003d self.get_test_validation_resources("},{"line_number":186,"context_line":"            self.os_primary)"},{"line_number":187,"context_line":"        server \u003d self.create_test_server("},{"line_number":188,"context_line":"            validatable\u003dTrue,"},{"line_number":189,"context_line":"            validation_resources\u003dvalidation_resources,"},{"line_number":190,"context_line":"            wait_until\u003d\"SSHABLE\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"fc8c37fd_bbbd8cdb","line":187,"in_reply_to":"5cee05bd_9904afed","updated":"2022-06-17 14:54:54.000000000","message":"Done","commit_id":"7a05fe60109722239d3b1ab1df6bfdd3558f31db"}]}