)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"72afe851ea5045dfcb565f4fe29e76bfdb14ae90","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bd4c6b39_b7d934be","updated":"2022-04-26 17:28:55.000000000","message":"Cinder hash LGTM, however i think it\u0027s still worth mentioning in the commit message as to why we are doing a major version bump and not releasing other branches. Usually the reason for not releasing is \"no functional changes in branch\" like here[1] but for major bumps, it helps maintain the history. Having said that, it shouldn\u0027t be a blocker but just a suggestion. LGTM.\n\n[1] https://review.opendev.org/c/openstack/releases/+/809892 ","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"93a76e91609ec82a8146cc65072c082ba2a95436","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5f41e0b2_f4c7682f","updated":"2022-04-26 17:11:44.000000000","message":"Thanks Jon, LGTM:\n* MINOR version bump looks OK to me (reno suggests MINOR bump due to patch 1034c9aec (Add warning message about slow volume backend) but I think what really worth the bump is d7bb194d0 (Fix: Schema validation for attachment create API) patch)\n* no issues found in test job logs\n\nI\u0027ll give +1 for now until PTL/release liaison approve arrives.","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b3cfc1128c030acf654c70730c9691951da0025c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7032ecc2_7590710c","in_reply_to":"5f41e0b2_f4c7682f","updated":"2022-04-26 17:31:50.000000000","message":"Jon is our new release liaison, just to confirm, if the liaison updates his/her name here[1] and propose a patch, it should automatically trigger the PTL-Approved or is it still only triggered by PTL\u0027s vote?\n\n[1] https://opendev.org/openstack/releases/src/branch/master/data/release_liaisons.yaml","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9b6501666966278915699cd1342e9d7f90a99dca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cb017e04_e96b248a","in_reply_to":"7032ecc2_7590710c","updated":"2022-04-26 17:43:53.000000000","message":"yes, if the release_liaisons.yaml is updated with the name/email then the PTL-Approved flag will be triggered even when the patch is pushed by the release liaison.","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9b6501666966278915699cd1342e9d7f90a99dca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"67219c5f_de1ce115","in_reply_to":"bd4c6b39_b7d934be","updated":"2022-04-26 17:43:53.000000000","message":"nit: \u0027minor\u0027 version bump o:) but yes, it\u0027s usually a good practice to add some info about the release as Rajat wrote (which is what i mentioned in my previous comment)","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"cbf297b1ff2663c41c6e1d756cf40a85adb1290a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"518261a4_eff0bc6e","in_reply_to":"cb017e04_e96b248a","updated":"2022-04-26 19:14:52.000000000","message":"I have proposed this change here https://review.opendev.org/c/openstack/releases/+/839430","commit_id":"b577fc0dd26194741efe74961ae10843285349c0"}]}