[node-patches] Change in ovirt-node[master]: media: Fix doctest, catch invalid return values

fabiand at redhat.com fabiand at redhat.com
Mon Jan 5 09:48:11 UTC 2015


Fabian Deutsch has uploaded a new change for review.

Change subject: media: Fix doctest, catch invalid return values
......................................................................

media: Fix doctest, catch invalid return values

Change-Id: Iccd5ad98fb341423465d622ce561a0a12aa3fab6
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/utils/system.py
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/58/36558/1

diff --git a/src/ovirt/node/utils/system.py b/src/ovirt/node/utils/system.py
index 5e45536..179b5db 100644
--- a/src/ovirt/node/utils/system.py
+++ b/src/ovirt/node/utils/system.py
@@ -23,7 +23,6 @@
 e.g. services, reboot ...
 """
 
-import glob
 import os
 import re
 import shlex
@@ -606,6 +605,9 @@
         data = get_installed_version_number()
         if data:
             self.version, self.release = data
+        else:
+            LOGGER.debug("Failed to retrieve installed media " +
+                         "version: %s" % data)
 
     def available(self):
         """Determin if there is an installed media


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iccd5ad98fb341423465d622ce561a0a12aa3fab6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>



More information about the node-patches mailing list