On Fri, Sep 5, 2025 at 6:38 AM Jeremy Stanley <fungi@yuggoth.org> wrote:
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/... --
Neat suggestion, I've had this issue in the past with manila repos; I think it's worth doing this universally across all openstack repos: https://review.opendev.org/c/openstack/project-config/+/959904
Jeremy Stanley