)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"cc0d658f5d78c3458a137dc983886683de7d779b","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Updated the README to indicate this change."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-on: Ib186ac5830e5920e264d79be946995e63e960426"},{"line_number":16,"context_line":"Change-Id: I4722b869033ad1bd357e36c4a258b6d3ea61f5d6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c73dc256_21456485","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":53},"updated":"2024-01-05 01:41:01.000000000","message":"I think this also needs to:\n\nDepends-On: https://review.opendev.org/c/openstack/project-config/+/903260\n\n(also major tangent but the preferred format of Depends-On, is change URLs rather than Change-Id","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"af5de1b05b856bf0f7150dee6794f824bbe9215c","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Updated the README to indicate this change."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-on: Ib186ac5830e5920e264d79be946995e63e960426"},{"line_number":16,"context_line":"Change-Id: I4722b869033ad1bd357e36c4a258b6d3ea61f5d6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"776cfbf9_5cd20bb4","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":53},"in_reply_to":"3850e4f5_3f5ef2cf","updated":"2024-01-05 14:41:45.000000000","message":"I went ahead and added the dependency Tony requested, just to keep the network of patches across repos clear.  (I didn\u0027t use the URL format, though, because all the other patches are using change-id, and I wanted to be consistent.)","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e9befcea04f8acb283fad5ea60f12e9756588c02","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Updated the README to indicate this change."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-on: Ib186ac5830e5920e264d79be946995e63e960426"},{"line_number":16,"context_line":"Change-Id: I4722b869033ad1bd357e36c4a258b6d3ea61f5d6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3850e4f5_3f5ef2cf","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":53},"in_reply_to":"c73dc256_21456485","updated":"2024-01-05 01:54:30.000000000","message":"that depends-on will not make any difference as project-config change are not taken until they are merged so with depends-on also CI will fail. We can recheck this change once project-config change is merged.","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"525eda940bb3d2be6d58e5ade1c3662ecff222a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bbcdfcea_c8ddb3c8","updated":"2024-01-05 14:34:41.000000000","message":"@Tony\n\n\u003e major tangent but the preferred format of Depends-On, is change URLs rather than Change-Id\n\nCould you explain this?  I tend to use a URL (didn\u0027t in this case because I prepared this series of patches offline before submitting to gerrit, and didn\u0027t have the URLs yet).  Some people I work with (though not on the infra team) insist that the Change-Id is better, and I\u0027d like to be able to say why URLs are preferred.","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"593fd394a907e38a2003b7528dadff30944c4553","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e27f237e_6d563446","updated":"2023-12-14 16:53:20.000000000","message":"I\u0027m pretty sure the failure here is because https://review.opendev.org/c/openstack/project-config/+/903260 has not merged yet.","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3630b51a200ce37516d21c597992d6d8305eafc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a09b225_560f5a7b","updated":"2024-01-05 14:29:32.000000000","message":"recheck - dependency has merged","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"afb28fa80cfdbed86c3092b0a7836cde2547503b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d33bf34f_e6b40bba","in_reply_to":"99a8e14c_f547b7af","updated":"2024-01-08 04:15:44.000000000","message":"Roughly 5-6 years ago using the Change-Id was deprecated in favor of URLs:\n\nhttps://lists.openstack.org/pipermail/openstack-dev/2018-January/126535.html","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fc1cbb8be8cd8cdee27530621e4a6dad976d2ad9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"99a8e14c_f547b7af","in_reply_to":"bbcdfcea_c8ddb3c8","updated":"2024-01-05 18:12:48.000000000","message":"yeah, not sure why URL is preferred than change-id. any documentation over that recommendation and reasoning?","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"acca53a8ca509253dce0ecee9a00a9f78e009206","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d56565f9_3190b07d","in_reply_to":"e27f237e_6d563446","updated":"2023-12-21 04:24:17.000000000","message":"yeah, zuul config change only in effect after merge and depends-on does not work in these case. Once project-config change merge you can recheck here.","commit_id":"6b5d5fef584c10bf8e5754800a2f8b8d956bb950"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0baeec7a85a1039c92e9399634e02ac7d10af69e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"63eff075_0d2287c7","updated":"2024-01-05 15:56:55.000000000","message":"We should probably merge the patches that stop cinderlib testing in master in cinder and cinder-tempest-plugin first:\n\n- https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/903262\n- https://review.opendev.org/c/openstack/cinder/+/903263","commit_id":"fdf6c7755a2bccc7ea5752712179e11b54a9ac34"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b1ea289c084a052e8e5ff5321ed7856eaddb4bd9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"91d1c80a_0957f458","updated":"2024-01-18 17:32:54.000000000","message":"All dependencies have merged.\nAll the code has been removed, except for the README file stating the deprecation of the project.\nGate jobs are noop now so it\u0027s passing.\nNot a good thing to see it go but given current state of development and use cases, it makes sense. LGTM.","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eb9b66045ea9457e1c5ed5c09ada25186928ed21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1d2a8c5c_b0c97f14","updated":"2024-01-05 16:04:38.000000000","message":"Explicitly added the cinder/cinder-tempest-plugin cinderlib test removal patches as dependencies.","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6a42939e2d4a754327f917567b1d46381fe46ee1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"87716f2e_074a3ab7","updated":"2024-01-17 17:17:42.000000000","message":"It would be helpful to the RDO project if some other stuff merges before we approve this patch:\n\n- https://github.com/openstack-k8s-operators/tcib/pull/119\n- https://review.rdoproject.org/r/c/rdoinfo/+/51258","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d4f86673a18d0d81c9231a356d5f26870fee7e82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ef9c899f_1ce743eb","updated":"2024-01-10 10:22:53.000000000","message":"There\u0027s also this other related patch:  https://review.opendev.org/c/openstack/tripleo-common/+/905204","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"68c3d6cf48a20ac377c0199b5ca7bba9ac604a7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0bd32182_80e30fff","updated":"2024-01-18 17:25:11.000000000","message":"This patch should be ok to merge now that the tcib PR and the RDO patch have merged","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"723ed4b71f8885b1c16ece50c7491b6301957346","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"871e534c_f42c6bfb","updated":"2024-01-05 18:11:29.000000000","message":"not sure it need deps on cinder-tempest-plugin or so but it is ok if you want to keep their gate unbroken. +1","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"945cb095954c2d42114a581f75c680592628732e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"30efcb1d_5b83cd48","updated":"2024-01-08 04:15:58.000000000","message":"recheck","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cf5ae23a8f532a0a55199968b784802068e731a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2b360a6f_7ac3f2e4","updated":"2024-01-16 22:55:26.000000000","message":"recheck - all dependencies have merged","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"}],"README.rst":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b1ea289c084a052e8e5ff5321ed7856eaddb4bd9","unresolved":true,"context_lines":[{"line_number":15,"context_line":"previous commit with \"git checkout HEAD^1\"."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"For any further questions, please email"},{"line_number":18,"context_line":"openstack-discuss@lists.openstack.org or join #openstack-dev on"},{"line_number":19,"context_line":"OFTC."}],"source_content_type":"text/x-rst","patch_set":3,"id":"d5d90db2_43af18bd","line":18,"range":{"start_line":18,"start_character":46,"end_line":18,"end_character":60},"updated":"2024-01-18 17:32:54.000000000","message":"or #openstack-cinder?","commit_id":"f165c6ff5e894d57b843380803caccecc96934a8"}]}