)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b863a56d02ca9853d8c80696451c2a8f63a1e6fa","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing deletion for the second volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Reason: the volume is created and not deleted."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic7473e880225cefdf3517b2d2f1372fadd072743"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"62fd04a5_dee58e92","line":9,"range":{"start_line":9,"start_character":34,"end_line":9,"end_character":45},"updated":"2025-12-05 14:28:57.000000000","message":"All volumes are deleted at the cleanup phase of the test. The create_volume() method adds a delete call that happens there.","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"},{"author":{"_account_id":38652,"name":"Mykhailo Ivanchenko","display_name":"Mykhailo Ivanchenko","email":"mykhailo.ivanchenko@netways.de","username":"mischa"},"change_message_id":"c0855065628c8995cb4f599fe75408413b5058ec","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing deletion for the second volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Reason: the volume is created and not deleted."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic7473e880225cefdf3517b2d2f1372fadd072743"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"c0bb3358_50936e67","line":9,"range":{"start_line":9,"start_character":34,"end_line":9,"end_character":45},"in_reply_to":"62fd04a5_dee58e92","updated":"2025-12-08 09:42:27.000000000","message":"Acknowledged","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"}],"cinder_tempest_plugin/api/volume/test_volume_dependency.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b863a56d02ca9853d8c80696451c2a8f63a1e6fa","unresolved":true,"context_lines":[{"line_number":210,"context_line":"        self.volumes_client.delete_volume(volume1[\u0027id\u0027])"},{"line_number":211,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume1[\u0027id\u0027])"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"        self.volumes_client.delete_volume(volume2[\u0027id\u0027])"},{"line_number":214,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume2[\u0027id\u0027])"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"        self.del_image(image[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"69781fd7_7806ccc4","line":213,"updated":"2025-12-05 14:28:57.000000000","message":"This changes the purpose of this test - currently it is ensuring you can delete the image before volume2 is deleted.","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"dfe4ef03b2be75b6b02a6acfb9b5b11302eab307","unresolved":true,"context_lines":[{"line_number":210,"context_line":"        self.volumes_client.delete_volume(volume1[\u0027id\u0027])"},{"line_number":211,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume1[\u0027id\u0027])"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"        self.volumes_client.delete_volume(volume2[\u0027id\u0027])"},{"line_number":214,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume2[\u0027id\u0027])"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"        self.del_image(image[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"82651a6d_7157bc28","line":213,"in_reply_to":"22cd2236_a03a6ff7","updated":"2026-02-02 17:45:31.000000000","message":"This test will pass as-is when Ceph is used if it is configured to use the RBD v2 image format. This is the recommended configuration for Cinder now, but I think we aren\u0027t documenting that expectation currently.","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"},{"author":{"_account_id":38652,"name":"Mykhailo Ivanchenko","display_name":"Mykhailo Ivanchenko","email":"mykhailo.ivanchenko@netways.de","username":"mischa"},"change_message_id":"c0855065628c8995cb4f599fe75408413b5058ec","unresolved":true,"context_lines":[{"line_number":210,"context_line":"        self.volumes_client.delete_volume(volume1[\u0027id\u0027])"},{"line_number":211,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume1[\u0027id\u0027])"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"        self.volumes_client.delete_volume(volume2[\u0027id\u0027])"},{"line_number":214,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume2[\u0027id\u0027])"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"        self.del_image(image[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"22cd2236_a03a6ff7","line":213,"in_reply_to":"69781fd7_7806ccc4","updated":"2025-12-08 09:42:27.000000000","message":"In cases where Ceph is used as a backend for storage, volumes and images are not independent, meaning that this configuration fails the test as the functionality is not intended. Would it be best to skip the test in that case?","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b863a56d02ca9853d8c80696451c2a8f63a1e6fa","unresolved":true,"context_lines":[{"line_number":258,"context_line":"        self.volumes_client.delete_volume(volume1[\u0027id\u0027])"},{"line_number":259,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume1[\u0027id\u0027])"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        self.del_image(image[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"17796b4e_7f71e06b","line":261,"updated":"2025-12-05 14:28:57.000000000","message":"This does not meaningfully change the test, this image is deleted in the cleanup phase already.","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"},{"author":{"_account_id":38652,"name":"Mykhailo Ivanchenko","display_name":"Mykhailo Ivanchenko","email":"mykhailo.ivanchenko@netways.de","username":"mischa"},"change_message_id":"c0855065628c8995cb4f599fe75408413b5058ec","unresolved":true,"context_lines":[{"line_number":258,"context_line":"        self.volumes_client.delete_volume(volume1[\u0027id\u0027])"},{"line_number":259,"context_line":"        self.volumes_client.wait_for_resource_deletion(volume1[\u0027id\u0027])"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        self.del_image(image[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"221e614b_0bfa3163","line":261,"in_reply_to":"17796b4e_7f71e06b","updated":"2025-12-08 09:42:27.000000000","message":"As described in the other comment - ceph-backends do not have the functionality of independently deleting images if said image has been used in creating a volume. This makes the test pass, but ultimately undermines the purpose of the test. Would it be best to skip the test in that case?","commit_id":"eec96e13dd307ae7f2d1d3ccd53e1cc6575710c3"}]}