8:25 a.m.
Hi all,
Up until now the jenkins jobs on the gerrit patches included a findbugs job, that failed
only in case of a warning of level NORMAL or higher.
Now we update this threshold to fail on any findbugs warning, including LOW ones.
Please make sure to rebase your current patches and check that the findbugs job finish
successfully.
It will probably fail without a rebase, as the last patches clearing the warnings were
merged a few hours ago.
Kodus to everyone involved in clearing all the LOW level warnings... mostly Allon and
Alissa, but others helped as well! :-)
Cheers,
Oved
10:25 a.m.
On 01/01/2014 04:25 PM, Oved Ourfalli wrote:
Hi all,
Up until now the jenkins jobs on the gerrit patches included a findbugs job, that failed
only in case of a warning of level NORMAL or higher.
Now we update this threshold to fail on any findbugs warning, including LOW ones.
Please make sure to rebase your current patches and check that the findbugs job finish
successfully.
It will probably fail without a rebase, as the last patches clearing the warnings were
merged a few hours ago.
Kodus to everyone involved in clearing all the LOW level warnings... mostly Allon and
Alissa, but others helped as well! :-)
Cheers,
Oved
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel
indeed - a BIG thank you for cleaning this - we finally have at least
java naming conventions and various other nits fixed.
lets keep this clean - and that's only possible when its at a clear
round 0. if something is a false positive, we can add a specific ignore
for it (which was done for some issues).
Thanks!
Itamar
4:08 p.m.
----- Original Message -----
From: "Oved Ourfalli" <ovedo(a)redhat.com>
To: "engine-devel" <engine-devel(a)ovirt.org>, "Allon Mureinik"
<amureini(a)redhat.com>, "Alissa Bonas"
<abonas(a)redhat.com>
Sent: Wednesday, January 1, 2014 4:25:50 PM
Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
Hi all,
Up until now the jenkins jobs on the gerrit patches included a findbugs job,
that failed only in case of a warning of level NORMAL or higher.
Now we update this threshold to fail on any findbugs warning, including LOW
ones.
Could you provide instructions of running findbugs locally and how to evaluate
the result the same as done by the jenkins job ?
Please make sure to rebase your current patches and check that the
findbugs
job finish successfully.
It will probably fail without a rebase, as the last patches clearing the
warnings were merged a few hours ago.
Kodus to everyone involved in clearing all the LOW level warnings... mostly
Allon and Alissa, but others helped as well! :-)
Well done!
Cheers,
Oved
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel
3:12 a.m.
----- Original Message -----
From: "Moti Asayag" <masayag(a)redhat.com>
To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>
Cc: "engine-devel" <engine-devel(a)ovirt.org>
Sent: Thursday, January 2, 2014 12:08:43 AM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Oved Ourfalli" <ovedo(a)redhat.com>
> To: "engine-devel" <engine-devel(a)ovirt.org>, "Allon
Mureinik"
> <amureini(a)redhat.com>, "Alissa Bonas"
> <abonas(a)redhat.com>
> Sent: Wednesday, January 1, 2014 4:25:50 PM
> Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
>
> Hi all,
>
> Up until now the jenkins jobs on the gerrit patches included a findbugs
> job,
> that failed only in case of a warning of level NORMAL or higher.
> Now we update this threshold to fail on any findbugs warning, including LOW
> ones.
Moti -
1. Aren't use running a local jenkins instance on one of your hosts? Maybe Eyal can
publish the findbugs job?
2. What about running the findbugs UI? I find it kinda handy...
>
Could you provide instructions of running findbugs locally and how to
evaluate
the result the same as done by the jenkins job ?
> Please make sure to rebase your current patches and check that the findbugs
> job finish successfully.
> It will probably fail without a rebase, as the last patches clearing the
> warnings were merged a few hours ago.
>
> Kodus to everyone involved in clearing all the LOW level warnings... mostly
> Allon and Alissa, but others helped as well! :-)
>
Well done!
Great news - good work guys!
> Cheers,
> Oved
> _______________________________________________
> 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
3:46 a.m.
----- Original Message -----
From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
To: "Moti Asayag" <masayag(a)redhat.com>
Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>, "engine-devel" <engine-devel(a)ovirt.org>
Sent: Thursday, January 2, 2014 11:12:17 AM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Moti Asayag" <masayag(a)redhat.com>
> To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>
> Cc: "engine-devel" <engine-devel(a)ovirt.org>
> Sent: Thursday, January 2, 2014 12:08:43 AM
> Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
>
>
>
> ----- Original Message -----
> > From: "Oved Ourfalli" <ovedo(a)redhat.com>
> > To: "engine-devel" <engine-devel(a)ovirt.org>, "Allon
Mureinik"
> > <amureini(a)redhat.com>, "Alissa Bonas"
> > <abonas(a)redhat.com>
> > Sent: Wednesday, January 1, 2014 4:25:50 PM
> > Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
> >
> > Hi all,
> >
> > Up until now the jenkins jobs on the gerrit patches included a findbugs
> > job,
> > that failed only in case of a warning of level NORMAL or higher.
> > Now we update this threshold to fail on any findbugs warning, including
> > LOW
> > ones.
Moti -
1. Aren't use running a local jenkins instance on one of your hosts? Maybe
Eyal can publish the findbugs job?
Well...that server is on maintenance for quite a while...
2. What about running the findbugs UI? I find it kinda handy...
The findbugs UI expects the xml created by findbugs with all of the violations.
The question is how those violation are created ? where are the rules by which
findbugs verifies the code (can be findbugs jar or jenkins findbugs plugin or other...).
Currently by running 'mvn findbugs:findbugs' i don't get a a single xml
aggregating
all of the violations and the produced ones contain a lot of warnings.
So I'd like to know how can i be able to run this test locally ?
> >
>
> Could you provide instructions of running findbugs locally and how to
> evaluate
> the result the same as done by the jenkins job ?
>
> > Please make sure to rebase your current patches and check that the
> > findbugs
> > job finish successfully.
> > It will probably fail without a rebase, as the last patches clearing the
> > warnings were merged a few hours ago.
> >
> > Kodus to everyone involved in clearing all the LOW level warnings...
> > mostly
> > Allon and Alissa, but others helped as well! :-)
> >
>
> Well done!
Great news - good work guys!
>
> > Cheers,
> > Oved
> > _______________________________________________
> > 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
>
2:06 a.m.
----- Original Message -----
From: "Moti Asayag" <masayag(a)redhat.com>
To: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>, "engine-devel" <engine-devel(a)ovirt.org>
Sent: Thursday, January 2, 2014 11:46:29 AM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> To: "Moti Asayag" <masayag(a)redhat.com>
> Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>,
> "engine-devel" <engine-devel(a)ovirt.org>
> Sent: Thursday, January 2, 2014 11:12:17 AM
> Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
>
>
>
> ----- Original Message -----
> > From: "Moti Asayag" <masayag(a)redhat.com>
> > To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>
> > Cc: "engine-devel" <engine-devel(a)ovirt.org>
> > Sent: Thursday, January 2, 2014 12:08:43 AM
> > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> >
> >
> >
> > ----- Original Message -----
> > > From: "Oved Ourfalli" <ovedo(a)redhat.com>
> > > To: "engine-devel" <engine-devel(a)ovirt.org>, "Allon
Mureinik"
> > > <amureini(a)redhat.com>, "Alissa Bonas"
> > > <abonas(a)redhat.com>
> > > Sent: Wednesday, January 1, 2014 4:25:50 PM
> > > Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
> > >
> > > Hi all,
> > >
> > > Up until now the jenkins jobs on the gerrit patches included a findbugs
> > > job,
> > > that failed only in case of a warning of level NORMAL or higher.
> > > Now we update this threshold to fail on any findbugs warning, including
> > > LOW
> > > ones.
>
> Moti -
> 1. Aren't use running a local jenkins instance on one of your hosts? Maybe
> Eyal can publish the findbugs job?
Well...that server is on maintenance for quite a while...
> 2. What about running the findbugs UI? I find it kinda handy...
The findbugs UI expects the xml created by findbugs with all of the
violations.
The question is how those violation are created ? where are the rules by
which
findbugs verifies the code (can be findbugs jar or jenkins findbugs plugin or
other...).
Currently by running 'mvn findbugs:findbugs' i don't get a a single xml
aggregating
all of the violations and the produced ones contain a lot of warnings.
So I'd like to know how can i be able to run this test locally ?
did you try using eclipse findbugs plugin?
http://findbugs.sourceforge.net/manual/eclipse.html
i think the most easiest will be is just to check the failures you get on jenkins per
patch job.
and if there are false positives - report to infra and we'll check and fix.
you're missing the aggregate part jenkins findbugs plugin does -
http://fpaste.org/65793/09122138/.
and i'm not sure if the mvn findbugs plugin has that option on the cli.
>
>
> > >
> >
> > Could you provide instructions of running findbugs locally and how to
> > evaluate
> > the result the same as done by the jenkins job ?
> >
> > > Please make sure to rebase your current patches and check that the
> > > findbugs
> > > job finish successfully.
> > > It will probably fail without a rebase, as the last patches clearing
> > > the
> > > warnings were merged a few hours ago.
> > >
> > > Kodus to everyone involved in clearing all the LOW level warnings...
> > > mostly
> > > Allon and Alissa, but others helped as well! :-)
> > >
> >
> > Well done!
>
> Great news - good work guys!
>
> >
> > > Cheers,
> > > Oved
> > > _______________________________________________
> > > 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
> >
>
10:44 a.m.
----- Original Message -----
From: "Moti Asayag" <masayag(a)redhat.com>
To: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "engine-devel"
<engine-devel(a)ovirt.org>
Sent: Thursday, January 2, 2014 11:46:29 AM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> To: "Moti Asayag" <masayag(a)redhat.com>
> Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>,
> "engine-devel" <engine-devel(a)ovirt.org>
> Sent: Thursday, January 2, 2014 11:12:17 AM
> Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
>
>
>
> ----- Original Message -----
> > From: "Moti Asayag" <masayag(a)redhat.com>
> > To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>
> > Cc: "engine-devel" <engine-devel(a)ovirt.org>
> > Sent: Thursday, January 2, 2014 12:08:43 AM
> > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> >
> >
> >
> > ----- Original Message -----
> > > From: "Oved Ourfalli" <ovedo(a)redhat.com>
> > > To: "engine-devel" <engine-devel(a)ovirt.org>, "Allon
Mureinik"
> > > <amureini(a)redhat.com>, "Alissa Bonas"
> > > <abonas(a)redhat.com>
> > > Sent: Wednesday, January 1, 2014 4:25:50 PM
> > > Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
> > >
> > > Hi all,
> > >
> > > Up until now the jenkins jobs on the gerrit patches included a findbugs
> > > job,
> > > that failed only in case of a warning of level NORMAL or higher.
> > > Now we update this threshold to fail on any findbugs warning, including
> > > LOW
> > > ones.
>
> Moti -
> 1. Aren't use running a local jenkins instance on one of your hosts? Maybe
> Eyal can publish the findbugs job?
Well...that server is on maintenance for quite a while...
> 2. What about running the findbugs UI? I find it kinda handy...
The findbugs UI expects the xml created by findbugs with all of the
violations.
The question is how those violation are created ? where are the rules by
which
findbugs verifies the code (can be findbugs jar or jenkins findbugs plugin or
other...).
Currently by running 'mvn findbugs:findbugs' i don't get a a single xml
aggregating
all of the violations and the produced ones contain a lot of warnings.
So I'd like to know how can i be able to run this test locally ?
You're
missing the profile with the project's exclusions.
What you'd want to do is probably this:
mvn findbugs:check -Pfindbugs-general
This way you won't have to go over any report, you will just fail the build if you
introduce a regression.
>
>
> > >
> >
> > Could you provide instructions of running findbugs locally and how to
> > evaluate
> > the result the same as done by the jenkins job ?
> >
> > > Please make sure to rebase your current patches and check that the
> > > findbugs
> > > job finish successfully.
> > > It will probably fail without a rebase, as the last patches clearing
> > > the
> > > warnings were merged a few hours ago.
> > >
> > > Kodus to everyone involved in clearing all the LOW level warnings...
> > > mostly
> > > Allon and Alissa, but others helped as well! :-)
> > >
> >
> > Well done!
>
> Great news - good work guys!
>
> >
> > > Cheers,
> > > Oved
> > > _______________________________________________
> > > 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
11:07 p.m.
----- Original Message -----
From: "Allon Mureinik" <amureini(a)redhat.com>
To: "Moti Asayag" <masayag(a)redhat.com>
Cc: "Yair Zaslavsky" <yzaslavs(a)redhat.com>, "Oved Ourfalli"
<ovedo(a)redhat.com>, "engine-devel"
<engine-devel(a)ovirt.org>
Sent: Monday, January 6, 2014 6:44:48 PM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Moti Asayag" <masayag(a)redhat.com>
> To: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "engine-devel"
> <engine-devel(a)ovirt.org>
> Sent: Thursday, January 2, 2014 11:46:29 AM
> Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
>
>
>
> ----- Original Message -----
> > From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> > To: "Moti Asayag" <masayag(a)redhat.com>
> > Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal Edri"
<eedri(a)redhat.com>,
> > "engine-devel" <engine-devel(a)ovirt.org>
> > Sent: Thursday, January 2, 2014 11:12:17 AM
> > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> >
> >
> >
> > ----- Original Message -----
> > > From: "Moti Asayag" <masayag(a)redhat.com>
> > > To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal
Edri" <eedri(a)redhat.com>
> > > Cc: "engine-devel" <engine-devel(a)ovirt.org>
> > > Sent: Thursday, January 2, 2014 12:08:43 AM
> > > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> > >
> > >
> > >
> > > ----- Original Message -----
> > > > From: "Oved Ourfalli" <ovedo(a)redhat.com>
> > > > To: "engine-devel" <engine-devel(a)ovirt.org>,
"Allon Mureinik"
> > > > <amureini(a)redhat.com>, "Alissa Bonas"
> > > > <abonas(a)redhat.com>
> > > > Sent: Wednesday, January 1, 2014 4:25:50 PM
> > > > Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
> > > >
> > > > Hi all,
> > > >
> > > > Up until now the jenkins jobs on the gerrit patches included a
> > > > findbugs
> > > > job,
> > > > that failed only in case of a warning of level NORMAL or higher.
> > > > Now we update this threshold to fail on any findbugs warning,
> > > > including
> > > > LOW
> > > > ones.
> >
> > Moti -
> > 1. Aren't use running a local jenkins instance on one of your hosts?
> > Maybe
> > Eyal can publish the findbugs job?
>
> Well...that server is on maintenance for quite a while...
>
> > 2. What about running the findbugs UI? I find it kinda handy...
>
> The findbugs UI expects the xml created by findbugs with all of the
> violations.
> The question is how those violation are created ? where are the rules by
> which
> findbugs verifies the code (can be findbugs jar or jenkins findbugs plugin
> or
> other...).
>
> Currently by running 'mvn findbugs:findbugs' i don't get a a single xml
> aggregating
> all of the violations and the produced ones contain a lot of warnings.
>
> So I'd like to know how can i be able to run this test locally ?
You're missing the profile with the project's exclusions.
What you'd want to do is probably this:
mvn findbugs:check -Pfindbugs-general
This way you won't have to go over any report, you will just fail the build
if you introduce a regression.
Thanks Allon,
This worked - however - you get a full textual description of the "bug" and not
the code.
In some cases one may want to filter out a specific bug report, so the "bug
code" is important.
Do you know by any chance how to produce this using mvn findbugs ?
Yair
>
> >
> >
> > > >
> > >
> > > Could you provide instructions of running findbugs locally and how to
> > > evaluate
> > > the result the same as done by the jenkins job ?
> > >
> > > > Please make sure to rebase your current patches and check that the
> > > > findbugs
> > > > job finish successfully.
> > > > It will probably fail without a rebase, as the last patches clearing
> > > > the
> > > > warnings were merged a few hours ago.
> > > >
> > > > Kodus to everyone involved in clearing all the LOW level warnings...
> > > > mostly
> > > > Allon and Alissa, but others helped as well! :-)
> > > >
> > >
> > > Well done!
> >
> > Great news - good work guys!
> >
> > >
> > > > Cheers,
> > > > Oved
> > > > _______________________________________________
> > > > 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
>
2:49 p.m.
----- Original Message -----
From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
To: "Allon Mureinik" <amureini(a)redhat.com>
Cc: "Moti Asayag" <masayag(a)redhat.com>, "Oved Ourfalli"
<ovedo(a)redhat.com>, "engine-devel" <engine-devel(a)ovirt.org>
Sent: Tuesday, January 7, 2014 7:07:48 AM
Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
----- Original Message -----
> From: "Allon Mureinik" <amureini(a)redhat.com>
> To: "Moti Asayag" <masayag(a)redhat.com>
> Cc: "Yair Zaslavsky" <yzaslavs(a)redhat.com>, "Oved
Ourfalli"
> <ovedo(a)redhat.com>, "engine-devel"
> <engine-devel(a)ovirt.org>
> Sent: Monday, January 6, 2014 6:44:48 PM
> Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
>
>
>
> ----- Original Message -----
> > From: "Moti Asayag" <masayag(a)redhat.com>
> > To: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> > Cc: "Oved Ourfalli" <ovedo(a)redhat.com>,
"engine-devel"
> > <engine-devel(a)ovirt.org>
> > Sent: Thursday, January 2, 2014 11:46:29 AM
> > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> >
> >
> >
> > ----- Original Message -----
> > > From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> > > To: "Moti Asayag" <masayag(a)redhat.com>
> > > Cc: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal
Edri" <eedri(a)redhat.com>,
> > > "engine-devel" <engine-devel(a)ovirt.org>
> > > Sent: Thursday, January 2, 2014 11:12:17 AM
> > > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> > >
> > >
> > >
> > > ----- Original Message -----
> > > > From: "Moti Asayag" <masayag(a)redhat.com>
> > > > To: "Oved Ourfalli" <ovedo(a)redhat.com>, "Eyal
Edri"
> > > > <eedri(a)redhat.com>
> > > > Cc: "engine-devel" <engine-devel(a)ovirt.org>
> > > > Sent: Thursday, January 2, 2014 12:08:43 AM
> > > > Subject: Re: [Engine-devel] IMPORTANT: FindBugs threshold update
> > > >
> > > >
> > > >
> > > > ----- Original Message -----
> > > > > From: "Oved Ourfalli" <ovedo(a)redhat.com>
> > > > > To: "engine-devel" <engine-devel(a)ovirt.org>,
"Allon Mureinik"
> > > > > <amureini(a)redhat.com>, "Alissa Bonas"
> > > > > <abonas(a)redhat.com>
> > > > > Sent: Wednesday, January 1, 2014 4:25:50 PM
> > > > > Subject: [Engine-devel] IMPORTANT: FindBugs threshold update
> > > > >
> > > > > Hi all,
> > > > >
> > > > > Up until now the jenkins jobs on the gerrit patches included a
> > > > > findbugs
> > > > > job,
> > > > > that failed only in case of a warning of level NORMAL or
higher.
> > > > > Now we update this threshold to fail on any findbugs warning,
> > > > > including
> > > > > LOW
> > > > > ones.
> > >
> > > Moti -
> > > 1. Aren't use running a local jenkins instance on one of your hosts?
> > > Maybe
> > > Eyal can publish the findbugs job?
> >
> > Well...that server is on maintenance for quite a while...
> >
> > > 2. What about running the findbugs UI? I find it kinda handy...
> >
> > The findbugs UI expects the xml created by findbugs with all of the
> > violations.
> > The question is how those violation are created ? where are the rules by
> > which
> > findbugs verifies the code (can be findbugs jar or jenkins findbugs
> > plugin
> > or
> > other...).
> >
> > Currently by running 'mvn findbugs:findbugs' i don't get a a single
xml
> > aggregating
> > all of the violations and the produced ones contain a lot of warnings.
> >
> > So I'd like to know how can i be able to run this test locally ?
> You're missing the profile with the project's exclusions.
> What you'd want to do is probably this:
> mvn findbugs:check -Pfindbugs-general
>
> This way you won't have to go over any report, you will just fail the build
> if you introduce a regression.
Thanks Allon,
This worked - however - you get a full textual description of the "bug" and
not the code.
In some cases one may want to filter out a specific bug report, so the "bug
code" is important.
Do you know by any chance how to produce this using mvn findbugs ?
Yair
What I usually do is
mvn findbugs:findbugs -Pfindbugs-general
This will create an xml report under each module's target/site folder with all the
details you may need in order to exclude the resulting bug.
Having said that, if you need to add an exclusion, there's a better-than-fair chance
you're going the wrong way.
>
> >
> > >
> > >
> > > > >
> > > >
> > > > Could you provide instructions of running findbugs locally and how
to
> > > > evaluate
> > > > the result the same as done by the jenkins job ?
> > > >
> > > > > Please make sure to rebase your current patches and check that
the
> > > > > findbugs
> > > > > job finish successfully.
> > > > > It will probably fail without a rebase, as the last patches
> > > > > clearing
> > > > > the
> > > > > warnings were merged a few hours ago.
> > > > >
> > > > > Kodus to everyone involved in clearing all the LOW level
> > > > > warnings...
> > > > > mostly
> > > > > Allon and Alissa, but others helped as well! :-)
> > > > >
> > > >
> > > > Well done!
> > >
> > > Great news - good work guys!
> > >
> > > >
> > > > > Cheers,
> > > > > Oved
> > > > > _______________________________________________
> > > > > 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
> >
>
3755
days inactive
3767
days old
8 comments
6 participants
participants (6)
-
Allon Mureinik -
Eyal Edri -
Itamar Heim -
Moti Asayag -
Oved Ourfalli -
Yair Zaslavsky