----- Original Message -----
From: "Fabian Deutsch" <fabiand@redhat.com> To: "Allon Mureinik" <amureini@redhat.com> Cc: "Eyal Edri" <eedri@redhat.com>, "Yair Zaslavsky" <yzaslavs@redhat.com>, "Juan Hernandez" <jhernand@redhat.com>, infra@ovirt.org Sent: Tuesday, June 11, 2013 9:21:21 PM Subject: Re: Can ovirt_engine_find_bugs_gerrit be run against changed files, instead of all?
Am Dienstag, den 11.06.2013, 09:48 -0400 schrieb Allon Mureinik:
----- Original Message -----
From: "Eyal Edri" <eedri@redhat.com> To: "Fabian Deutsch" <fabiand@redhat.com>, "Yair Zaslavsky" <yzaslavs@redhat.com>, "Allon Mureinik" <amureini@redhat.com>, "Juan Hernandez" <jhernand@redhat.com> Cc: infra@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.
it's not about effort, it's about resources. and we'll have them once we migrate the jenkins slaves to the new rackspace servers. (currently pending request for a vm + storage to continue).
- fabian
Eyal.
----- Original Message -----
From: "Fabian Deutsch" <fabiand@redhat.com> To: infra@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@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra