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