Change in ovirt-engine[master]: core: Unbreak CorrelationIdValidationTest

Code Review gerrit at ovirt.org
Thu Nov 10 16:25:17 UTC 2016


>From Allon Mureinik <amureini at redhat.com>:

Allon Mureinik has submitted this change and it was merged.

Change subject: core: Unbreak CorrelationIdValidationTest
......................................................................


core: Unbreak CorrelationIdValidationTest

CorrelationIdValidationTest fails intermittently upstream. The root
cause seems to be the static member in CorrelationIdTracker which may or
may not get set in previous tests.

This patch assures that it's initialized per test run in order to make
the test reliable.

Change-Id: I8727447aefd5b7b203366faa3143440365f583c8
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/job/CorrelationIdValidationTest.java
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Verified; Passed CI tests

Objections:
  Jenkins CI: Failed CI tests



-- 
To view, visit https://gerrit.ovirt.org/66400
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8727447aefd5b7b203366faa3143440365f583c8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list