--aZoGpuMECXJckB41
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On 03/09 15:32, Eyal Edri wrote:
On Wed, Mar 9, 2016 at 3:31 PM, David Caro Estevez
<dcaro(a)redhat.com> wro=
te:
=20
> On 03/09 15:28, Eyal Edri wrote:
> > Isn't there a plugin like x-unit for findbugs that can just get XML f=
ile
> > and present the graphs?
> > for the alerts on limit maybe we can implement something ourselves.
>
> There is, the issue is that if that xml file does not exist it fails the
> job,
> so it's not good to be added to the ci-standard
>
=20
What is the reason for not having that xml job? if we don't choose to run
findbugs mvn goal?
It's running the findbugs, and it will fail if there are any errors, the
question is if it would show nice graphs and compare the job historic resul=
ts
(what the jenkins findbugs plugin does). And you can't do that out of jenki=
ns
(not easily at least)
=20
=20
>
> >
> > On Wed, Mar 9, 2016 at 3:21 PM, David Caro Estevez <dcaro(a)redhat.com>
> wrote:
> >
> > > On 03/09 15:19, Eyal Edri wrote:
> > > > I think the idea is to add report.html generic file that will show
> > > anything
> > > > we'd like.
> > > > Having said that, it might be too complex or overkill to do it ri=
ght
> now
> > > > just for findbugs, so maybe we can still use the findbugs plugin =
and
> > > > provide the xml file (?)
> > > > to the plugin?
> > > >
> > > > Maybe we can use [1] as a generic static analysis single plugin t=
hat
in
> > >
> > >
> > > This has the same issues than using each specific plugin, as it
> requires
> > > each
> > > of them to be configured (it just aggregates them, but does not do =
the
> > > result computations)
> > >
> > > >
> > > >
> > > > On Wed, Mar 9, 2016 at 2:57 PM, Sandro Bonazzola <
> sbonazzo(a)redhat.com>
> > > > wrote:
> > > >
> > > > > About
https://gerrit.ovirt.org/#/c/46925/
> > > > >
> > > > >
> > > > > *Juan Hernandez* Mar 7 5:34 PM
> > > > >
> > > > > Patch Set 14:
> > > > >
> > > > > The only important thing that is missing is what will happen wi=
th
> the
> > > > > findbugs reports. Will we still have a report that tells us
what
> bugs
> > > > > exist, what have been fixed since the last build, etc? If I
> understand
> > > > > correctly that is generated by the Jenkins findbugs plugin, and
> this
> > > patch
> > > > > doesn't use it. Will we preserve the separate findbugs job?
> > > > >
> > > > > In addition I have recently learned that the
"findbugs:findbugs"
> mojo
> > > runs
> > > > > the checks, but doesn't fail the build if it finds issues.
There
> is a
> > > > > different "findbugs:check" mojo that runs the check
and fails t=
he
> build
> > > > > with the first issue. It may be more appropriate for this purpo=
se.
> > > > >
> > > > >
> > > > > Eyal, David, other infra members, any hint on how to progress w=
ith
> this
> > > > > patch?
> > > > >
> > > > >
> > > > > --
> > > > > Sandro Bonazzola
> > > > > Better technology. Faster innovation. Powered by community
> > > collaboration.
> > > > > See how it works at
redhat.com
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Eyal Edri
> > > > Associate Manager
> > > > EMEA ENG Virtualization R&D
> > > > Red Hat Israel
> > > >
> > > > phone: +972-9-7692018
> > > > irc: eedri (on #tlv #rhev-dev #rhev-integ)
> > >
> > > --
> > > David Caro
> > >
> > > Red Hat S.L.
> > > Continuous Integration Engineer - EMEA ENG Virtualization R&D
> > >
> > > Tel.: +420 532 294 605
> > > Email: dcaro(a)redhat.com
> > > IRC: dcaro|dcaroest@{freenode|oftc|redhat}
> > > Web:
www.redhat.com
> > > RHT Global #: 82-62605
> > >
> >
> >
> >
> > --
> > Eyal Edri
> > Associate Manager
> > EMEA ENG Virtualization R&D
> > Red Hat Israel
> >
> > phone: +972-9-7692018
> > irc: eedri (on #tlv #rhev-dev #rhev-integ)
>
> --
> David Caro
>
> Red Hat S.L.
> Continuous Integration Engineer - EMEA ENG Virtualization R&D
>
> Tel.: +420 532 294 605
> Email: dcaro(a)redhat.com
> IRC: dcaro|dcaroest@{freenode|oftc|redhat}
> Web:
www.redhat.com
> RHT Global #: 82-62605
>
=20
=20
=20
--=20
Eyal Edri
Associate Manager
EMEA ENG Virtualization R&D
Red Hat Israel
=20
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
--=20
David Caro
Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D
Tel.: +420 532 294 605
Email: dcaro(a)redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web:
www.redhat.com
RHT Global #: 82-62605
--aZoGpuMECXJckB41
Content-Type: application/pgp-signature; name="signature.asc"
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQEcBAEBAgAGBQJW4EK1AAoJEEBxx+HSYmnDTLcH/0BVB3jHVI0vlGQR1ZscgtN4
boOeQ++PG+l6XMEx0cETqXIylECiwTKEnisliaRrcdiSLb4cv8qXhqDtT6S/eury
97qe1evijN/yZQXZD4sgGYzvM5OPq2QdHDgltlFd5lQIciu/a5Of15ABKizFhu5Y
W3mO7+bkWaYDBTzSiFmBLsWrIEJXH0krL92cxdczdpecsi53ro26PlQv/phEXWFe
VZeN/WvovgtNw2MLgdP4RPHpux3jxshtO495Ofia7W8vMxPupI1cYZMn+QYW1pjX
ry9bZopQvlV+Uc49Iipd7cdSEpzxZGk1Kwzwl0e8O6l6UNLOF8qLqapMHuEmelQ=
=xky6
-----END PGP SIGNATURE-----
--aZoGpuMECXJckB41--