Dan Kenigsberg has submitted this change and it was merged.

Change subject: vdsm: Add force and timeout options to VM.shutdown
......................................................................


vdsm: Add force and timeout options to VM.shutdown

Added optional parameters 'timeout' and 'force' to
shutdown in vdsm and vdsClient.

Timeout represents number of seconds to wait before falling back to next
alternative method of shutdown/reboot.

If 'force' is True and all graceful methods fail, we reset/destroy the VM.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1054070
Change-Id: I30180f9906eb45fa485aa7d773a436e62bd3c815
Signed-off-by: Martin Betak <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/27054
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Vinzenz Feenstra <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M client/vdsClient.py
M lib/vdsm/config.py.in
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/virt/vm.py
M vdsm/virt/vmpowerdown.py
M vdsm_api/vdsmapi-schema.json
7 files changed, 65 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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I30180f9906eb45fa485aa7d773a436e62bd3c815
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Martin Betak <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to