That was always the case, master maintainers should have +2 on the stable branches without merge rights, this helps me distinguished between:

1. Who is actually a maintainer from the ones acking the patch

2. If a maintainer meant we're good to go or gave his review yet would like another reviewer to have a look

On Tue, Dec 20, 2016 at 5:12 PM, Eyal Edri <eedri@redhat.com> wrote:

I wasn't aware we were allowing +2 permissions without merge rights on stable branch,
If this should be supported then we need to update the permissions, so all master branch maintainers need +2 (w/o merge) to stable branches?

On Tue, Dec 20, 2016 at 4:58 PM, Tal Nisan <tnisan@redhat.com> wrote:
Master

On Tue, Dec 20, 2016 at 4:53 PM, Eyal Edri <eedri@redhat.com> wrote:
stable branch maintainers or master branch maintainers?

On Dec 20, 2016 4:40 PM, "Tal Nisan" <tnisan@redhat.com> wrote:
Heard it from a couple of guys, yesterday it appeared to be working fine, today it's not

_______________________________________________
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

--
Eyal Edri
Associate Manager
RHV DevOps
EMEA ENG Virtualization R&D
Red Hat Israel

phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)