)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f3c7c98e5d4b7758281bdd4a0dede4f44983480","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, there is no coverage for the scenario of creating"},{"line_number":10,"context_line":"a volume from an instance snapshot. This patch adds a test for"},{"line_number":11,"context_line":"this functionality. Additionally, an issue was encountered when"},{"line_number":12,"context_line":"the Cinder backend is configured with generic NFS."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia860876fff50ec7a628bea1dab68e7bf56e5047b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"64cc5042_d78ddc95","line":12,"range":{"start_line":11,"start_character":20,"end_line":12,"end_character":50},"updated":"2025-04-21 13:43:19.000000000","message":"Probably no point mentioning this.  The issue occurs after the second snapshot is created on NFS.  There\u0027s already a cinder-tempest-plugin test that covers that scenario [0], but unfortunately, it wasn\u0027t being run on NFS (though there is a patch up to fix that [1], which I think was ignored because it was failing [0], which is addressed by [2]).\n\n[0] https://opendev.org/openstack/cinder-tempest-plugin/src/commit/bb5594c061370316bfc881e24d11b3350fb0aef9/cinder_tempest_plugin/scenario/test_snapshots.py#L42-L44\n[1] https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/898965\n[2] https://review.opendev.org/c/openstack/cinder/+/946995","commit_id":"b8bb9378165cbf9c081f4d5b08db5bb2e8cc1edb"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"4fba25ae789cd0a1609d4cdabb71d18c1f43289a","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, there is no coverage for the scenario of creating"},{"line_number":10,"context_line":"a volume from an instance snapshot. This patch adds a test for"},{"line_number":11,"context_line":"this functionality. Additionally, an issue was encountered when"},{"line_number":12,"context_line":"the Cinder backend is configured with generic NFS."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia860876fff50ec7a628bea1dab68e7bf56e5047b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"823d3ff4_f86970fc","line":12,"range":{"start_line":11,"start_character":20,"end_line":12,"end_character":50},"in_reply_to":"64cc5042_d78ddc95","updated":"2025-04-21 18:55:26.000000000","message":"You\u0027re right. I shouldn\u0027t have mentioned the issue. The reason I submitted this patch is that I noticed there\u0027s currently no dedicated test coverage for the scenario of creating a volume from an instance snapshot. This test is not related to the issue seen with the generic NFS backend.","commit_id":"b8bb9378165cbf9c081f4d5b08db5bb2e8cc1edb"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f5c81eba399d9f83fbb4bed21558946e0d84fe5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"412f3771_3ea30363","updated":"2025-03-20 01:46:17.000000000","message":"The new test faied with \"tempest.exceptions.VolumeResourceBuildErrorException: volume XXX failed to build and is in ERROR status\" in these jobs:\ntempest-full-2024.2\ntempest-full-2023.2\ndevstack-plugin-ceph-tempest-py3\n\nThe first 2 could be because something needs to be backported, though the third one is running on master, so maybe that\u0027s not it.  Also, the test did pass in at least tempest-full-py3 and tempest-full-ubuntu-jammy (maybe more, i didn\u0027t check everything).\n\nSeems worth trying a recheck.","commit_id":"ea16e0e6dc1e914b58289c798d63690e0fe98e31"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7fe7236c7832eb11aaa788fc008f14135daa552a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4dcd9ce_01fe2546","updated":"2025-03-20 01:46:59.000000000","message":"recheck devstack-plugin-ceph-tempest-py3 - volume went to error status","commit_id":"ea16e0e6dc1e914b58289c798d63690e0fe98e31"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f3c7c98e5d4b7758281bdd4a0dede4f44983480","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"50591cf9_8d488472","updated":"2025-04-21 13:43:19.000000000","message":"Issue about the commit message noted inline; otherwise, LGTM.","commit_id":"b8bb9378165cbf9c081f4d5b08db5bb2e8cc1edb"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"deb8bb74637d171ebd722f02befa4abca615fd14","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ee35c196_2b9e83d9","updated":"2025-04-08 08:11:59.000000000","message":"recheck","commit_id":"b8bb9378165cbf9c081f4d5b08db5bb2e8cc1edb"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"c1662440e96f221f5fbe176168473daf4c6d3199","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7355dcec_104f32f2","updated":"2025-04-24 09:07:32.000000000","message":"Thanks, Liron!\nLGTM","commit_id":"80ccfe7c4f214effeda9f94b3953f0b429957128"}],"tempest/scenario/test_volume_boot_pattern.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5c8ab7df86109a22b355f457085b7933f16f8fa","unresolved":true,"context_lines":[{"line_number":328,"context_line":""},{"line_number":329,"context_line":"class TestVolumeBootPatternV358(manager.EncryptionScenarioTest):"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"    volume_min_microversion \u003d \u00273.58\u0027"},{"line_number":332,"context_line":"    volume_max_microversion \u003d \u0027latest\u0027"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    @decorators.idempotent_id(\u00270b1e347b-bd19-47d4-b311-8374ca3b3c12\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ebb00ce2_6e74393a","line":331,"range":{"start_line":331,"start_character":0,"end_line":331,"end_character":36},"updated":"2025-04-03 17:13:19.000000000","message":"is 3.58 microversion added the functionality of create volume from snapshot? I see 3.58 only project_id is added in response field - https://docs.openstack.org/cinder/latest/contributor/api_microversion_history.html#id53","commit_id":"e1a9f3423231f1d6007b63adc7576512c272da0d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f3c7c98e5d4b7758281bdd4a0dede4f44983480","unresolved":true,"context_lines":[{"line_number":328,"context_line":""},{"line_number":329,"context_line":"class TestVolumeBootPatternV358(manager.EncryptionScenarioTest):"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"    volume_min_microversion \u003d \u00273.58\u0027"},{"line_number":332,"context_line":"    volume_max_microversion \u003d \u0027latest\u0027"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    @decorators.idempotent_id(\u00270b1e347b-bd19-47d4-b311-8374ca3b3c12\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5c10a7e7_8b0446af","line":331,"range":{"start_line":331,"start_character":0,"end_line":331,"end_character":36},"in_reply_to":"46b34deb_1b381e61","updated":"2025-04-21 13:43:19.000000000","message":"Yes, the documentation was bad on that one.  It\u0027s since been cleared up:\nhttps://review.opendev.org/#/q/I3e9723bb691c","commit_id":"e1a9f3423231f1d6007b63adc7576512c272da0d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"be44e1603924943d747b0ab0808dce4be2cb8379","unresolved":true,"context_lines":[{"line_number":328,"context_line":""},{"line_number":329,"context_line":"class TestVolumeBootPatternV358(manager.EncryptionScenarioTest):"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"    volume_min_microversion \u003d \u00273.58\u0027"},{"line_number":332,"context_line":"    volume_max_microversion \u003d \u0027latest\u0027"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    @decorators.idempotent_id(\u00270b1e347b-bd19-47d4-b311-8374ca3b3c12\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"46b34deb_1b381e61","line":331,"range":{"start_line":331,"start_character":0,"end_line":331,"end_character":36},"in_reply_to":"ebb00ce2_6e74393a","updated":"2025-04-03 19:58:37.000000000","message":"The correct microversion is 3.46, though this is not clearly stated in the description.\nhttps://docs.openstack.org/cinder/latest/contributor/api_microversion_history.html#id43","commit_id":"e1a9f3423231f1d6007b63adc7576512c272da0d"}]}