(adding devel because this might be interesting for others).

On 1 May 2018 at 21:16, Greg Sheremeta <gshereme@redhat.com> wrote:
How would I get this plugin added to ovirt-engine-dashboard and ovirt-engine-node* ?

Its already available for all projects in gerrit.ovirt.org.
 
And then how would I configure it? (Add myself as a default, at least)

Two ways:

With the GUI:
-----------------

In Gerrit UI go to:

Admin -> Projects -> (Old UI) -> (Your project) -> Reviewers (from top bar)

Fill in filter (A file glob pattern - can be '*') and reviewer (An email address ora gerrit group ) and click "Add".

Don't forget to switch back to new UI from the bottom right link.
 

From the command line
-------------------------------
In a Git clone of the relevant project do:

# git fetch origin refs/meta/config:meta-config
# git checkout meta-config

Then you can edit the 'reviewers.config' file (Make it if it isn't there) and put in something like the following:

    [filter "*"]
        reviewer = jenkins-maintainers
        reviewer = dron@redhat.com

Once you're done with it commit the change and push it with:

# git push -u origin meta-config refs/meta/config

Since we have '-u' there, the next time around you change the 'meta-config' branch you can just do:

# git push -u origin

And that's about it.

One thing to note - every project has a '<project-name>-maintainers' group, so its useful to add it it default.


---------- Forwarded message ----------
From: Roy Golan <rgolan@redhat.com>
Date: Mon, Jun 19, 2017 at 7:17 AM
Subject: Re: [ovirt-devel] [ENGINE][ACTION_NEEDED] - default maintainers, per path
To: devel <devel@ovirt.org>


The patch is now merged and the it should start working immediately. For anyone who is listed as a default reviewer I suggest to test it with a test patch by some other submitter. (except for those who used groups of course)

Thanks everyone for the cooperation on this.


On Mon, May 29, 2017 at 2:24 PM Roy Golan <rgolan@redhat.com> wrote:
Hi all,

Some time ago the infra team enabled gerrit default reviewer plugin which you probably noticed if you logged in to gerrit.

What we can do is for example set the stable branch maintainers per branch automatically:

    [filter "branch:ovirt-engine-4.1"]
        reviewer = stableBranchMaintainersGroup

Put people based on path:

   [filter "branch:master file:^frontend/.*"]
       reviewer = j.r.r.t@shire.me


Action Needed:
Nominate yourself, or others, by amending this patch online [1]. Once this patch is fully acked and agreed we will merge it. If something can't get consensus we will defer it from the patch till maintainers agree.

[1] https://gerrit.ovirt.org/#/c/77488/



 

_______________________________________________
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel



--

GREG SHEREMETA

SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX

Red Hat NA

gshereme@redhat.com    IRC: gshereme


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




--
Barak Korren
RHV DevOps team , RHCE, RHCi
Red Hat EMEA
redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted