)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"d7ce5452580a87a18e5fcc2e2df579821d63d89c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d1330e31_cd7f80e3","updated":"2022-11-07 20:53:40.000000000","message":"I\u0027d prefer this not merge until there\u0027s at least a +1 (either in gerrit or verbally in IRC) from another Ironic core :). Don\u0027t wanna go unilaterally changing the meaning of core for Ironic","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"59be2497a42f454314594c0aa082de50b94db864","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"45ee8952_3ab7694f","updated":"2022-11-21 14:26:50.000000000","message":"Don\u0027t we have separate permissions for sushy as well?","commit_id":"dc6b5ace5a7d19a18d996230a04efed2f9d9dfd1"}],"gerrit/acls/openstack/ironic-inspector.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"67843994f14c3054e7cce4d41448c14f37338dde","unresolved":true,"context_lines":[{"line_number":25,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":27,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":28,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":31,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":1,"id":"572e9e97_6abde6d7","line":28,"updated":"2022-11-07 21:11:31.000000000","message":"See comments in the first file.","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"c5768a9760b371a55cba052a2a471840cb0e186b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":27,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":28,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":31,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":1,"id":"74fe9ff1_670c665b","line":28,"in_reply_to":"572e9e97_6abde6d7","updated":"2022-11-16 21:34:30.000000000","message":"Done","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"67843994f14c3054e7cce4d41448c14f37338dde","unresolved":true,"context_lines":[{"line_number":41,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":42,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":43,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":44,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"[receive]"},{"line_number":47,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"360bfe5a_c69a8d46","line":44,"updated":"2022-11-07 21:11:31.000000000","message":"See comments in the first file.","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"c5768a9760b371a55cba052a2a471840cb0e186b","unresolved":false,"context_lines":[{"line_number":41,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":42,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":43,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":44,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"[receive]"},{"line_number":47,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"b02c3e55_e48f9600","line":44,"in_reply_to":"360bfe5a_c69a8d46","updated":"2022-11-16 21:34:30.000000000","message":"Done","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"}],"gerrit/acls/openstack/ironic.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"67843994f14c3054e7cce4d41448c14f37338dde","unresolved":true,"context_lines":[{"line_number":23,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-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":"[access \"refs/heads/stable/*\"]"},{"line_number":29,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":1,"id":"b765c070_acb1410b","line":26,"updated":"2022-11-07 21:11:31.000000000","message":"You may want to add the permission for stable maint groups as well. See comments on the octavia change: https://review.opendev.org/c/openstack/project-config/+/863934","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"4197ddfe42e07e083166bcd42f0e26144236968c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-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":"[access \"refs/heads/stable/*\"]"},{"line_number":29,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":1,"id":"bacef017_d5bbaccc","line":26,"in_reply_to":"b765c070_acb1410b","updated":"2022-11-16 21:34:46.000000000","message":"Done","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"67843994f14c3054e7cce4d41448c14f37338dde","unresolved":true,"context_lines":[{"line_number":39,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":40,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":41,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":42,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":45,"context_line":"copyAllScoresIfNoCodeChange \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"bc30f5e6_d4523730","line":42,"updated":"2022-11-07 21:11:31.000000000","message":"See above.","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"c5768a9760b371a55cba052a2a471840cb0e186b","unresolved":false,"context_lines":[{"line_number":39,"context_line":"label-Workflow \u003d -1..+0 group Change Owner"},{"line_number":40,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":41,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":42,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":45,"context_line":"copyAllScoresIfNoCodeChange \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"f38cf85d_eafb13c0","line":42,"in_reply_to":"bc30f5e6_d4523730","updated":"2022-11-16 21:34:30.000000000","message":"Done","commit_id":"17e9d3804780e17aec17e92917f22cb36a0a9cf3"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"9ed0d5359f30bd436fc563a0b0ab939f56c05346","unresolved":false,"context_lines":[{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":"toggleWipState \u003d group stable-maint-core"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":30,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":2,"id":"02106689_4ede2cb1","line":27,"updated":"2022-11-08 00:23:37.000000000","message":"Did you want to include ironic-stable-maint too?","commit_id":"dd0fcc04126409d093c7658e62c2abdb079d4731"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1a984539d3b8fc7e031035e77a37c9560e7944b0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":"toggleWipState \u003d group stable-maint-core"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":30,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":2,"id":"b2eec2d8_bced9478","line":27,"updated":"2022-11-08 15:39:30.000000000","message":"Yes, since it\u0027s not covered by exclusiveGroupPermissions the parent ref pattern applies, which grants toggleWipState for ironic-core on stable branch changes already. I was merely wondering why stable-maint-core would be added here but not ironic-stable-maint, when both have approval rights on stable branch changes.","commit_id":"dd0fcc04126409d093c7658e62c2abdb079d4731"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"415192b943cbd764a081b3ad99618c07328f4b40","unresolved":false,"context_lines":[{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":"toggleWipState \u003d group stable-maint-core"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":30,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":2,"id":"ab79621e_5ea48a83","line":27,"in_reply_to":"02106689_4ede2cb1","updated":"2022-11-08 10:13:51.000000000","message":"toggleWipState is not in exclusiveGroupPermissions, so it\u0027s probably fine? Or should it rather be added to exclusiveGroupPermissions and then ironic-stable-maint added here?","commit_id":"dd0fcc04126409d093c7658e62c2abdb079d4731"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"c5768a9760b371a55cba052a2a471840cb0e186b","unresolved":false,"context_lines":[{"line_number":24,"context_line":"label-Workflow \u003d -1..+1 group Project Bootstrappers"},{"line_number":25,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":26,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":27,"context_line":"toggleWipState \u003d group stable-maint-core"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"[access \"refs/heads/stable/*\"]"},{"line_number":30,"context_line":"abandon \u003d group Change Owner"}],"source_content_type":"text/x-ini","patch_set":2,"id":"d799a397_cf0c48ce","line":27,"in_reply_to":"b2eec2d8_bced9478","updated":"2022-11-16 21:34:30.000000000","message":"got it, thanks","commit_id":"dd0fcc04126409d093c7658e62c2abdb079d4731"}]}