Change in vdsm[master]: vm: destroy: retry to gracefully destroy

danken at redhat.com danken at redhat.com
Thu Mar 31 10:20:00 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: vm: destroy: retry to gracefully destroy
......................................................................


vm: destroy: retry to gracefully destroy

Add support to retry gracefully to destroy a VM
before to go forcefully.
Clients (Engine) could use a new optional parameter
to tell Vdsm how many times it should retry
gracefully. This is meant to reduce the risk
of data corruption in the case the first graceful
destroy fails (e.g. EBUSY, or system-level timeout).

The default number of graceful attempt is one for
backward compatibility.

Change-Id: I513b16c5091cb22313dc18e2587ed6db386e982a
Bug-Url: https://bugzilla.redhat.com/1309884
Backport-To: 3.6
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/55224
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Continuous-Integration: Jenkins CI
---
M lib/api/vdsmapi-schema.json
M tests/vmTests.py
M tests/vmfakelib.py
M vdsm/API.py
M vdsm/virt/vm.py
5 files changed, 93 insertions(+), 13 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified



-- 
To view, visit https://gerrit.ovirt.org/55224
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I513b16c5091cb22313dc18e2587ed6db386e982a
Gerrit-PatchSet: 12
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: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list