Change in ovirt-engine[ovirt-engine-3.6]: Engine: Hide bad 'Done' message for PM test

oourfali at redhat.com oourfali at redhat.com
Wed Sep 23 08:25:17 UTC 2015


Oved Ourfali has submitted this change and it was merged.

Change subject: Engine: Hide bad 'Done' message for PM test
......................................................................


Engine: Hide bad 'Done' message for PM test

VSDM returns 'Done' by default for fencing operations. This is not useful and appending
it to log messages results in a confusing message. So if the message is 'Done', it's
better to ignore it. This patch does that.

Bug-Url: https://bugzilla.redhat.com/1203741
Change-Id: Icd4a0a788fba25c491f693747f00f4c64f4413d4
Signed-off-by: Ori Liel <oliel at redhat.com>
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/pm/FenceOperationResult.java
1 file changed, 9 insertions(+), 3 deletions(-)

Approvals:
  Martin Peřina: Verified
  Jenkins CI: Passed CI tests
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd4a0a788fba25c491f693747f00f4c64f4413d4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list