Change in vdsm[master]: vm: report exit reason if restore state fails
danken at redhat.com
danken at redhat.com
Fri May 16 10:53:14 UTC 2014
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: report exit reason if restore state fails
......................................................................
vm: report exit reason if restore state fails
The exit reason code needs to be reported an all error paths.
It was mistakenly left out on the restoreState error path,
most likely because this path is different because the exitMessage
is overwritten.
This patch ensures exitReason is always reported.
Change-Id: Ifda6852e5d91cfb17d8c7c2d6a0997346854ec83
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/27703
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
--
To view, visit http://gerrit.ovirt.org/27703
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifda6852e5d91cfb17d8c7c2d6a0997346854ec83
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list