)]}'
{"gerrit/acls/openstack/openstacksdk.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1f8b843160c520f280257b6df94f790f31592adf","unresolved":true,"context_lines":[{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group openstacksdk-stable-maint"},{"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":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"}],"source_content_type":"text/x-ini","patch_set":5,"id":"f25bfa92_6b4de801","line":28,"updated":"2022-10-07 16:54:31.000000000","message":"I mentioned this on IRC in the OpenDev meeting. It would be great if we could come up with a label name that is a bit more descriptive as this has a high probability of being cargo culted. But I\u0027m not coming up with anything better.","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"557ddb24de8bd532305ad58f4d188c910e5d28c1","unresolved":false,"context_lines":[{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group openstacksdk-stable-maint"},{"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":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"}],"source_content_type":"text/x-ini","patch_set":5,"id":"272fd024_f3d82cc9","line":28,"in_reply_to":"f25bfa92_6b4de801","updated":"2022-11-23 08:47:40.000000000","message":"Maybe \"Trigger-Post-Review-Pipeline\"? Or do you think that that would be too long? But in the end educating core reviewers will be the only proper solution I guess.","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1f8b843160c520f280257b6df94f790f31592adf","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":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":5,"id":"d4d61f22_b147b6da","line":29,"updated":"2022-10-07 16:54:31.000000000","message":"Considering the intent of this is to trigger access to secrets after review I\u0027m on the fence if we should copy this score across trivial updates. I think the two risks here are misunderstanding the cases gerrit treats as trivial and allowing something unexpected through and/or someone potentially rebasing in a loop to trigger things over and over again.","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"557ddb24de8bd532305ad58f4d188c910e5d28c1","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":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"},{"line_number":32,"context_line":"value \u003d 0 Abstain"}],"source_content_type":"text/x-ini","patch_set":5,"id":"1dd0a8e8_53608d4d","line":29,"in_reply_to":"d4d61f22_b147b6da","updated":"2022-11-23 08:47:40.000000000","message":"Done","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1f8b843160c520f280257b6df94f790f31592adf","unresolved":true,"context_lines":[{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"},{"line_number":32,"context_line":"value \u003d 0 Abstain"},{"line_number":33,"context_line":"value \u003d +1 Approved by Core Reviewer"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[receive]"},{"line_number":36,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":5,"id":"e9d1e084_cf75e022","line":33,"updated":"2022-10-07 16:54:31.000000000","message":"I think we should try to be more specific in the text description of this label. Maybe:\n\n  Approved for Zuul Post Review Pipeline","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"557ddb24de8bd532305ad58f4d188c910e5d28c1","unresolved":false,"context_lines":[{"line_number":30,"context_line":"defaultValue \u003d 0"},{"line_number":31,"context_line":"function \u003d NoOp"},{"line_number":32,"context_line":"value \u003d 0 Abstain"},{"line_number":33,"context_line":"value \u003d +1 Approved by Core Reviewer"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[receive]"},{"line_number":36,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":5,"id":"1a077a41_b645ca95","line":33,"in_reply_to":"e9d1e084_cf75e022","updated":"2022-11-23 08:47:40.000000000","message":"Done","commit_id":"d6badfd4c54910b0e94fe2f769e0964ff0692512"}]}