Change in vdsm[master]: vdsm: Add support for Guest Reboot

danken at redhat.com danken at redhat.com
Fri Dec 20 16:49:15 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: vdsm: Add support for Guest Reboot
......................................................................


vdsm: Add support for Guest Reboot

Added optional parameter reboot to shutdown in vdsm and vdsClient.
This parameter represents whether the user wants to reboot the VM
(reboot=True) or just a regular shutdown (reboot=False). To keep
backwards compatibility, the default is False.

Current implementation only utilizes the guest agent for reboot and does
nothing in its absence. The semantics of existing shutdown should be
unchanged by this patch.

Note that older versions of GA don't support the reboot flag and
consqeuently will issue a shutdown instead of reboot.

Change-Id: Ic117574365fc6f7a0dd6e69ba203b2d82d4cefa1
Signed-off-by: Martin Betak <mbetak at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22440
Reviewed-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M client/vdsClient.py
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/guestIF.py
M vdsm/vm.py
M vdsm_api/vdsmapi-schema.json
6 files changed, 39 insertions(+), 23 deletions(-)

Approvals:
  Martin Betak: Verified
  Vinzenz Feenstra: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve
  Michal Skrivanek: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/22440
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic117574365fc6f7a0dd6e69ba203b2d82d4cefa1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list