[Bugs] [Bug 1350477] Test to check if the maintainer reviewed the patch

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1350477

Nigel Babu <nigelb at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |Triaged
         Whiteboard|Triaged                     |



--- Comment #3 from Nigel Babu <nigelb at redhat.com> ---
Thanks Jeff for these details. This is precisely why I filed a bug:

1) For now, we'll start with A *or* B to review the code. As we figure out how
many special cases we have, we'll deal with figuring out how to handle the
*and* special case. I'm making an assumption that it can temporarily be dealt
with as something reviewer A can trigger "Make sure you also get this reviewed
by reviewer B" and then you can have the Maintainer +1.

2) Overrides, I've already thought of. My idea was to have the ultimate power
in the release manager's hands. Since it's their call on what goes into each
branch. But if we want to define project-level architects (superreviewers?),
I'm happy to roll with that.

3) This is something we'll need to refine as we deploy. Either we make these
components owned by whoever can review tiering or geo-rep or we make a few of
the geo-repo and tiering people superreviewers.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Ii3E3bsTD3&a=cc_unsubscribe