Change in ovirt-hosted-engine-ha[master]: storage: avoid assuming lockspace volume always exists

msivak at redhat.com msivak at redhat.com
Tue Mar 15 08:55:57 UTC 2016


Martin Sivák has submitted this change and it was merged.

Change subject: storage: avoid assuming lockspace volume always exists
......................................................................


storage: avoid assuming lockspace volume always exists

On oVirt 3.4 hosted-engine lockspace and metadata were file based
(it was NFS only), supporting also block devices we moved new
deployment over VDSM managed volumes but system initially
deployed with 3.4 still uses file based lockspace.
Relying on lockspace volume existence was not really
needed to identify the image path.

Change-Id: If24937e9c3b71f4ad4df00065740145e332630dd
Bug-Url: https://bugzilla.redhat.com/1316143
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
---
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/lib/image.py
M ovirt_hosted_engine_ha/lib/ovf/ovf_store.py
M ovirt_hosted_engine_ha/lib/upgrade.py
4 files changed, 38 insertions(+), 27 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Martin Sivák: Looks good to me, approved
  Simone Tiraboschi: Verified
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If24937e9c3b71f4ad4df00065740145e332630dd
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list