)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f2cb8ce0d0a7fe94a4fc98120ee54dda5f91eb96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9ab07d98_15ee8303","updated":"2025-11-04 15:12:14.000000000","message":"recheck","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"9ed494b12a9266c21489b0613b04d4662c04a1c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0a5b9b06_443b9539","updated":"2025-11-03 20:50:28.000000000","message":"recheck\n\nUnrelated spurious error building tempest VM","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"1ebaced6c3a07e4a7ad9e4ac0acf21f5eeb85ec5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"44b4869c_815d3d0c","updated":"2025-11-04 04:02:20.000000000","message":"recheck\n\nUnrelated tempest ceph failure","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"f3c949d25dad671cfe9935d9e0482ad15027b268","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31d04f44_d513c1c6","updated":"2025-11-13 14:54:47.000000000","message":"@abishop@redhat.com Thanks for working on this patch.\nPatch looking good,As I already reviewed the code on github.\nI think this improvement should be mention in ``dell-emc-powerflex-driver.rst``.","commit_id":"3ac19bfea80636fb6e00e7a4b478e83cdece850e"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"f3a160b0d35460b276d81f5bf8cf70be9a1912bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9a83295e_33dcc181","updated":"2025-11-14 15:19:26.000000000","message":"Code looks good to me, CI passes, I checked test coverage - that looks good, tests are good, release note is in order, dell has reportedly tested this already.  No objection from me.","commit_id":"3ac19bfea80636fb6e00e7a4b478e83cdece850e"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"e2c90bc80123196d06574f39d2ee6cd12aa11eba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dfe58d30_bfc77ca7","updated":"2025-11-14 11:41:47.000000000","message":"LGTM","commit_id":"3ac19bfea80636fb6e00e7a4b478e83cdece850e"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"b488d40d242f53a359a37f4b0163f5c0b8b6ba2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ccd56546_7f945d7a","updated":"2025-11-14 09:55:47.000000000","message":"LGTM, thank you Alan for bringing this up.","commit_id":"3ac19bfea80636fb6e00e7a4b478e83cdece850e"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"29aa57473e01f5136fcb5e3745a7b34dfab8070d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f22394c7_7c01257a","updated":"2025-11-14 17:30:37.000000000","message":"recheck\n\nunrelated failure in cinder-plugin-ceph-tempest","commit_id":"3ac19bfea80636fb6e00e7a4b478e83cdece850e"}],"cinder/tests/unit/test_volume_utils.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0b5c62f8a1e3dee9f7bae2e8c4eae753d6a4983e","unresolved":true,"context_lines":[{"line_number":1344,"context_line":""},{"line_number":1345,"context_line":"    def test_is_image_cache_entry_no_context(self):"},{"line_number":1346,"context_line":"        \"\"\"Test volume returns False when no context is available.\"\"\""},{"line_number":1347,"context_line":"        from cinder import objects"},{"line_number":1348,"context_line":"        volume \u003d objects.Volume()"},{"line_number":1349,"context_line":""},{"line_number":1350,"context_line":"        result \u003d volume_utils.is_image_cache_entry(volume)"}],"source_content_type":"text/x-python","patch_set":1,"id":"54bcd3af_197820eb","line":1347,"updated":"2025-11-04 15:32:54.000000000","message":"This import should be at the top of the file.","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"15bf303dee74292e37341306834a567146e5c7a0","unresolved":false,"context_lines":[{"line_number":1344,"context_line":""},{"line_number":1345,"context_line":"    def test_is_image_cache_entry_no_context(self):"},{"line_number":1346,"context_line":"        \"\"\"Test volume returns False when no context is available.\"\"\""},{"line_number":1347,"context_line":"        from cinder import objects"},{"line_number":1348,"context_line":"        volume \u003d objects.Volume()"},{"line_number":1349,"context_line":""},{"line_number":1350,"context_line":"        result \u003d volume_utils.is_image_cache_entry(volume)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f9c57d90_54f9026e","line":1347,"in_reply_to":"54bcd3af_197820eb","updated":"2025-11-04 17:25:45.000000000","message":"Done","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"}],"cinder/tests/unit/volume/drivers/dell_emc/powerflex/test_create_cloned_volume.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9c5c8354b3f191b62cad4407bc2e3b06d048f54b","unresolved":true,"context_lines":[{"line_number":155,"context_line":"        mock_create.return_value \u003d {}"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        # Set clone limit to 20"},{"line_number":158,"context_line":"        setattr(self.driver.configuration,"},{"line_number":159,"context_line":"                options.POWERFLEX_MAX_IMAGE_CACHE_VTREE_SIZE, 20)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"        # Mock REST client and vtree statistics response"},{"line_number":162,"context_line":"        mock_rest_client \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"f52b89fd_df8bdaa8","line":159,"range":{"start_line":158,"start_character":8,"end_line":159,"end_character":65},"updated":"2025-11-04 15:35:59.000000000","message":"This will leak config settings between tests, should be done in a way where the option is unset after the test.","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"89bec6f5169b06f1f2fe69fca6f90ff87461bcd1","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        mock_create.return_value \u003d {}"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        # Set clone limit to 20"},{"line_number":158,"context_line":"        setattr(self.driver.configuration,"},{"line_number":159,"context_line":"                options.POWERFLEX_MAX_IMAGE_CACHE_VTREE_SIZE, 20)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"        # Mock REST client and vtree statistics response"},{"line_number":162,"context_line":"        mock_rest_client \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"40648f75_d68402c4","line":159,"range":{"start_line":158,"start_character":8,"end_line":159,"end_character":65},"in_reply_to":"58e76104_664e157d","updated":"2025-11-14 15:11:02.000000000","message":"Done","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"15bf303dee74292e37341306834a567146e5c7a0","unresolved":true,"context_lines":[{"line_number":155,"context_line":"        mock_create.return_value \u003d {}"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        # Set clone limit to 20"},{"line_number":158,"context_line":"        setattr(self.driver.configuration,"},{"line_number":159,"context_line":"                options.POWERFLEX_MAX_IMAGE_CACHE_VTREE_SIZE, 20)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"        # Mock REST client and vtree statistics response"},{"line_number":162,"context_line":"        mock_rest_client \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"58e76104_664e157d","line":159,"range":{"start_line":158,"start_character":8,"end_line":159,"end_character":65},"in_reply_to":"f52b89fd_df8bdaa8","updated":"2025-11-04 17:25:45.000000000","message":"Done","commit_id":"11664fd0c7ac026eadc66162833bb6367973937c"}]}
