Change in vdsm[master]: tool: Use space after command when formatting lists

nsoffer at redhat.com nsoffer at redhat.com
Thu Aug 21 16:52:24 UTC 2014


Nir Soffer has uploaded a new change for review.

Change subject: tool: Use space after command when formatting lists
......................................................................

tool: Use space after command when formatting lists

When formating list of modules in error message, we used to do:

    foo,bar,baz

Now lists are formatted in more readable way:

    foo, bar, baz

Change-Id: I88d742b72ba57ad1de481a589ca53e71c8202176
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M lib/vdsm/tool/configurator.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/31784/1

diff --git a/lib/vdsm/tool/configurator.py b/lib/vdsm/tool/configurator.py
index 268ae46..12cc0af 100644
--- a/lib/vdsm/tool/configurator.py
+++ b/lib/vdsm/tool/configurator.py
@@ -101,7 +101,7 @@
 
     if m:
         sys.stdout.write(
-            "Modules %s are not configured\n " % ','.join(m),
+            "Modules %s are not configured\n " % ', '.join(m),
         )
         ret = False
 
@@ -135,7 +135,7 @@
 
     if m:
         sys.stdout.write(
-            "Modules %s contains invalid configuration\n " % ','.join(m),
+            "Modules %s contains invalid configuration\n " % ', '.join(m),
         )
         ret = False
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I88d742b72ba57ad1de481a589ca53e71c8202176
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list