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/openstack/cinder.config
> [**] https://opendev.org/openstack/project-config/src/commit/61c8a5d/gerrit/acls/openstack/neutron.config#L13
> --
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