Hi Gilad[/infra], I would like to merge patch [1], however I am getting 2 findbugs warnings on the BLL [2].
The patch is a frontend patch, so it has nothing to do with the BLL. findbugs is shouting on the 'equals' method of the PolicyUnit class [2]. it seems like the relevant code was merged quite a while ago [3] so I don't understand why it started shouting all of a sudden. any idea? Many thanks in advance. ---- Regards, Einav [1] http://gerrit.ovirt.org/#/c/23027/ [2] http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/4708/findbugsResult/source.399/#119 [3] http://gerrit.ovirt.org/#/c/16107/ ----- Forwarded Message ----- From: "oVirt Jenkins CI Server" <[email protected]> To: "Alexander Wels" <[email protected]>, "Einav Cohen" <[email protected]>, "Gilad Chaplik" <[email protected]> Sent: Wednesday, January 15, 2014 7:38:37 AM Subject: Change in ovirt-engine[master]: webadmin: user session is kept alive oVirt Jenkins CI Server has posted comments on this change. Change subject: webadmin: user session is kept alive ...................................................................... Patch Set 3: Code-Review-1 Build Unstable http://jenkins.ovirt.org/job/ovirt_engine_master_unit_tests_gerrit_old_using_plugin/5775/ : SUCCESS http://jenkins.ovirt.org/job/ovirt_engine_master_unit_tests_gerrit/5/ : SUCCESS http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/16237/ : SUCCESS http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/4708/ : UNSTABLE -- To view, visit http://gerrit.ovirt.org/23027 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib99861bfdc41b98ceac0df7103acff5b9074f851 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Alexander Wels <[email protected]> Gerrit-Reviewer: Einav Cohen <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Infra mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/infra
