)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5fae2205f542c752e59d8a641ee743a94591f859","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cf4e0a1a_2e8791f8","updated":"2025-04-25 20:35:03.000000000","message":"I just realized that this has been sitting for over a year.  Sorry about that.  Code and test change LGTM.","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"0c9dc266710f0d342f19a2fbffda9755a7c4c593","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bc9cbbfc_bb4b302d","updated":"2025-10-30 17:04:33.000000000","message":"Looks okay to me.","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"330de8086588634b2d30d1547495f4fc13cc9427","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f267a32c_f73dadfe","updated":"2025-10-31 15:50:29.000000000","message":"recheck\n\nceph job failed with \u0027qemu-img: /opt/stack/data/cinder/conversion/tmpl_qvlrox: Unable to get accurate CPU usage\\n\u0027\n\nwhich is a qemu-img bug","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"7b2481dcd4c46f9bc2ef82aa50c495d7cb2bc7d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a80d122a_015d5afe","updated":"2024-03-14 17:59:04.000000000","message":"recheck reno job fix merged now","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"49520b18f85960b5c0d70395fc98e6e6b81beb2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c4461f11_a8e4bdbc","updated":"2024-03-14 15:29:22.000000000","message":"recheck the ceph job was breaking for some reason and someone mentioned it\u0027s fixed now","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"}],"cinder/tests/unit/volume/test_image.py":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"1a1523f475156e216c103facb503d8d163ac6e61","unresolved":true,"context_lines":[{"line_number":518,"context_line":"                                             gigabytes_vol_type_name\u003dvol.size)"},{"line_number":519,"context_line":"        mock_commit.assert_called_once_with(self.context, [\"RESERVATION\"],"},{"line_number":520,"context_line":"                                            project_id\u003dvol.project_id)"},{"line_number":521,"context_line":"        # Validate that correct properties are set in the clone volume"},{"line_number":522,"context_line":"        self.assertEqual(vol.volume_type_id, result.volume_type_id)"},{"line_number":523,"context_line":"        self.assertEqual(\u0027detached\u0027, result.attach_status)"},{"line_number":524,"context_line":"        self.assertEqual(\u0027available\u0027, result.status)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7cba755e_319ee084","line":521,"updated":"2024-12-18 22:09:32.000000000","message":"These asserts are good additions to the _clone_image_volume tests.","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"}],"releasenotes/notes/honor-user-id-clone-vol-2aced8b2bbc2cc3e.yaml":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"1a1523f475156e216c103facb503d8d163ac6e61","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #2056081 \u003chttps://bugs.launchpad.net/cinder/+bug/2056081\u003e`_:"},{"line_number":5,"context_line":"    When cloning a volume from a source volume, we used the user ID"},{"line_number":6,"context_line":"    of source volume user and not of the current user. Now we use the"},{"line_number":7,"context_line":"    user ID as provided in the user context that initiated the operation."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"44bbf555_1aa8fa43","line":6,"updated":"2024-12-18 22:09:32.000000000","message":"NIT: IMO the user word from \"source volume user\" seems a bit redundant and could be removed, tho I don\u0027t see it as an issue if you decide keeping this way.","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5fae2205f542c752e59d8a641ee743a94591f859","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #2056081 \u003chttps://bugs.launchpad.net/cinder/+bug/2056081\u003e`_:"},{"line_number":5,"context_line":"    When cloning a volume from a source volume, we used the user ID"},{"line_number":6,"context_line":"    of source volume user and not of the current user. Now we use the"},{"line_number":7,"context_line":"    user ID as provided in the user context that initiated the operation."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a8f66d04_4444bf5c","line":6,"in_reply_to":"44bbf555_1aa8fa43","updated":"2025-04-25 20:35:03.000000000","message":"My nit would be that we don\u0027t offer end users a \u0027clone\u0027 operation, we offer volume-create with a source_volid option.  But likewise, I don\u0027t think this is an issue to hold the patch up over.","commit_id":"c4b0b0ba15f11cc67d716c862d0ff04bbf607b57"}]}
