)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"ea1c01701dccdf495e08e8c78ff8d82879cbff3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"06c182d2_4e1a0a01","updated":"2026-07-04 09:33:37.000000000","message":"recheck\nseems CI did not ran after PS2","commit_id":"eecda939709e8a97f5a564c58c14747f38cb9ff5"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"4d02f76a2152ae025343eb37ca6b66431efb2808","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2bbdef11_08258fc1","updated":"2026-07-05 14:01:52.000000000","message":"as expected\n\n```\n\nTraceback (most recent call last):\n  File \"/opt/stack/tempest/tempest/api/compute/servers/test_server_actions.py\", line 1028, in test_rebuild_bfv_server_with_snapshot\n    waiters.wait_for_volume_resource_status(\n    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^\n        self.volumes_client, restored_volume_id, \u0027available\u0027)\n        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File \"/opt/stack/tempest/tempest/common/waiters.py\", line 379, in wait_for_volume_resource_status\n    raise exceptions.VolumeResourceBuildErrorException(\n        resource_name\u003dresource_name, resource_id\u003dresource_id)\ntempest.exceptions.VolumeResourceBuildErrorException: volume 0dd7f54d-a675-4090-9147-d2c95bb23f2b failed to build and is in ERROR status\n```\n\nin `nova-ceph-multistore` job\nceph backend - https://zuul.opendev.org/t/openstack/build/96ab5a9e1ef84e7b8282fdaf83d250d3/log/controller/logs/etc/cinder/cinder_conf.txt#29\n\ntempest.conf - https://zuul.opendev.org/t/openstack/build/96ab5a9e1ef84e7b8282fdaf83d250d3/log/controller/logs/tempest_conf.txt#38-41","commit_id":"9536bef2b452ea089f51e4bad0a41489a3759149"},{"author":{"_account_id":38173,"name":"Tanvi Nautiyal","email":"tnautiya@redhat.com","username":"tnautiyal"},"change_message_id":"d04f6cba95f39dc80f817837e33e924fd788e15f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a71c63a1_22fa301e","updated":"2026-07-13 09:25:59.000000000","message":"The test LGTM","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","display_name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc","status":"Senior Software Quality Engineer @ Red Hat"},"change_message_id":"c662b51d9dbd1728e78967024c2f4711c82e0b84","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"226eceba_5b36caf2","updated":"2026-07-24 12:48:08.000000000","message":"LGTM and gates are passing","commit_id":"564d1a686a3b74147f6aa175767d669cbe2698d8"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"ee3c04d08861e19a3188720e320e97369a6e1b5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d4c74306_15afd144","updated":"2026-07-28 07:51:20.000000000","message":"recheck\ntempest-slow-py3 failed again - but unrelated","commit_id":"564d1a686a3b74147f6aa175767d669cbe2698d8"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"3b64e14dfa66d370362e6bdd564af5dc25a9dd02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c9080f70_bbeb31b1","updated":"2026-07-28 04:46:48.000000000","message":"recheck \nfailed test is unrealted\n`tempest.scenario.test_instances_with_cinder_volumes.TestInstancesWithCinderVolumes.test_instances_with_cinder_volumes_on_all_compute_nodes`\n\ntempest-slow-py3 one failed more then few times yesterday - but not really blocked, I think !!!\n\nhttps://zuul.opendev.org/t/openstack/builds?job_name\u003dtempest-slow-py3\u0026pipeline\u003dcheck\u0026skip\u003d0","commit_id":"564d1a686a3b74147f6aa175767d669cbe2698d8"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"7c61c07a68e54477dabea36f7e613f755e612af1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"75253557_b2dc143f","updated":"2026-07-27 06:15:12.000000000","message":"recheck \nthere is an update on fix","commit_id":"564d1a686a3b74147f6aa175767d669cbe2698d8"}],"tempest/api/compute/servers/test_server_actions.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"61e032a91c358c9e29eac14638f6dfc6079f9989","unresolved":true,"context_lines":[{"line_number":1023,"context_line":"        # Verify snapshot still exists after rebuild"},{"line_number":1024,"context_line":"        snap_after \u003d self.snapshots_client.show_snapshot("},{"line_number":1025,"context_line":"            snapshot_id)[\u0027snapshot\u0027]"},{"line_number":1026,"context_line":"        self.assertEqual(\u0027available\u0027, snap_after[\u0027status\u0027])"},{"line_number":1027,"context_line":"        self.assertEqual(volume_id, snap_after[\u0027volume_id\u0027])"},{"line_number":1028,"context_line":""},{"line_number":1029,"context_line":"        # Verify snapshot is usable by creating volume from it"}],"source_content_type":"text/x-python","patch_set":1,"id":"e3125938_14349369","line":1026,"updated":"2026-07-03 10:53:13.000000000","message":"so this should fail right now, but with only RBD.\nthen with depends-on it should pass","commit_id":"4117103533257a9ee7cf328db10203d1a56e248d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a2f953a0111312bfea0077f598874983121bc4e9","unresolved":false,"context_lines":[{"line_number":1023,"context_line":"        # Verify snapshot still exists after rebuild"},{"line_number":1024,"context_line":"        snap_after \u003d self.snapshots_client.show_snapshot("},{"line_number":1025,"context_line":"            snapshot_id)[\u0027snapshot\u0027]"},{"line_number":1026,"context_line":"        self.assertEqual(\u0027available\u0027, snap_after[\u0027status\u0027])"},{"line_number":1027,"context_line":"        self.assertEqual(volume_id, snap_after[\u0027volume_id\u0027])"},{"line_number":1028,"context_line":""},{"line_number":1029,"context_line":"        # Verify snapshot is usable by creating volume from it"}],"source_content_type":"text/x-python","patch_set":1,"id":"aca59ab1_be68ef2a","line":1026,"in_reply_to":"49d03cca_cb72ace3","updated":"2026-07-13 13:40:42.000000000","message":"passed with  depends-on\nhttps://c4eada0370586e541fb2-090e0b161d681e3915fde898eedb8514.ssl.cf2.rackcdn.com/openstack/fa3fa5118c4549f2a8290ede076388cb/testr_results.html","commit_id":"4117103533257a9ee7cf328db10203d1a56e248d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"50fe61d58ab47928d6c82caf69cf65a5ee2066e1","unresolved":true,"context_lines":[{"line_number":1023,"context_line":"        # Verify snapshot still exists after rebuild"},{"line_number":1024,"context_line":"        snap_after \u003d self.snapshots_client.show_snapshot("},{"line_number":1025,"context_line":"            snapshot_id)[\u0027snapshot\u0027]"},{"line_number":1026,"context_line":"        self.assertEqual(\u0027available\u0027, snap_after[\u0027status\u0027])"},{"line_number":1027,"context_line":"        self.assertEqual(volume_id, snap_after[\u0027volume_id\u0027])"},{"line_number":1028,"context_line":""},{"line_number":1029,"context_line":"        # Verify snapshot is usable by creating volume from it"}],"source_content_type":"text/x-python","patch_set":1,"id":"49d03cca_cb72ace3","line":1026,"in_reply_to":"e3125938_14349369","updated":"2026-07-13 06:39:59.000000000","message":"`nova-ceph-multistore` job before depends-on\nhttps://70a366f81cf63ddc4b03-580e84ccb0b17d3ff354ab4c1b56697e.ssl.cf1.rackcdn.com/openstack/86f8e2c42a7b4fb7a9a41810cef2d525/testr_results.html","commit_id":"4117103533257a9ee7cf328db10203d1a56e248d"},{"author":{"_account_id":38173,"name":"Tanvi Nautiyal","email":"tnautiya@redhat.com","username":"tnautiyal"},"change_message_id":"bdb2a99e0e09fd2e28e872c92e3755d6fead763b","unresolved":true,"context_lines":[{"line_number":984,"context_line":""},{"line_number":985,"context_line":"    @decorators.idempotent_id(\u00277f8a9b0c-1d2e-3f4a-5b6c-7d8e9f0a1b2c\u0027)"},{"line_number":986,"context_line":"    def test_rebuild_bfv_server_with_snapshot(self):"},{"line_number":987,"context_line":"        \"\"\"Test rebuilding a BFV server with reimage_boot_volume flag\"\"\""},{"line_number":988,"context_line":"        server \u003d self.reader_servers_client.show_server("},{"line_number":989,"context_line":"            self.server_id)[\u0027server\u0027]"},{"line_number":990,"context_line":"        volume_id \u003d server[\u0027os-extended-volumes:volumes_attached\u0027][0][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"4717016c_9e2463a5","line":987,"updated":"2026-07-08 09:59:32.000000000","message":"The docstring here might be little bit misleading as there is no reimage_boot_volume flag in the below code","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"39a754a837ec46bc322c0dde9d090063650d5c47","unresolved":false,"context_lines":[{"line_number":984,"context_line":""},{"line_number":985,"context_line":"    @decorators.idempotent_id(\u00277f8a9b0c-1d2e-3f4a-5b6c-7d8e9f0a1b2c\u0027)"},{"line_number":986,"context_line":"    def test_rebuild_bfv_server_with_snapshot(self):"},{"line_number":987,"context_line":"        \"\"\"Test rebuilding a BFV server with reimage_boot_volume flag\"\"\""},{"line_number":988,"context_line":"        server \u003d self.reader_servers_client.show_server("},{"line_number":989,"context_line":"            self.server_id)[\u0027server\u0027]"},{"line_number":990,"context_line":"        volume_id \u003d server[\u0027os-extended-volumes:volumes_attached\u0027][0][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"4d8f1fb0_1f60bf1e","line":987,"in_reply_to":"24c80bdb_857e1984","updated":"2026-07-13 01:18:48.000000000","message":"Done","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"eaf1c97678bcad14a14e6bb8061f99866d4b6f25","unresolved":true,"context_lines":[{"line_number":984,"context_line":""},{"line_number":985,"context_line":"    @decorators.idempotent_id(\u00277f8a9b0c-1d2e-3f4a-5b6c-7d8e9f0a1b2c\u0027)"},{"line_number":986,"context_line":"    def test_rebuild_bfv_server_with_snapshot(self):"},{"line_number":987,"context_line":"        \"\"\"Test rebuilding a BFV server with reimage_boot_volume flag\"\"\""},{"line_number":988,"context_line":"        server \u003d self.reader_servers_client.show_server("},{"line_number":989,"context_line":"            self.server_id)[\u0027server\u0027]"},{"line_number":990,"context_line":"        volume_id \u003d server[\u0027os-extended-volumes:volumes_attached\u0027][0][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"24c80bdb_857e1984","line":987,"in_reply_to":"4717016c_9e2463a5","updated":"2026-07-08 10:04:01.000000000","message":"yeah that\u0027s true, the flag is part of client only and not API.\nI realized this later on CI test. will update tis in next PS.","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":38173,"name":"Tanvi Nautiyal","email":"tnautiya@redhat.com","username":"tnautiyal"},"change_message_id":"95b162275d5be060155af6a470d8b4671b7e880c","unresolved":true,"context_lines":[{"line_number":1026,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1027,"context_line":"        self.addCleanup(self.delete_volume, restored_volume_id)"},{"line_number":1028,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1029,"context_line":"            self.volumes_client, restored_volume_id, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"40dce723_bc21768d","line":1029,"updated":"2026-07-08 11:27:19.000000000","message":"After reviewing the entire test, I found that this test case is missing explicit\nassertions unlike other tests in the file, and instead relies on implicit checks\n\nProblem: The test could pass even if:\n- The snapshot was deleted during rebuild (the core bug being tested)\n- The boot volume was detached/replaced\n- Wrong snapshot/volume IDs were used\n\nRecommended Additions:\n\n#1. Verify snapshot still exists after rebuild\nsnapshot \u003d self.snapshots_client.show_snapshot(snapshot_id)[\u0027snapshot\u0027]\nself.assertEqual(\u0027available\u0027, snapshot[\u0027status\u0027])\n\n#2. Verify that volume ID remains same before and after rebuild\nrebuilt_server \u003d self.servers_client.show_server(server[\u0027id\u0027])[\u0027server\u0027]\nvolume_after_rebuild \u003d rebuilt_server[\u0027os-extended-volumes:volumes_attached\u0027][0][\u0027id\u0027]\nself.assertEqual(volume_id, volume_after_rebuild)","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"39a754a837ec46bc322c0dde9d090063650d5c47","unresolved":true,"context_lines":[{"line_number":1026,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1027,"context_line":"        self.addCleanup(self.delete_volume, restored_volume_id)"},{"line_number":1028,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1029,"context_line":"            self.volumes_client, restored_volume_id, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9fef4429_93484169","line":1029,"in_reply_to":"40dce723_bc21768d","updated":"2026-07-13 01:18:48.000000000","message":"\u003e boot volume was detached/replaced\n\nnice catch, added assertion for vol id. \n\n\u003e Problem: The test could pass even if:\n- The snapshot was deleted during rebuild (the core bug being tested)\n\nfor snapshot check, I don\u0027t think snapshot show really help much, as we are able to create volume from snapshot after rebuild. if snapshot was deleted during rebuild , create_vol will fail with 404, so the test does catch that.","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":38173,"name":"Tanvi Nautiyal","email":"tnautiya@redhat.com","username":"tnautiyal"},"change_message_id":"d04f6cba95f39dc80f817837e33e924fd788e15f","unresolved":false,"context_lines":[{"line_number":1026,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1027,"context_line":"        self.addCleanup(self.delete_volume, restored_volume_id)"},{"line_number":1028,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1029,"context_line":"            self.volumes_client, restored_volume_id, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5e3ff454_839a83ba","line":1029,"in_reply_to":"9fef4429_93484169","updated":"2026-07-13 09:25:59.000000000","message":"Acknowledged","commit_id":"fe48665be326882eabfb0705a9be139ed23ba25c"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c49742596e34ab1728575305ff72c8ba021a25b9","unresolved":true,"context_lines":[{"line_number":995,"context_line":"                                       server[\u0027id\u0027], \u0027SHUTOFF\u0027)"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"        # Create volume snapshot"},{"line_number":998,"context_line":"        snapshot_id \u003d self.snapshots_client.create_snapshot("},{"line_number":999,"context_line":"            volume_id\u003dvolume_id,"},{"line_number":1000,"context_line":"            force\u003dTrue)[\u0027snapshot\u0027][\u0027id\u0027]"},{"line_number":1001,"context_line":"        self.addCleanup(self.snapshots_client.wait_for_resource_deletion,"},{"line_number":1002,"context_line":"                        snapshot_id)"},{"line_number":1003,"context_line":"        self.addCleanup(self.snapshots_client.delete_snapshot, snapshot_id)"},{"line_number":1004,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1005,"context_line":"            self.snapshots_client, snapshot_id, \u0027available\u0027)"},{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        # Create volume from snapshot to verify it works before rebuild"},{"line_number":1008,"context_line":"        test_volume_id \u003d self.volumes_client.create_volume("}],"source_content_type":"text/x-python","patch_set":7,"id":"117acee0_0df32c11","line":1005,"range":{"start_line":998,"start_character":0,"end_line":1005,"end_character":60},"updated":"2026-07-13 10:11:24.000000000","message":"you can use helper method `create_volume_snapshot`\nhttps://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L661","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a2f953a0111312bfea0077f598874983121bc4e9","unresolved":true,"context_lines":[{"line_number":995,"context_line":"                                       server[\u0027id\u0027], \u0027SHUTOFF\u0027)"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"        # Create volume snapshot"},{"line_number":998,"context_line":"        snapshot_id \u003d self.snapshots_client.create_snapshot("},{"line_number":999,"context_line":"            volume_id\u003dvolume_id,"},{"line_number":1000,"context_line":"            force\u003dTrue)[\u0027snapshot\u0027][\u0027id\u0027]"},{"line_number":1001,"context_line":"        self.addCleanup(self.snapshots_client.wait_for_resource_deletion,"},{"line_number":1002,"context_line":"                        snapshot_id)"},{"line_number":1003,"context_line":"        self.addCleanup(self.snapshots_client.delete_snapshot, snapshot_id)"},{"line_number":1004,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1005,"context_line":"            self.snapshots_client, snapshot_id, \u0027available\u0027)"},{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        # Create volume from snapshot to verify it works before rebuild"},{"line_number":1008,"context_line":"        test_volume_id \u003d self.volumes_client.create_volume("}],"source_content_type":"text/x-python","patch_set":7,"id":"2cb7386b_298ba422","line":1005,"range":{"start_line":998,"start_character":0,"end_line":1005,"end_character":60},"in_reply_to":"117acee0_0df32c11","updated":"2026-07-13 13:40:42.000000000","message":"done, so now switched to `create_volume_snapshot` helper, thanks for noticing.\n\ninitially I had used direct API calls because I thought rebuild needed a raw reimage_boot_volume parameter, but that\u0027s just a CLI flag and this is automatically handled in mv 2.93.\nso I kept the rest of the test consistent with direct calls, the helpers are fine fot the rest too.","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"97ab36fb8baa1183de26ac36bd0a75a22a2f77b3","unresolved":false,"context_lines":[{"line_number":995,"context_line":"                                       server[\u0027id\u0027], \u0027SHUTOFF\u0027)"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"        # Create volume snapshot"},{"line_number":998,"context_line":"        snapshot_id \u003d self.snapshots_client.create_snapshot("},{"line_number":999,"context_line":"            volume_id\u003dvolume_id,"},{"line_number":1000,"context_line":"            force\u003dTrue)[\u0027snapshot\u0027][\u0027id\u0027]"},{"line_number":1001,"context_line":"        self.addCleanup(self.snapshots_client.wait_for_resource_deletion,"},{"line_number":1002,"context_line":"                        snapshot_id)"},{"line_number":1003,"context_line":"        self.addCleanup(self.snapshots_client.delete_snapshot, snapshot_id)"},{"line_number":1004,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1005,"context_line":"            self.snapshots_client, snapshot_id, \u0027available\u0027)"},{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        # Create volume from snapshot to verify it works before rebuild"},{"line_number":1008,"context_line":"        test_volume_id \u003d self.volumes_client.create_volume("}],"source_content_type":"text/x-python","patch_set":7,"id":"b0305edc_720fdff5","line":1005,"range":{"start_line":998,"start_character":0,"end_line":1005,"end_character":60},"in_reply_to":"2cb7386b_298ba422","updated":"2026-07-23 09:50:29.000000000","message":"Done","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c49742596e34ab1728575305ff72c8ba021a25b9","unresolved":true,"context_lines":[{"line_number":1005,"context_line":"            self.snapshots_client, snapshot_id, \u0027available\u0027)"},{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        # Create volume from snapshot to verify it works before rebuild"},{"line_number":1008,"context_line":"        test_volume_id \u003d self.volumes_client.create_volume("},{"line_number":1009,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1010,"context_line":"        self.addCleanup(self.delete_volume, test_volume_id)"},{"line_number":1011,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1012,"context_line":"            self.volumes_client, test_volume_id, \u0027available\u0027)"},{"line_number":1013,"context_line":""},{"line_number":1014,"context_line":"        # Start server before rebuild"},{"line_number":1015,"context_line":"        self.servers_client.start_server(server[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":7,"id":"77ff3b94_5678bdfd","line":1012,"range":{"start_line":1008,"start_character":0,"end_line":1012,"end_character":61},"updated":"2026-07-13 10:11:24.000000000","message":"same here, you can use `create_volume`:\n\nhttps://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L516\n\nthis is a classmethod, so if the volume needs to be deleted at the end of the test, you only need to add `self.addCleanup`","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"97ab36fb8baa1183de26ac36bd0a75a22a2f77b3","unresolved":false,"context_lines":[{"line_number":1005,"context_line":"            self.snapshots_client, snapshot_id, \u0027available\u0027)"},{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        # Create volume from snapshot to verify it works before rebuild"},{"line_number":1008,"context_line":"        test_volume_id \u003d self.volumes_client.create_volume("},{"line_number":1009,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1010,"context_line":"        self.addCleanup(self.delete_volume, test_volume_id)"},{"line_number":1011,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1012,"context_line":"            self.volumes_client, test_volume_id, \u0027available\u0027)"},{"line_number":1013,"context_line":""},{"line_number":1014,"context_line":"        # Start server before rebuild"},{"line_number":1015,"context_line":"        self.servers_client.start_server(server[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":7,"id":"4aa381ac_6958f14e","line":1012,"range":{"start_line":1008,"start_character":0,"end_line":1012,"end_character":61},"in_reply_to":"77ff3b94_5678bdfd","updated":"2026-07-23 09:50:29.000000000","message":"Done","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c49742596e34ab1728575305ff72c8ba021a25b9","unresolved":true,"context_lines":[{"line_number":1031,"context_line":"        self.assertEqual(volume_id, attached_vol_id)"},{"line_number":1032,"context_line":""},{"line_number":1033,"context_line":"        # Verify snapshot still exists and is usable after rebuild"},{"line_number":1034,"context_line":"        restored_volume_id \u003d self.volumes_client.create_volume("},{"line_number":1035,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1036,"context_line":"        self.addCleanup(self.delete_volume, restored_volume_id)"},{"line_number":1037,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1038,"context_line":"            self.volumes_client, restored_volume_id, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"0e2ffd54_bff14afb","line":1038,"range":{"start_line":1034,"start_character":0,"end_line":1038,"end_character":65},"updated":"2026-07-13 10:11:24.000000000","message":"Same","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"97ab36fb8baa1183de26ac36bd0a75a22a2f77b3","unresolved":false,"context_lines":[{"line_number":1031,"context_line":"        self.assertEqual(volume_id, attached_vol_id)"},{"line_number":1032,"context_line":""},{"line_number":1033,"context_line":"        # Verify snapshot still exists and is usable after rebuild"},{"line_number":1034,"context_line":"        restored_volume_id \u003d self.volumes_client.create_volume("},{"line_number":1035,"context_line":"            snapshot_id\u003dsnapshot_id)[\u0027volume\u0027][\u0027id\u0027]"},{"line_number":1036,"context_line":"        self.addCleanup(self.delete_volume, restored_volume_id)"},{"line_number":1037,"context_line":"        waiters.wait_for_volume_resource_status("},{"line_number":1038,"context_line":"            self.volumes_client, restored_volume_id, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"ba0e5ec2_65dd3bc2","line":1038,"range":{"start_line":1034,"start_character":0,"end_line":1038,"end_character":65},"in_reply_to":"0e2ffd54_bff14afb","updated":"2026-07-23 09:50:29.000000000","message":"Done","commit_id":"cba61522e9bca747b48d6ac1e79281117327d405"}]}
