Sandro Bonazzola has uploaded a new change for review.

Change subject: Add missing call to prepareImage
......................................................................

Add missing call to prepareImage

adding the missing call to prepareImage for
allowing to write the image imported from the OVA
to an iSCSI or FC storage.
On NFS storage it worked also without this call but
it's required on iSCSI.

Change-Id: If42529e6f02b5803e0c2e1fd8d0ab7550b1402dd
Bug-Url: https://bugzilla.redhat.com/1222058
Signed-off-by: Sandro Bonazzola <sbona...@redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/vm/image.py
1 file changed, 12 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-hosted-engine-setup 
refs/changes/65/41965/1

diff --git a/src/plugins/ovirt-hosted-engine-setup/vm/image.py 
b/src/plugins/ovirt-hosted-engine-setup/vm/image.py
index e60f210..90d33f2 100644
--- a/src/plugins/ovirt-hosted-engine-setup/vm/image.py
+++ b/src/plugins/ovirt-hosted-engine-setup/vm/image.py
@@ -231,6 +231,7 @@
             preallocate = ohostedcons.VolumeTypes.PREALLOCATED_VOL
 
         diskType = 2
+        # creates a volume on the storage (SPM verb)
         status = cli.createVolume(
             sdUUID,
             spUUID,
@@ -263,6 +264,17 @@
             raise RuntimeError(status['status']['message'])
         waiter = tasks.TaskWaiter(self.environment)
         waiter.wait()
+        # Expose the image (e.g., activates the lv) on the host (HSM verb).
+        self.logger.debug('prepareImage')
+        response = cli.prepareImage(
+            spUUID,
+            sdUUID,
+            imgUUID,
+            volUUID
+        )
+        self.logger.debug(response)
+        if response['status']['code'] != 0:
+            raise RuntimeError(response['status']['message'])
 
 
 # vim: expandtab tabstop=4 shiftwidth=4


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If42529e6f02b5803e0c2e1fd8d0ab7550b1402dd
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to