Change in vdsm[master]: vdsm-reg: avoid issuing errors to console

Alon Bar-Lev alonbl at redhat.com
Wed Apr 3 14:10:09 UTC 2013


Alon Bar-Lev has uploaded a new change for review.

Change subject: vdsm-reg: avoid issuing errors to console
......................................................................

vdsm-reg: avoid issuing errors to console

Using logging.error() during the ovirt-node dialog results in writing
messages to console. This is undesired in dialog mode.

Change the error to debug to avoid.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=947905
Change-Id: Ib25f516b74dc03cca4fe696467121c8e8db06b76
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M vdsm_reg/deployUtil.py.in
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/61/13561/1

diff --git a/vdsm_reg/deployUtil.py.in b/vdsm_reg/deployUtil.py.in
index f2bef9d..3b056ca 100644
--- a/vdsm_reg/deployUtil.py.in
+++ b/vdsm_reg/deployUtil.py.in
@@ -1720,7 +1720,7 @@
 
         return True
     except:
-        logging.error(
+        logging.debug(
             "Failed acquiring web certificates",
             exc_info=True
         )
@@ -1736,10 +1736,10 @@
             fp = filter(lambda l: 'Fingerprint' in
                         l, out.split())[0].split('=')[1]
         except Exception:
-            logging.error("Failed generating finger print for %s", path,
+            logging.debug("Failed generating finger print for %s", path,
                           exc_info=True)
     else:
-        logging.error("Command %s failed with return value %d",
+        logging.debug("Command %s failed with return value %d",
                       ' '.join(cmd), rc)
 
     return fp


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib25f516b74dc03cca4fe696467121c8e8db06b76
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>


More information about the vdsm-patches mailing list