Re: [ovirt-devel] Findbugs on master patches
On 08/12, Yevgeny Zaspitsky wrote: The findbugs failure is related to patch set 1, which was previously was marked by Jenkins as +1. If that wasn't +1'ed, we would've wait until Jenkins job is finished, but since it has +1 we were under impression that the patch is OK. You are right, the findbugs was triggered by the verified +1 on patch 1. Looks like findbugs job was triggered by Verified+1, whereas test were run upon patch set upload. IMHO the split between the jobs makes their behavior misleading. Agree, but so far there's no agreement on the flow for it, and there's no effort being done to reach that agreement (it's on hold) so it will not change any time soon. I think we should drop the triggering by flag, or make it generic but not mix both. That's why I wanted to introduce the workflow flag, to trigger only by that flag on demand of the reviewer. Feel free to try and start that discussion again BTW, what was merged is patch set 2, which wasn't checked by findbugs at all. On Wed, Aug 12, 2015 at 3:42 PM, David Caro dc...@redhat.com wrote: On 08/12, Yevgeny Zaspitsky wrote: My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, That much later is after merge, the findbugs ran once the patch was merged. You should understand that each event triggers a tests run, and for each tests run you get only one review. In this case the tests that check the code when you send the patch passed, but the ones that check the code after it's merged failed. Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpUoszkTFRqZ.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Findbugs on master patches
On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Findbugs on master patches
My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Findbugs on master patches
On 08/12, Yevgeny Zaspitsky wrote: My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, That much later is after merge, the findbugs ran once the patch was merged. You should understand that each event triggers a tests run, and for each tests run you get only one review. In this case the tests that check the code when you send the patch passed, but the ones that check the code after it's merged failed. Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgp10ZaBV1htj.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Findbugs on master patches
On Wed, Aug 12, 2015 at 2:42 PM, David Caro dc...@redhat.com wrote: On 08/12, Yevgeny Zaspitsky wrote: My bad, sorry. The findbugs job runs indeed. However look at that patch https://gerrit.ovirt.org/#/c/44604/ . Patch set 1 got Continuous-Integration+1 from Jenkins after running unit tests and then, much later, That much later is after merge, the findbugs ran once the patch was merged. You should understand that each event triggers a tests run, and for each tests run you get only one review. In this case the tests that check the code when you send the patch passed, but the ones that check the code after it's merged failed. In this case, I think we need the job which failed after merge to be run also on patch creation. Jenkins changed its mind and gave -1 after findbugs job has finished. IMHO that's is a misleading behavior. Regards, Yevgeny On Wed, Aug 12, 2015 at 12:12 PM, Sandro Bonazzola sbona...@redhat.com wrote: On Tue, Aug 11, 2015 at 6:13 PM, Yevgeny Zaspitsky yzasp...@redhat.com wrote: Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? AFAIK findbugs is still up and running: http://jenkins.ovirt.org/search/?q=find-bugs Have you any specific patch as example? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Findbugs on master patches
Hi All, Seems like recently Jenkins has stopped running findbugs check on the patches before they get merged into master. Although that could save some Jenkins resources and make the its reaction faster, that makes the integration (with master) check less effective. Was that done on purpose? Can that be restored for master patches? Regards, Yevgeny ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel