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(a)redhat.com>
Signed-off-by: Martin Perina <mperina(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org