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

oliel at redhat.com oliel at redhat.com
Mon Sep 7 12:58:28 UTC 2015


Ori Liel 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>
---
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:
  Ori Liel: Verified; Looks good to me, approved; Passed CI tests
  Martin Peřina: Looks good to me, but someone else must approve
  Oved Ourfali: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd4a0a788fba25c491f693747f00f4c64f4413d4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <oliel 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