Can ovirt_engine_find_bugs_gerrit be run against changed files, instead of all?

Fabian Deutsch fabiand at redhat.com
Tue Jun 11 18:21:21 UTC 2013


Am Dienstag, den 11.06.2013, 09:48 -0400 schrieb Allon Mureinik:
> 
> ----- Original Message -----
> > From: "Eyal Edri" <eedri at redhat.com>
> > To: "Fabian Deutsch" <fabiand at redhat.com>, "Yair Zaslavsky" <yzaslavs at redhat.com>, "Allon Mureinik"
> > <amureini at redhat.com>, "Juan Hernandez" <jhernand at redhat.com>
> > Cc: infra at ovirt.org
> > Sent: Tuesday, June 11, 2013 4:42:51 PM
> > Subject: Re: Can ovirt_engine_find_bugs_gerrit be run against changed files,	instead of all?
> > 
> > right now it runs on **/java/** (i.e patches that changed java files).
> > 
> > i'm not sure jenkins gerrit job can run findbugs (actually it's maven
> > findbugs plugin) only on changed files/module.
> > 
> > if you know how to do it in jenkins/gerrit/maven i'd be happy to apply the
> > change.
> > 
> > allon/yair/juan - do know if that's possible?
> I'm not aware of such a feature, nor could I find a decent way of doing it, sorry.

Is it worth putting some efforts in it? I mean, we speak about a runtime
of ~1hr. For each patch. This really takes resources.

- fabian

> > 
> > Eyal.
> > 
> > ----- Original Message -----
> > > From: "Fabian Deutsch" <fabiand at redhat.com>
> > > To: infra at ovirt.org
> > > Sent: Tuesday, June 11, 2013 4:32:13 PM
> > > Subject: Can ovirt_engine_find_bugs_gerrit be run against changed files,
> > > 	instead of all?
> > > 
> > > Hey,
> > > I noted that ovirt_engine_find_bugs_gerrit eat's a lot of our builders
> > > resources. I wondered if the job can be changed to only check changed
> > > files, instead of the whole tree?
> > > With git you can do
> > >   git diff --name-only HEAD~1..HEAD
> > > to retrieve the changed files between two versions.
> > > 
> > > Greetings
> > > fabian
> > > 
> > > _______________________________________________
> > > Infra mailing list
> > > Infra at ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/infra
> > > 
> > 





More information about the Infra mailing list