On Mon, Feb 25, 2019 at 9:08 PM Dan Kenigsberg <danken@redhat.com> wrote:
On Mon, 25 Feb 2019, 20:52 Nir Soffer, <nsoffer@redhat.com> wrote:
Hi all,

I learned recently that we have these groups in the project:

And we have this complicated access control list:

Developers in vdsm-master-maintainers can approve patches, but cannot submit. 
I find this strange and more complicated than needed. I think we can trust people
in the vdsm-master-maintainers group to do the right thing.

I sent this patch to simplify the ACL (with help of Gerrit UI):

Thoughts?

I don't recall the origin of this differentiation, and I'd rather simplify it.  It makes simple sense that vdsm-master-maintainers are in full control of the master branch of vdsm.

+1 from me.

Barak merged the patch so people in
Should be able to submit now.

Nir