Simone Tiraboschi has uploaded a new change for review.

Change subject: packaging: setup: better handling of dirty storage
......................................................................

packaging: setup: better handling of dirty storage

Better handling of dirty storage

Change-Id: I7b9c05d26db929c408943455c7c9ac15764be3d4
Bug-Url: https://bugzilla.redhat.com/1155637
Signed-off-by: Simone Tiraboschi <stira...@redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
1 file changed, 10 insertions(+), 8 deletions(-)


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

diff --git a/src/plugins/ovirt-hosted-engine-setup/storage/storage.py 
b/src/plugins/ovirt-hosted-engine-setup/storage/storage.py
index 9f70949..7075968 100644
--- a/src/plugins/ovirt-hosted-engine-setup/storage/storage.py
+++ b/src/plugins/ovirt-hosted-engine-setup/storage/storage.py
@@ -283,9 +283,9 @@
                 vginfo = self.cli.getVGInfo(
                     self.environment[ohostedcons.StorageEnv.VG_UUID]
                 )
+                self.logger.debug(vginfo)
                 if vginfo['status']['code'] != 0:
                     raise RuntimeError(vginfo['status']['message'])
-                self.domain_exists = True
                 self.environment[
                     ohostedcons.CoreEnv.ADDITIONAL_HOST_ENABLED
                 ] = True
@@ -297,13 +297,15 @@
                         ohostedcons.StorageEnv.SD_UUID
                     ]
                 )
-                pool_list = domain_info['pool']
-                if pool_list:
-                    self.pool_exists = True
-                    spUUID = pool_list[0]
-                    self.environment[
-                        ohostedcons.StorageEnv.SP_UUID
-                    ] = spUUID
+                if domain_info['status']['code'] == 0:
+                    self.domain_exists = True
+                    pool_list = domain_info['pool']
+                    if pool_list:
+                        self.pool_exists = True
+                        spUUID = pool_list[0]
+                        self.environment[
+                            ohostedcons.StorageEnv.SP_UUID
+                        ] = spUUID
                 self._handleHostId()
                 if self.pool_exists:
                     pool_info = self._getStoragePoolInfo(spUUID)


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

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

Reply via email to