Re: [ovirt-devel] Gerrit, only infra can remove reviewers

From: "David Caro" <dcaroest(a)redhat.com&gt; To: "Moti Asayag" <masayag(a)redhat.com&gt; Cc: devel(a)ovirt.org, infra(a)ovirt.org Sent: Monday, April 13, 2015 12:47:23 PM Subject: Re: [ovirt-devel] Gerrit, only infra can remove reviewers On 04/13, Moti Asayag wrote: > > > ----- Original Message ----- > > From: "David Caro" <dcaroest(a)redhat.com&gt; > > To: devel(a)ovirt.org, infra(a)ovirt.org > > Sent: Wednesday, April 8, 2015 7:24:42 PM > > Subject: [ovirt-devel] Gerrit, only infra can remove reviewers > > > > > > Effective now, only infra team members can remove reviewers from engine > > master > > and vdsm master. > > Can't you limit the list of removed reviewers only for the relevant > reviewer > list ? specifically for "gerrit-hooks" and "oVirt Jenkins CI Server" ? > No, that's not an option right now (unsupported) > I'd like to have the ability to remove unintended reviewer from a patch > (i.e. > if was mistakenly added ). > > Also, can a user remove himself from the reviewer list of a given patch ? Yes, you should, have you tried?

> > > That means that you'll have to poke one of us each time you > > want to prevent jenkins (or anyone) from blocking any patches. > > > > > > Enjoy! > > > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R&D > > > > Tel.: +420 532 294 605 > > Email: dcaro(a)redhat.com > > Web: www.redhat.com > > RHT Global #: 82-62605 > > > > _______________________________________________ > > Devel mailing list > > Devel(a)ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R&D Tel.: +420 532 294 605 Email: dcaro(a)redhat.com Web: www.redhat.com RHT Global #: 82-62605