)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"12bc30ed91541b79a529460b3d159984e251650a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8af9be59_fe2a20ab","updated":"2026-10-20 06:52:48.000000000","message":"The fake RBD image state is consistently populated for the supported creation paths and removed on deletion. The fallback in initialize_connection also preserves compatibility with existing functional-test setup. No blocking issues found.","commit_id":"fe5c7190059e7903f2517a815abc146bdcda07b7"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"b92ec637c1ac5c52f3972fc1dafd38cd01286ebd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"da4a3c19_f89281e7","updated":"2026-11-05 13:58:15.000000000","message":"recheck\n\ntempest slow failed unrealted to this test patch\n\n```\n2026-11-05 11:52:01.133718 | controller |     26-11-05 11:51:11,539 103985 ERROR    [tempest.lib.common.utils.linux.remote_client] (TestVolumeMigrateRetypeAttached:test_volume_retype_attached_data_volume) Executing command on 172.24.5.31 failed. Error: Command \u0027set -eu -o pipefail; PATH\u003d$PATH:/sbin:/usr/sbin; sudo mount /dev/vdb /mnt/vdb\u0027, exit status: 255, stderr:\n2026-11-05 11:52:01.133727 | controller | mount: mounting /dev/vdb on /mnt/vdb failed: Device or resource busy\n2026-11-05 11:52:01.133736 | controller |\n2026-11-05 11:52:01.133745 | controller | stdout:\n2026-11-05 11:52:01.133754 | controller |\n2026-11-05 11:52:01.133763 | controller | 2026-11-05 11:51:11,540 103985 WARNING  [tempest.common.utils.linux.remote_client] Mounting /dev/vdb on /mnt/vdb failed on attempt 1 of 3, retrying in 2 seconds\n2026-11-05 11:52:01.133772 | controller | 2026-11-05 11:51:13,540 103985 DEBUG    [tempest.lib.common.utils.linux.remote_client] Remote command: set -eu -o pipefail; PATH\u003d$PATH:/sbin:/usr/sbin; sudo mount /dev/vdb /mnt/vdb\n```","commit_id":"fe31655628608f43e8944b13e5dd3a6964733ec2"}],"cinder/tests/fake_driver.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"982a060c920f531e27511b33824553b6cf7625f6","unresolved":true,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":270,"context_line":"        super(FakeRBDDriver, self).extend_volume(volume, new_size)"},{"line_number":271,"context_line":"        self._rbd_images[volume.id][\u0027size\u0027] \u003d new_size"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    def delete_volume(self, volume):"},{"line_number":274,"context_line":"        super(FakeRBDDriver, self).delete_volume(volume)"}],"source_content_type":"text/x-python","patch_set":2,"id":"4cd7d836_92f36134","line":271,"updated":"2026-10-25 15:13:31.000000000","message":"self._rbd_images[volume.id][\u0027size\u0027] \u003d new_size\n\nThis raises KeyError when volume.id was never inserted. super() is FakeLoggingVolumeDriver.extend_volume (fake_driver.py:200-201), which returns immediately, so the KeyError is from this line. VolumeManager.extend_volume catches Exception and calls extend_volume_completion(..., error\u003dTrue), which sets status error_extending and rolls the quota back (cinder/volume/manager.py:2985-2989 and 3018-3032).\n\ntest_resize_volume does not hit this: _create_volume records the id first. The image-download path does not hit it either: driver.create_volume (flows/manager/create_volume.py:800) records the image before the later extend (create_volume.py:1075).\n\nmanage_existing is not overridden on FakeRBDDriver. It is LVMVolumeDriver.manage_existing (cinder/volume/drivers/lvm.py:639) and does not call _record_image. Extending that volume takes the KeyError path. initialize_connection (line 278) uses .get and does not raise. Please use the same missing-id behavior in both methods.","commit_id":"f1a6799d2773f70739fd7375dd4bd3019cac4ac0"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"72820d73b480018cab4a36818d9eb9ddc18baf1e","unresolved":true,"context_lines":[{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    def extend_volume(self, volume, new_size):"},{"line_number":270,"context_line":"        super(FakeRBDDriver, self).extend_volume(volume, new_size)"},{"line_number":271,"context_line":"        self._rbd_images[volume.id][\u0027size\u0027] \u003d new_size"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    def delete_volume(self, volume):"},{"line_number":274,"context_line":"        super(FakeRBDDriver, self).delete_volume(volume)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8ca8d0c9_0777aa0f","line":271,"in_reply_to":"4cd7d836_92f36134","updated":"2026-11-05 10:19:30.000000000","message":"thanks for noting this - not here, but this will be used later.\n\nadded manage_existing (and manage_existing_get_size, the inherited LVM one\nbreaks on FakeBrickLVM) - now `_record_image` takes an optional size because volume.size is not set yet when the flow calls the driver","commit_id":"f1a6799d2773f70739fd7375dd4bd3019cac4ac0"}]}