Darshan N has uploaded a new change for review.

Change subject: ovirt-host-deploy:Fix: Password to be asked before engine log 
collection.
......................................................................

ovirt-host-deploy:Fix: Password to be asked before engine log collection.

The Rest-Api password to be asked before collecting the engine log
whenever needed. Currently the password is asked after collecting
engine and postgres data.

Change-Id: I5e49f3196f5dc5076fe784d7819160dba24e0184
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1098029
Signed-off-by: ndarshan <dnara...@redhat.com>
---
M src/__main__.py
1 file changed, 6 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-log-collector 
refs/changes/65/28065/1

diff --git a/src/__main__.py b/src/__main__.py
index c1a4d86..17d3617 100755
--- a/src/__main__.py
+++ b/src/__main__.py
@@ -1662,16 +1662,19 @@
 The directory is: %s'""" % (conf["local_scratch_dir"]))
 
         if conf.command == "collect":
-            collector.get_engine_data()
-            collector.get_postgres_data()
             if not conf.get("no_hypervisor"):
-                if collector.set_hosts():
+                hosts_present = collector.set_hosts()
+                collector.get_engine_data()
+                collector.get_postgres_data()
+                if hosts_present:
                     collector.get_hypervisor_data()
                 else:
                     logging.info(
                         "No hypervisors were selected, therefore no \
 hypervisor data will be collected.")
             else:
+                collector.get_engine_data()
+                collector.get_postgres_data()
                 logging.info("Skipping hypervisor collection...")
             stdout = collector.archive()
             logging.info(stdout)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e49f3196f5dc5076fe784d7819160dba24e0184
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnara...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to