5 Sep
2025
5 Sep
'25
5:38 a.m.
On 2025-09-05 09:33:44 +0200 (+0200), Dmitriy Rabotyagov wrote:
I think it's only patch owner (or possible Gerrit Admins) who can remove WIP status. AFAIK it's not possible for the Core-Reviewers group to remove WIP from the patch as of today. [...]
With the current state of cinder's ACL[*] that's correct, but it's a permission that can be delegated easily. It might be a good idea to grant toggleWipState to the cinder-core group similar to what's done for neutron[**]. [*] https://opendev.org/openstack/project-config/src/commit/61c8a5d/gerrit/acls/... [**] https://opendev.org/openstack/project-config/src/commit/61c8a5d/gerrit/acls/... -- Jeremy Stanley