)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f207b0589f46b42c30a15892e4aadc7d09e16b62","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create volume from image delete the used image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Based on LP1951433."},{"line_number":10,"context_line":"This testcase verify:"},{"line_number":11,"context_line":"- Create a volume from image"},{"line_number":12,"context_line":"- Delete the used image"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"0635d6ad_0873d560","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":18},"updated":"2022-01-13 16:17:55.000000000","message":":-1: Since this is a cinder-tempest-plugin project I\u0027ve added it to the launchpad report so we could track it. We really need to add the closes bug tag at the bottom of the commit msg [1]\n\nCloses-bug: #1951433\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f207b0589f46b42c30a15892e4aadc7d09e16b62","unresolved":true,"context_lines":[{"line_number":12,"context_line":"- Delete the used image"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This test will be skipped till issue solved."},{"line_number":15,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782624/"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I772b22a0054ada26337b91d3f3f885e40b2c46f5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"d9dea4a7_60f3ba42","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":81},"updated":"2022-01-13 16:17:55.000000000","message":"I added this line here just to check the job output but i\u0027m not sure if we need it.","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f3c5cd12187e07428c861a448bd764964df4e631","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9b5e5e1a_63c7c921","updated":"2021-12-07 18:41:19.000000000","message":"Looks good to me, thanks Benny. We need to remember to enable the test after the fix.","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"1baef250dd131ff47a33e8767931c93b58e153e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"74bb018c_c314f857","updated":"2021-12-16 15:44:48.000000000","message":"Lowering my vote until the test stop being skipped","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"1485e5e9f2093ef8af05eeb23f297dbbc0f921a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cfdc766e_8427ba9a","updated":"2021-12-16 15:55:48.000000000","message":"also it seems there will be a difference in the behavior when RBD is used (the image couldn\u0027t be deleted anyway), unless I\u0027ve mistaken the current status","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"4ea865c442f841403729db39fa15063220409ba4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95330fe7_0358923a","in_reply_to":"cfdc766e_8427ba9a","updated":"2021-12-16 16:04:30.000000000","message":"Yes , if with the clonev2 we are not able to delete the image , i would vote to merge current one and i will write another case for ceph only because its a different behaviour,\n\nunless we are going to keep ceph aligned as other backends and image will be deleted.","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"d1b70246126cbfc9de7e78e6db036155f5a0edd3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"137768b5_339f8fed","updated":"2022-01-13 13:33:12.000000000","message":"shouldn\u0027t this work because of https://review.opendev.org/c/openstack/glance_store/+/819481 ?","commit_id":"db74f9b5d5a26a3a947abea4a0e86d78780918af"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"8bab068c329f2514c597ede288120b07a9843ede","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"942e7170_345ebeea","in_reply_to":"137768b5_339f8fed","updated":"2022-01-13 14:31:23.000000000","message":"I think not until new glance_store library is released with the fix (unless it is using master branch)","commit_id":"db74f9b5d5a26a3a947abea4a0e86d78780918af"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"cabf4785f9af2b81ed4e6de8f2b88351f2c1a821","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2275403c_4fa445c0","in_reply_to":"137768b5_339f8fed","updated":"2022-01-17 12:11:49.000000000","message":"let me fix the pep8 issue , re-run . i think it should work.","commit_id":"db74f9b5d5a26a3a947abea4a0e86d78780918af"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f207b0589f46b42c30a15892e4aadc7d09e16b62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e1afc455_61bd37bc","in_reply_to":"942e7170_345ebeea","updated":"2022-01-13 16:17:55.000000000","message":"Looks like Zuul is using master \\o/\n\n```\n+ inc/python:use_library_from_git:252      :   [[ ,cinder,cinder-tempest-plugin,devstack,glance,keystone,neutron,nova,placement,requirements,swift,tempest, \u003d~ ,glance_store, ]]\n\nSuccessfully installed glance-store-2.7.0\n```\nFinally, as expected, the test failed with 409.\n```\n\nDetails: {\u0027message\u0027: \u0027The image cannot be deleted because it has snapshot(s).\u003cbr /\u003e\u003cbr /\u003e\\n\\n\\n\u0027, \u0027code\u0027: \u0027409 Conflict\u0027, \u0027title\u0027: \u0027Conflict\u0027}\n\n```","commit_id":"db74f9b5d5a26a3a947abea4a0e86d78780918af"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f207b0589f46b42c30a15892e4aadc7d09e16b62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cfc2221c_03ce1a05","updated":"2022-01-13 16:17:55.000000000","message":"I guess we need to wait until the fix ? Because skipping the test will be useless for us. Or should we catch the 409 error when using the ceph backend?","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f7fa7a4c8aa80dce71ff13dadd874ce3fca5dec7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a4921b99_c2a421e4","updated":"2024-05-29 12:15:47.000000000","message":"I think this same scenario is covered here now:\n    https://opendev.org/openstack/cinder-tempest-plugin/src/commit/e0f76426c045ab0f8ee1c6dcb84d62c987379104/cinder_tempest_plugin/api/volume/test_volume_dependency.py#L217\n\n\nThough, test_image_volume_dependencies_3 is slightly more complex.  If we want a simpler test for this specific scenario it would probably make sense to add it to test_volume_dependency.py in the same structure as the tests present there, since they are all about similar scenarios.","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":33612,"name":"yuval","email":"yuval@lightbitslabs.com","username":"yuval"},"change_message_id":"60fe122a62dca0adabec111c980f82ed1c1b54bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9abea382_6663a624","updated":"2022-01-17 11:26:05.000000000","message":"Looks like something is not working as expected in the tempest for example:\ntempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_rebuild_server\nfailed on:\nDetails: {\u0027code\u0027: 500, \u0027created\u0027: \u00272022-01-13T15:50:15Z\u0027, \u0027message\u0027: \u0027libvirtError\u0027}\n\nand most of the test that failed have the same error.\nin the libvirtd log I see many times:\n2022-01-13 15:38:46.331+0000: 144873: info : qemuMonitorIOWrite:439 : QEMU_MONITOR_IO_WRITE: mon\u003d0x7f73101b25d0 buf\u003d{\"execute\":\"blockdev-del\",\"arguments\":{\"node-name\":\"libvirt-6-format\"},\"id\":\"libvirt-412\"}\n len\u003d92 ret\u003d92 errno\u003d0\n2022-01-13 15:38:46.343+0000: 144873: debug : qemuMonitorJSONIOProcessLine:222 : Line [{\"id\": \"libvirt-412\", \"error\": {\"class\": \"GenericError\", \"desc\": \"Failed to find node with node-name\u003d\u0027libvirt-6-format\u0027\"}}]\n\nsomething is missing to delelte the blockdev  - if that is expected ignore this comment","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":33612,"name":"yuval","email":"yuval@lightbitslabs.com","username":"yuval"},"change_message_id":"f206131ed14fab616f5a3ab88c8e19446c8ae2b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"834dfa20_346573b1","updated":"2022-01-13 14:17:06.000000000","message":"This looks really interesting - is there a guide about how to create new tempest tests? ","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":33612,"name":"yuval","email":"yuval@lightbitslabs.com","username":"yuval"},"change_message_id":"b7e952e7e8eae51d5246aedabee42017da1e1453","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9cc2347a_526dc3a1","in_reply_to":"71b9f6c7_d6936a1b","updated":"2022-01-17 12:34:44.000000000","message":"sorry, ignore my comment I was going over the logs:\ncinder-tempest-plugin-lvm-lio-barbican-centos-8-stream https://zuul.opendev.org/t/openstack/build/6c18dd4c853e4626841ba59dfbbd1b62 : FAILURE in 1h 55m 35s (non-voting)\n\nwhich are non-voting to I guess its ok if it fails","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f207b0589f46b42c30a15892e4aadc7d09e16b62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"33103db9_af6c045a","in_reply_to":"834dfa20_346573b1","updated":"2022-01-13 16:17:55.000000000","message":"Sadly cinder-tempest-plugin doesn\u0027t have much documentation but let me share the general tempest doc https://docs.openstack.org/tempest/latest/write_tests.html","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"cabf4785f9af2b81ed4e6de8f2b88351f2c1a821","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"71b9f6c7_d6936a1b","in_reply_to":"9abea382_6663a624","updated":"2022-01-17 12:11:49.000000000","message":"Hey Yuval , \nIs it related to the current patch ? id it something you see here or something that not related to the cinder-tempest-plugin ?\n\nCould you share a link or the tempest testcase ?","commit_id":"bfdbc4d91f47da8aef6b83a7092c81df3952723c"}],"cinder_tempest_plugin/api/volume/test_create_from_image.py":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f3c5cd12187e07428c861a448bd764964df4e631","unresolved":true,"context_lines":[{"line_number":91,"context_line":"    @testtools.skipIf(CONF.volume.storage_protocol \u003d\u003d \u0027ceph\u0027,"},{"line_number":92,"context_line":"                      \u0027Skip because of Bug#1951433\u0027)"},{"line_number":93,"context_line":"    def test_create_volume_from_image_delete_in_use_image(self):"},{"line_number":94,"context_line":"        \"\"\"This testcase based on LP1951433"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        The test verify image deletion when in-use by cinder."},{"line_number":97,"context_line":"        Create an image"}],"source_content_type":"text/x-python","patch_set":1,"id":"f4747155_fe6ed4ee","line":94,"range":{"start_line":94,"start_character":34,"end_line":94,"end_character":43},"updated":"2021-12-07 18:41:19.000000000","message":"Sorry but not sure what that means","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"5a45497cdb7a1235ea933b71ab46b04adfe16560","unresolved":true,"context_lines":[{"line_number":91,"context_line":"    @testtools.skipIf(CONF.volume.storage_protocol \u003d\u003d \u0027ceph\u0027,"},{"line_number":92,"context_line":"                      \u0027Skip because of Bug#1951433\u0027)"},{"line_number":93,"context_line":"    def test_create_volume_from_image_delete_in_use_image(self):"},{"line_number":94,"context_line":"        \"\"\"This testcase based on LP1951433"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        The test verify image deletion when in-use by cinder."},{"line_number":97,"context_line":"        Create an image"}],"source_content_type":"text/x-python","patch_set":1,"id":"2f029f97_152a0c26","line":94,"range":{"start_line":94,"start_character":34,"end_line":94,"end_character":43},"in_reply_to":"f4747155_fe6ed4ee","updated":"2022-01-12 15:55:57.000000000","message":"Reference to Launchad bug;\nhttps://bugs.launchpad.net/tripleo/+bug/1951433","commit_id":"f367f49148a5a26e7a1a3b8e13570fa401f0ab35"}]}