On Mon, Feb 25, 2019 at 9:08 PM Dan Kenigsberg <dan...@redhat.com> wrote:
> On Mon, 25 Feb 2019, 20:52 Nir Soffer, <nsof...@redhat.com> wrote: > >> Hi all, >> >> I learned recently that we have these groups in the project: >> vdsm-maintainers: https://gerrit.ovirt.org/#/admin/groups/15,members >> vdsm-master-maintainers: >> https://gerrit.ovirt.org/#/admin/groups/106,members >> >> And we have this complicated access control list: >> https://gerrit.ovirt.org/#/admin/projects/vdsm,access >> >> 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): >> https://gerrit.ovirt.org/c/97738/ >> >> 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 https://gerrit.ovirt.org/#/admin/groups/106,members Should be able to submit now. Nir
_______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/MFKPK75ETYTBI7NQRKQJZTNTV24WWVGR/
