Change in vdsm[master]: bondOpts: Fix reporting [] instead of [''] for empty opts

asegurap at redhat.com asegurap at redhat.com
Fri Nov 22 16:40:28 UTC 2013


Antoni Segura Puimedon has uploaded a new change for review.

Change subject: bondOpts: Fix reporting [] instead of [''] for empty opts
......................................................................

bondOpts: Fix reporting [] instead of [''] for empty opts

Change-Id: I20963543ec6b0f9ff42c02431ec9758d7de0531e
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
---
M lib/vdsm/netinfo.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/54/21554/1

diff --git a/lib/vdsm/netinfo.py b/lib/vdsm/netinfo.py
index dd54a96..28643e8 100644
--- a/lib/vdsm/netinfo.py
+++ b/lib/vdsm/netinfo.py
@@ -214,7 +214,8 @@
     opts = {}
     for path in paths:
         with open(path) as optFile:
-            opts[os.path.basename(path)] = optFile.read().rstrip().split(' ')
+            opts[os.path.basename(path)] = [
+                el for el in optFile.read().rstrip().split(' ') if el]
     return opts
 
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I20963543ec6b0f9ff42c02431ec9758d7de0531e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>


More information about the vdsm-patches mailing list