Arik Hadas has submitted this change and it was merged. Change subject: core: set vm exit status to error on fencing ...................................................................... core: set vm exit status to error on fencing As part of fence operation, we set the VM status to down using SetVmStatusVDSCommand. This command did not filled the exit status of the VM which should be ERROR in this case. So this patch adds that missing part by passing the exit status that should be set for the VM while changing its status through the parameters of the command. Change-Id: If541c6d33a7cfa310ba776e4bc7fa0f1e45aa077 Bug-Url: https://bugzilla.redhat.com/1077706 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/SetVmStatusVDSCommandParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/SetVmStatusVDSCommand.java 4 files changed, 35 insertions(+), 17 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/26101 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If541c6d33a7cfa310ba776e4bc7fa0f1e45aa077 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server