As discussed in today's community meeting, we need a way to "dig ourselves out of the hole" quickly when work is blocked by sporadically failing tests. Specifically, there should be a way to exempt patches from testing which (a) revert a previous commit or (b) mark a test as bad. Gerrit has permissions that allow specified users to bypass tests by pushing to refs/heads/xxx instead of refs/for/xxx. This permission should be enabled for project architects (and probably not for others). If we can further extend or limit this ability by recognizing reverts and test markings for what they are, that's great but not immediately necessary.
Should we not have a discussion or at least a notification about this on the maintainers list? I think permissions to push changes without reviews can be done by release managers (because they need to be able to push tags?), but fine-tuning the permissions sounds like a good idea.
Just some guidelines or recommendations on which changes to push and 'rules' to notify relevant persons and track progress would be good.
Yes, there should be a policy discussion about this, but it's moot if the technical capability isn't implemented. Therefore, I think the two efforts can proceed in parallel.
The technical capability is now done. There's a team called gluster-plumbers who have permission to push to any branch directly. It has Vijay, Jeff, and Shyam. As far as infra is concerned this is fixed. If the policy dictates changes to this, please file a bug.