)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"25c55740fcded1a10cdef5e9b0b2dff00f38b79b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fb8616ad_3da3a8f9","updated":"2023-04-13 17:33:32.000000000","message":"I hesitate to -1 because I may be over concerned about my inline comment. But I think we should come up with an answer to that concern and record it in this change one way or another.","commit_id":"7f90fec609c4d7e4b1145d81e806c00f0c66f61e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d5d1b8ee014b25a8df1e7f13eca06ae6ddfaa61d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dadec3be_79bd6601","updated":"2023-04-20 01:52:51.000000000","message":"We had some discussion about doing this before the larger change to indent everything with tabs; I think it\u0027s probably better to merge this first so that we have in the project history gerrit updating the ACL, then us bringing our ACL up to date with that, even if that has tab differences.","commit_id":"6f70d8914590a290adf0c3947a96940fb6cb1b66"}],"gerrit/acls/openstack/openstacksdk.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"25c55740fcded1a10cdef5e9b0b2dff00f38b79b","unresolved":true,"context_lines":[{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[label \"Allow-Post-Review\"]"},{"line_number":29,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoBlock"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":1,"id":"5a978794_a5ca13c3","line":29,"updated":"2023-04-13 17:33:32.000000000","message":"This label is probably another one like workflow where not having it be copied is desirable. This label is used to trigger post review but pre merge builds in zuul. To avoid potential git tree hacks to retrigger things without core reviewer involvement maybe we should leave this one out?","commit_id":"7f90fec609c4d7e4b1145d81e806c00f0c66f61e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"448ab4cf131f9fd9add13dcfe6d05ec3780a2314","unresolved":true,"context_lines":[{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[label \"Allow-Post-Review\"]"},{"line_number":29,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoBlock"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":1,"id":"64d09439_f21bfd3f","line":29,"in_reply_to":"5a978794_a5ca13c3","updated":"2023-04-14 02:06:14.000000000","message":"I agree; but I think that does modify the behaviour from the current and prior behaviour -- even though it wasn\u0027t intended.\n\nI will make a follow-on with updates we can consider individually, and we can hold this and merge everything together.","commit_id":"7f90fec609c4d7e4b1145d81e806c00f0c66f61e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"4ab3de87c3d1687c39817f799357fd738de16cf4","unresolved":false,"context_lines":[{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[label \"Allow-Post-Review\"]"},{"line_number":29,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoBlock"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":1,"id":"db0d5960_d1154260","line":29,"in_reply_to":"64d09439_f21bfd3f","updated":"2023-04-19 05:06:04.000000000","message":"Done with https://review.opendev.org/c/openstack/project-config/+/880792","commit_id":"7f90fec609c4d7e4b1145d81e806c00f0c66f61e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"42290affd2561f307b281e456ca15a0b6931a9f1","unresolved":false,"context_lines":[{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[label \"Allow-Post-Review\"]"},{"line_number":29,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoBlock"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":1,"id":"a6e6ef91_ac99bd6e","line":29,"in_reply_to":"db0d5960_d1154260","updated":"2023-04-19 05:26:11.000000000","message":"Oh I should note I looked through all the others; I don\u0027t think any of the others this really matters for.","commit_id":"7f90fec609c4d7e4b1145d81e806c00f0c66f61e"}]}