)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"ddfcc03078fb5d99c261312afd8a1d86d7b14e5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cb1d2aac_337c3409","updated":"2023-06-28 07:58:30.000000000","message":"Code changes look fine.\nSeems to have some zuul failure. Re-running zuul.","commit_id":"22a72fa422905289e5e3fac80427c57e274df841"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0b23ca6712ce43d37f36835c726cdefae4ecec5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"852976ef_bbc26dd8","updated":"2023-06-28 07:58:39.000000000","message":"recheck","commit_id":"22a72fa422905289e5e3fac80427c57e274df841"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2e556213eb09632b73743c7d63af63d9b02d7550","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"28c4e0f0_dd19dbf5","updated":"2023-06-28 14:12:11.000000000","message":"The nfs job is failling because of `{1} tempest.api.compute.servers.test_server_actions.ServerActionsV293TestJSON.test_rebuild_volume_backed_server [380.484194s] ... FAILED` due to https://bugs.launchpad.net/cinder/+bug/2025096.\n\nPlease rebase the patch : https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/887003","commit_id":"06dcc3174462a24e0cdcfc48f3d371eeb5bbf455"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"5b6f4e4fead99ab752e0318003397516a4016127","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76e123d7_432629ee","updated":"2023-06-28 13:32:05.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican failure","commit_id":"06dcc3174462a24e0cdcfc48f3d371eeb5bbf455"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"2f5ff9c1bd9d4fc41e52f940b72592c8b47e678c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b04eff75_95ab808e","in_reply_to":"28c4e0f0_dd19dbf5","updated":"2023-06-28 14:27:13.000000000","message":"?\n\nSorry I don\u0027t understand what you mean.","commit_id":"06dcc3174462a24e0cdcfc48f3d371eeb5bbf455"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7b07cc6f94f397e1a6b22e34251f21e9b1623e60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0202d431_20478e03","updated":"2023-06-29 19:16:11.000000000","message":"rechech devstack-plugin-nfs-tempest-full failures","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"af62143fe7a08fb02d417c91c017452d0b10a580","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"723fb108_dd0421c5","updated":"2023-07-03 11:16:13.000000000","message":"recheck","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"701775e35ad4689765c6127293f4e9c7dad5dcc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"79265fcf_011b6842","updated":"2023-06-30 10:15:37.000000000","message":"recheck","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"6ea3b1080513987922dee1a5fceb25cd1fd218fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8bdd4e3f_90594a43","updated":"2023-08-04 04:46:20.000000000","message":"recheck","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"91bea3c7d300f783dd07ac45bfd385899418764d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9a49469e_b23ef8f5","updated":"2023-07-11 06:38:20.000000000","message":"recheck","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"0363d77ffa294696cd4ec79a3b62bde518e1a0d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b632fca_7e64dcf1","updated":"2023-06-30 05:51:22.000000000","message":"recheck devstack-plugin-nfs-tempest-full failures","commit_id":"5948ee3e2dea70e79a4ec2b8069a6c546fb0668d"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"0511642131f5c62a8cd1804c5fee6f42c2d87143","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"62f8869d_23a8a3e8","updated":"2023-08-07 07:19:06.000000000","message":"recheck","commit_id":"f56d0e8e59c2421d4e8098bf32119d43fed3ceab"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"a3dfdf49a639424b0f70d203b18425593dd037af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3d2fdf1d_e5039bd0","updated":"2023-09-12 12:59:59.000000000","message":"Can\u0027t have ZUUL checks pass. I don\u0027t think this is related to our change.","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7d950ae713178b311159fd66dbb889bea9f070b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"16fa9082_6c4a197e","updated":"2023-08-30 12:38:54.000000000","message":"recheck","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"a0f92dc68d2d82d6e0b28d66e0babcdade1b066b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5d1b1190_e8e68e6a","updated":"2023-08-23 13:06:50.000000000","message":"recheck","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"9dc6dc4e7d525e50a8417e14f5203da2ab60550d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7aaa6b7c_d9440805","updated":"2023-09-06 14:13:57.000000000","message":"recheck","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"1d9c1eca55b3112d5a18ecae5315785765dd91c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1f0d6898_836b992c","updated":"2023-09-11 09:49:18.000000000","message":"recheck many failures","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"762cdbb36dc95d8948a1e3c363b7a0ddf0ac4ec4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c6e61e5c_d1cf333d","updated":"2023-09-12 13:43:49.000000000","message":"recheck several failures","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"ca970547af54bdb4a80addd495d60ac8fd37fc2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c16e6a1d_1fce0e6d","updated":"2023-09-13 14:59:38.000000000","message":"tempest.api.compute.servers.test_server_actions.ServerActionsV293TestJSON.test_rebuild_volume_backed_server fails in the NFS job with\n\n    self.assertEqual(self.image_ref_alt, image_after_rebuild)\n    testtools.matchers._impl.MismatchError: !\u003d:\n    reference \u003d \u0027379bdf3a-7b0a-4d77-af00-fed8ce99494c\u0027\n    actual    \u003d \u002786182c71-5229-44c3-94f5-5730ebf4e8ed\u0027\n\nIs that caused by this patch?","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"4d3c0973eb989bb0d0a54d9a00befbf386c1ac0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"25432f22_ad580ed2","in_reply_to":"c16e6a1d_1fce0e6d","updated":"2023-09-13 16:04:37.000000000","message":"That patch adds a check for format, and I don\u0027t think it is related.\nhttps://review.opendev.org/c/openstack/cinder/+/887081/5/cinder/volume/drivers/remotefs.py#550","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"d8831f7f8ae785c989a957ca4df73974eb178644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"46fda498_691fe57e","updated":"2023-09-20 15:30:22.000000000","message":"recheck","commit_id":"1f1965517856260788786a4d9a051fa16836e387"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"12e2fbaa877d08c82c4a96742e26c7127c01a24d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8443c838_fdaf6c86","updated":"2023-09-20 10:00:47.000000000","message":"recheck","commit_id":"1f1965517856260788786a4d9a051fa16836e387"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"88e92a79653529adc709972c63d4163760290e32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d35a9b43_d62db92b","updated":"2023-10-03 11:27:18.000000000","message":"As the original bug reviewed with current release doesn\u0027t appear anymore, I\u0027ll abandon this patch. \nThank you.","commit_id":"8025ceba92f5e45912b7b08de1176be24721863b"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"e1494b0f1044f7554c58b6ee22b373c1e232c3de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"19eb3d0e_902f033d","updated":"2023-09-21 14:01:04.000000000","message":"recheck","commit_id":"8025ceba92f5e45912b7b08de1176be24721863b"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"1cb89f050809b67c9987ac1031805e340f9ee192","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"90c75875_4da47b7c","updated":"2023-09-27 07:55:58.000000000","message":"run-DellEMC PowerStore CI","commit_id":"8025ceba92f5e45912b7b08de1176be24721863b"}],"cinder/volume/drivers/remotefs.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f457417f9cf7f0fa89d0d93a33304bf08f44a121","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"49f765f0_387e0ded","line":550,"updated":"2023-09-15 14:27:38.000000000","message":"I\u0027m not sure that we can always consume the file_format from qemu_img_info directly like this -- I think it will return \"luks\" for encrypted volumes which should be marked as \"raw\" in Cinder.","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"1eac9565892a7b5fd89ce63b44d0bc25e21690cf","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"add38165_3e1e9e84","line":550,"in_reply_to":"49f765f0_387e0ded","updated":"2023-09-18 10:24:01.000000000","message":"Ok, will give a try on LUKS devices and see what we\u0027ll get.","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"9d028d06802361f6200b2c19f28686e404f4c207","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"af29a17e_46d3bdda","line":550,"in_reply_to":"add38165_3e1e9e84","updated":"2023-09-19 12:03:31.000000000","message":"While I was trying to create an volume from image with LUKS encryption, I stumbled across this bug https://bugs.launchpad.net/cinder/+bug/1888680 which was supposed to be fixed.","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"04fbb83a361b982ce4174e6fbd623aa30a7fbc31","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"df97ae66_fef78333","line":550,"in_reply_to":"af29a17e_46d3bdda","updated":"2023-09-19 14:00:30.000000000","message":"It turns out that the patch which should fix that is not merged yet.\nhttps://review.opendev.org/c/openstack/cinder/+/749155","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"d651cac37e51c576ef2e36c49d3cfee62ec43af9","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"e9db2be7_3d1be89f","line":550,"in_reply_to":"df97ae66_fef78333","updated":"2023-09-20 06:41:08.000000000","message":"After several tests, I have concluded that img file can be of the following format\n- raw\n- qcow2\n- luks\n\nFor the latter, and once Sofia\u0027s fix will be merged, a luks volume will be encapsulated into a qcow2 volume.\n\nFor now, I\u0027ll make a change and add a test which align the current img format with the one present in the volume metadata except for luks volume which will remain in \u0027raw\u0027 format.\n\nHope it makes sense.","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"32fc297e059304f187b6f210a2622d7c25dd27b0","unresolved":false,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"        data \u003d image_utils.qemu_img_info(self.local_path(volume),"},{"line_number":549,"context_line":"                                         run_as_root\u003dself._execute_as_root)"},{"line_number":550,"context_line":"        disk_fmt \u003d data.file_format"},{"line_number":551,"context_line":"        curr_disk_fmt \u003d volume.admin_metadata.get(\u0027format\u0027)"},{"line_number":552,"context_line":"        # NOTE (happystacker): updating metadata when current format does"},{"line_number":553,"context_line":"        # not match the metadata format"}],"source_content_type":"text/x-python","patch_set":5,"id":"c1532391_38c81007","line":550,"in_reply_to":"e9db2be7_3d1be89f","updated":"2023-09-20 07:32:18.000000000","message":"Done","commit_id":"0c33a1b583d2f10c57c5c4271e808966c9b673f2"}]}