)]}'
{".zuul.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a3eed769714a306fc8378fd97be6349915b971bb","unresolved":true,"context_lines":[{"line_number":214,"context_line":"        post-config:"},{"line_number":215,"context_line":"          $CINDER_CONF:"},{"line_number":216,"context_line":"            DEFAULT:"},{"line_number":217,"context_line":"              cluster: ceph"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"- job:"},{"line_number":220,"context_line":"    name: cinder-grenade-mn-sub-bak"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"34f2d6bc_3421fd2d","line":217,"updated":"2024-05-14 15:15:20.000000000","message":"This job doesn\u0027t have cinder-plugin-ceph-tempest as a parent, so I think we need to turn off the volume-image-dep tests here?","commit_id":"7aca828dd2dc4add159d8006b2f25be23c015a77"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8ca355387713bbbdb1786acd5df3a373ebe909cd","unresolved":true,"context_lines":[{"line_number":214,"context_line":"        post-config:"},{"line_number":215,"context_line":"          $CINDER_CONF:"},{"line_number":216,"context_line":"            DEFAULT:"},{"line_number":217,"context_line":"              cluster: ceph"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"- job:"},{"line_number":220,"context_line":"    name: cinder-grenade-mn-sub-bak"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"78f3ce57_0c8a0c55","line":217,"in_reply_to":"34f2d6bc_3421fd2d","updated":"2024-05-14 15:18:21.000000000","message":"yes, I\u0027d say we should too (even thought we should need to recheck the status of this job)","commit_id":"7aca828dd2dc4add159d8006b2f25be23c015a77"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7554f55354230eda6f94c150b097cd9e75765c74","unresolved":false,"context_lines":[{"line_number":214,"context_line":"        post-config:"},{"line_number":215,"context_line":"          $CINDER_CONF:"},{"line_number":216,"context_line":"            DEFAULT:"},{"line_number":217,"context_line":"              cluster: ceph"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"- job:"},{"line_number":220,"context_line":"    name: cinder-grenade-mn-sub-bak"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d6c96722_b69453fc","line":217,"in_reply_to":"78f3ce57_0c8a0c55","updated":"2024-05-14 20:43:33.000000000","message":"Done","commit_id":"7aca828dd2dc4add159d8006b2f25be23c015a77"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"1a0d8e9feea0f80fd81b2665033d4751d4f1e473","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"addc7161_386b1974","updated":"2024-05-10 16:40:44.000000000","message":"Uhm, it looks like the depends-on was not applied?","commit_id":"cce162a3e91d6ca43b10abe85052186cc5ee88af"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0b2b2750e13ca51c6ee0564a0446a28e579862bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3950c851_b400ceaa","in_reply_to":"addc7161_386b1974","updated":"2024-05-10 21:39:46.000000000","message":"Yeah, that\u0027s what I thought too.  I wonder if whether the redirect caused by the short format URL I used is the culprit?","commit_id":"cce162a3e91d6ca43b10abe85052186cc5ee88af"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"46b078a5e905effb7b776fcaa9309e1e62f4a953","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4d49f8c4_d4c6f634","updated":"2024-05-13 13:30:13.000000000","message":"Guess the URL format in the Depends-on was the culprit.  The depends-on is being applied now:\nhttps://zuul.opendev.org/t/openstack/build/8c4df422e274415d90a056aa329b91b8/log/job-output.txt#850-851\n\n... and you can see the skip happening here:\nhttps://zuul.opendev.org/t/openstack/build/8c4df422e274415d90a056aa329b91b8/log/job-output.txt#24654","commit_id":"2e125456a8429073441997b3b674e157ec7fbbb3"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"28f350873afcc2db2fed877f60df2c86d0e0ff7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"89612e5e_23f42a53","updated":"2024-05-14 14:11:22.000000000","message":"There are unrelated errors, but cinder-plugin-ceph-tempest  shows that the new tests are correctly skipped.\nMy suggestion is:\n- remove the Depends-On\n- create a cherry-pick on 2023.1 (zed was just removed, so no need)\n- merge this review and its 2023.1 backport\n- merge https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/880077","commit_id":"2e125456a8429073441997b3b674e157ec7fbbb3"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8cd18b8ece7e4187faba006cc95743203a9eb969","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2e84383e_49afcdf1","updated":"2024-05-14 07:52:30.000000000","message":"recheck\n\nThe review in depends-on was heavily changed","commit_id":"2e125456a8429073441997b3b674e157ec7fbbb3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7554f55354230eda6f94c150b097cd9e75765c74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6beb5ae2_3c78c0dd","updated":"2024-05-14 20:43:33.000000000","message":"You can see that the skips are working correctly:\n\ncinder-plugin-ceph-tempest \nhttps://zuul.opendev.org/t/openstack/build/06a3a48f45424f4da343e56c5a1596d7/log/job-output.txt#24208-24210\n\ncinder-plugin-ceph-tempest-mn-aa\nhttps://zuul.opendev.org/t/openstack/build/f45eac0eb12c410c9bdd0f6725ec5aba/log/job-output.txt#37254-37256","commit_id":"9ddb8e4d06416401dc58c94b4330981783642607"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"5b6f83312fb8a7cd66f2eefee6e4e11da66119b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3b07d0b2_966b822e","updated":"2024-05-15 14:18:32.000000000","message":"Looks good to me.  Also is submitted by a stable core and has a +2 from another one.  Safe to merge.","commit_id":"1ec51a0af0bc505ef201503afb13eab210a30fa3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"871e420ab64dff790643d6a9d876376db01989cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"46918764_d00d730e","updated":"2024-05-14 20:46:03.000000000","message":"Removed the depends-on so we can merge the stable gate patches before change I5fee23951958fc00","commit_id":"1ec51a0af0bc505ef201503afb13eab210a30fa3"}]}