5:08 p.m.
--oTHb8nViIGeoXxdp
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On 03/09 15:57, Sandro Bonazzola wrote:
About https://gerrit.ovirt.org/#/c/46925/
=20
=20
*Juan Hernandez* Mar 7 5:34 PM
=20
Patch Set 14:
=20
The only important thing that is missing is what will happen with the
findbugs reports. Will we still have a report that tells us what bugs
exist, what have been fixed since the last build, etc?=20
No, it won't
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?
No, we shouldn't (that's the idea if generalizing the jobs, so there are no
special ones)
=20
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 the build
with the first issue. It may be more appropriate for this purpose.
That's a nice improvement, as allows removing the check for errors in the
report file that it's doing right now.
=20
=20
Eyal, David, other infra members, any hint on how to progress with this
patch?
My idea of this was to solve it creating dynamic jobs with the pipeline plu=
gin
whenever we started using it for the standard ci as it allows you to
dynamically generate jobs, you might be able to see if there was a report
generated before enabling certain plugin (like findbugs), as the current is=
sue
is that if you enable it for the standard ci, it will fail if the report is=
not
generated (and not all the jobs generate it).
But that might take some time (more now that it's dropped from my prioritie=
s)
so you might want to find another solution.
I really don't recommend it, but it should be possible to use yaml macros a=
nd
write one macro for each job that needs those extra plugins including the
standard and adding the publisher. But that's bread for today and hunger for
tomorrow, so be careful.
=20
=20
--=20
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com
_______________________________________________
Infra mailing list
Infra(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra
--=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
--oTHb8nViIGeoXxdp
Content-Type: application/pgp-signature; name="signature.asc"
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQEcBAEBAgAGBQJW4DyCAAoJEEBxx+HSYmnDElwH/0yWD0XQhRwnuebXy1SBt0wo
vAjyT8fuj0OskJdLocbGv69Ry7fMpdEpeOVs6A0GSgzKwW9yexgoB5uo+c82XgNu
w6X0w8cGs1bslDZnIeMEC00dQzSv6pUobL2IPIxoW9alZz9xEM5y2ynKXi2QipqD
a3eo9HgvtnHSSWdPQB0cvB5pXn6/2erm/ZCqu8pXehm61Z/zOXmLTZUsdNn1Psps
Tw9pzaswH3E1pAi6uivbJnW08UrO7hPy9MFMU8L2wmJHk8w97pAPw2kGov1AYKFC
rNp2K2jhS61YEWJTFiG9/9b3sdXtESAXA9+CV8kQ1/PP7UZdha37dTGaWOakOXc=
=1Bpl
-----END PGP SIGNATURE-----
--oTHb8nViIGeoXxdp--