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