From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; To: "Moti Asayag" <masayag(a)redhat.com&gt; Cc: "engine-devel" <engine-devel(a)ovirt.org&gt; Sent: Thursday, January 2, 2014 11:54:28 PM Subject: Re: [Engine-devel] Strange findbugs warning for a phantom class ----- Original Message ----- > From: "Moti Asayag" <masayag(a)redhat.com&gt; > To: "engine-devel" <engine-devel(a)ovirt.org&gt; > Sent: Thursday, January 2, 2014 7:40:47 PM > Subject: [Engine-devel] Strange findbugs warning for a phantom class > > It seems that findbugs complains [1] about a class which never > existed in ovirt-engine (a job which is run per pushed patch): > > org.ovirt.engine.core.authentication.tls.TlsSocketFactoryN I can only say what is the source of this class - the AAA rewrite Juan and I are working on. > > This produces too much noise in gerrit. I'm not sure what is the > origin of the class, but i was under the assumption that findbugs > should be run for each patch on a clean env, without leftovers > from job on other patches. > > Any idea how to clear this class/error ? +1 on Moti's question - I think I saw this behavior last week as well. > > [1] > http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/3269/fi... > > Thanks, > Moti > > > > _______________________________________________ > Engine-devel mailing list > Engine-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/engine-devel > _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

The obvious is to enable wipe workspace per run, but i fear that will be add much more run time per patch and that jenkins won't be able to handle the queue of jobs, but it's worth a try. i will enable it for now, and we'll monitor the queue size.

eyal. ----- Original Message ----- > From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; > To: "Moti Asayag" <masayag(a)redhat.com&gt; > Cc: "engine-devel" ovirt.org> > Sent: Friday, January 3, 2014 7:39:45 AM > Subject: Re: [Engine-devel] Strange findbugs warning for a phantom class > > I saw this today as well - findbugs showed new warnings for jsonrpc java > client code , but my patches are not rebased against that work (the jsonrpc > code is not merged to master). > > > ----- Original Message ----- >> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; >> To: "Moti Asayag" <masayag(a)redhat.com&gt; >> Cc: "engine-devel" ovirt.org> >> Sent: Thursday, January 2, 2014 11:54:28 PM >> Subject: Re: [Engine-devel] Strange findbugs warning for a phantom class >> >> >> >> ----- Original Message ----- >>> From: "Moti Asayag" <masayag(a)redhat.com&gt; >>> To: "engine-devel" ovirt.org> >>> Sent: Thursday, January 2, 2014 7:40:47 PM >>> Subject: [Engine-devel] Strange findbugs warning for a phantom class >>> >>> It seems that findbugs complains [1] about a class which never >>> existed in ovirt-engine (a job which is run per pushed patch): >>> >>> org.ovirt.engine.core.authentication.tls.TlsSocketFactoryN >> >> I can only say what is the source of this class - the AAA rewrite Juan and >> I >> are working on. >>> >>> This produces too much noise in gerrit. I'm not sure what is the >>> origin of the class, but i was under the assumption that findbugs >>> should be run for each patch on a clean env, without leftovers >>> from job on other patches. >>> >>> Any idea how to clear this class/error ? >> >> +1 on Moti's question - I think I saw this behavior last week as well. >> >>> >>> [1] >>> http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/3269/fi... >>> >>> Thanks, >>> Moti >>> >>> >>> >>> _______________________________________________ >>> Engine-devel mailing list >>> Engine-devel(a)ovirt.org >>> http://lists.ovirt.org/mailman/listinfo/engine-devel >>> >> _______________________________________________ >> Engine-devel mailing list >> Engine-devel(a)ovirt.org >> http://lists.ovirt.org/mailman/listinfo/engine-devel >> > _______________________________________________ > Engine-devel mailing list > Engine-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/engine-devel > _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

not sure, the results are created from the src code, and if the same job checksout diff patches to the same place that can create the fp errors we see now.. deleting the results will just mean they will be created again the same...

On Jan 3, 2014 9:42 AM, Itamar Heim <iheim(a)redhat.com&gt; wrote: > > On 01/03/2014 08:58 AM, Eyal Edri wrote: >> The obvious is toOn 01/03/2014 08:58 AM, Eyal Edri wrote: > The obvious is to enable wipe workspace per run, but i fear that will be > add much more run time per patch and that jenkins won't be able to handle the > queue of jobs, but it's worth a try. > > i will enable it for now, and we'll monitor the queue size. I don't think you need to clean the entire workspace, just the findbugs results, which are probably showing results of a class no longer there. > > eyal. > > ----- Original Message ----- >> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; >> To: "Moti Asayag" <masayag(a)redhat.com&gt; >> Cc: "engine-devel" ovirt.org> >> Sent: Friday, January 3, 2014 7:39:45 AM >> Subject: Re: [Engine-devel] Strange findbugs warning for a phantom class >> >> I saw this today as well - findbugs showed new warnings for jsonrpc java >> client code , but my patches are not rebased against that work (the jsonrpc >> code is not merged to master). >> >> >> ----- Original Message ----- >>> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; >>> To: "Moti Asayag" <masayag(a)redhat.com&gt; >>> Cc: "engine-devel" ovirt.org> >>> Sent: Thursday, January 2, 2014 11:54:28 PM >>> Subject: Re: [Engine-devel] Strange findbugs warning for a phantom class >>> >>> >>> >>> ----- Original Message ----- >>>> From: "Moti Asayag" <masayag(a)redhat.com&gt; >>>> To: "engine-devel" ovirt.org> >>>> Sent: Thursday, January 2, 2014 7:40:47 PM >>>> Subject: [Engine-devel] Strange findbugs warning for a phantom class >>>> >>>> It seems that findbugs complains [1] about a class which never >>>> existed in ovirt-engine (a job which is run per pushed patch): >>>> >>>> org.ovirt.engine.core.authentication.tls.TlsSocketFactoryN >>> >>> I can only say what is the source of this class - the AAA rewrite Juan and >>> I >>> are working on. >>>> >>>> This produces too much noise in gerrit. I'm not sure what is the >>>> origin of the class, but i was under the assumption that findbugs >>>> should be run for each patch on a clean env, without leftovers >>>> from job on other patches. >>>> >>>> Any idea how to clear this class/error ? >>> >>> +1 on Moti's question - I think I saw this behavior last week as well. >>> >>>> >>>> [1] >>>> http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/3269/fi... >>>> >>>> Thanks, >>>> Moti >>>> >>>> >>>> >>>> _______________________________________________ >>>> Engine-devel mailing list >>>> Engine-devel(a)ovirt.org >>>> http://lists.ovirt.org/mailman/listinfo/engine-devel >>>> >>> _______________________________________________ >>> Engine-devel mailing list >>> Engine-devel(a)ovirt.org >>> http://lists.ovirt.org/mailman/listinfo/engine-devel >>> >> _______________________________________________ >> Engine-devel mailing list >> Engine-devel(a)ovirt.org >> http://lists.ovirt.org/mailman/listinfo/engine-devel >> > _______________________________________________ > Engine-devel mailing list > Engine-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/engine-devel >