Hello Greg Padgett,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/22341

to review the following change.

Change subject: Fix --vm-status when maintenance mode has never been enabled
......................................................................

Fix --vm-status when maintenance mode has never been enabled

Change-Id: I2b1a4f91e826a95d22a4e8b93a044aec2baf9003
Bug-Url: https://bugzilla.redhat.com/1041203
Signed-off-by: Greg Padgett <gpadg...@redhat.com>
---
M src/ovirt_hosted_engine_setup/vm_status.py
1 file changed, 1 insertion(+), 1 deletion(-)


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

diff --git a/src/ovirt_hosted_engine_setup/vm_status.py 
b/src/ovirt_hosted_engine_setup/vm_status.py
index 20866e3..a64ef0a 100644
--- a/src/ovirt_hosted_engine_setup/vm_status.py
+++ b/src/ovirt_hosted_engine_setup/vm_status.py
@@ -68,7 +68,7 @@
         except KeyError:
             # Stats were retrieved but the global section is missing.
             # This is not an error.
-            pass
+            cluster_stats = {}
         except (socket.error, AttributeError, IndexError):
             sys.stderr.write(
                 _('Cannot connect to the HA daemon, please check the logs.\n')


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

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

Reply via email to