Shlomi, Can you have a look? On Sun, May 8, 2016 at 10:34 PM, Tal Nisan <tnisan(a)redhat.com&gt; wrote: > Upon looking at another patch I saw that there's a new findbugs issue: > > http://jenkins-old.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/452... > > I've fixed the issue in this patch - https://gerrit.ovirt.org/57209 so > it's all good now but I'm wondering why the findbugs job did not run in the > first place. > You can see that in the patch that introduced the issue - > https://gerrit.ovirt.org/#/c/57054/ > Also, why did the resource consuming DAO test ran although there were no > changes in the dal module? > > > _______________________________________________ > Infra mailing list > Infra(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra > > -- Eyal Edri Associate Manager RHEV DevOps EMEA ENG Virtualization R&D Red Hat Israel phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)

Hi Nisan, 1. The findbugs job run at May 5 1:50 PM and returned with the following msg: 'The patch does not pass the findbugs' 2. The resource consuming DAO test run because there was a change in the following file in the patch: backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/BondMode.java It's configured in Jenkins gerrit filters as a trigger. Best Regards, Shlomi Ben-David | Software Engineer | Red Hat ISRAEL IRC: sbendavi OPEN SOURCE - 1 4 011 && 011 4 1 On Sun, May 8, 2016 at 10:34 PM, Tal Nisan <tnisan(a)redhat.com&gt; wrote: > Upon looking at another patch I saw that there's a new findbugs issue: > > http://jenkins-old.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/452... > > I've fixed the issue in this patch - https://gerrit.ovirt.org/57209 so > it's all good now but I'm wondering why the findbugs job did not run in the > first place. > You can see that in the patch that introduced the issue - > https://gerrit.ovirt.org/#/c/57054/ > Also, why did the resource consuming DAO test ran although there were no > changes in the dal module? > > > _______________________________________________ > Infra mailing list > Infra(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra > >