Sandro Bonazzola has uploaded a new change for review.

Change subject: glusterfs: better error messages
......................................................................

glusterfs: better error messages

Added volume info to Volume not found error.
Moved the warning about replica3 needed to info level.

Change-Id: I0a8622ede6be78a4fa58b6cea3032975315024d6
Signed-off-by: Sandro Bonazzola <sbona...@redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/storage/nfs.py
1 file changed, 6 insertions(+), 2 deletions(-)


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

diff --git a/src/plugins/ovirt-hosted-engine-setup/storage/nfs.py 
b/src/plugins/ovirt-hosted-engine-setup/storage/nfs.py
index 68a1c29..e0c89cc 100644
--- a/src/plugins/ovirt-hosted-engine-setup/storage/nfs.py
+++ b/src/plugins/ovirt-hosted-engine-setup/storage/nfs.py
@@ -172,7 +172,11 @@
             raise RuntimeError(response['status']['message'])
         volumes = response['volumes']
         if volume not in volumes:
-            raise RuntimeError(_('GlusterFS Volume does not exist!'))
+            raise RuntimeError(
+                _('GlusterFS Volume {volume} does not exist!').format(
+                    volume=volume,
+                )
+            )
         if str(volumes[volume]['replicaCount']) != '3':
             raise RuntimeError(
                 _(
@@ -248,7 +252,7 @@
         if self.environment[
             ohostedcons.StorageEnv.DOMAIN_TYPE
         ] == ohostedcons.DomainTypes.GLUSTERFS:
-            self.logger.warning(
+            self.logger.info(
                 _(
                     'Please note that Replica 3 support is required for '
                     'the shared storage.'


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a8622ede6be78a4fa58b6cea3032975315024d6
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