cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tall...@apache.org
Subject git commit: updated refs/heads/4.3 to b52a641
Date Thu, 12 Dec 2013 11:39:55 GMT
Updated Branches:
  refs/heads/4.3 7151ec63f -> b52a641de


CLOUDSTACK-5449: fixed pep8 errors
Signed-off-by: SrikanteswaraRao Talluri <talluri@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b52a641d
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b52a641d
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b52a641d

Branch: refs/heads/4.3
Commit: b52a641decfbf487bdf84b6037640aa42682c55b
Parents: 7151ec6
Author: SrikanteswaraRao Talluri <talluri@apache.org>
Authored: Thu Dec 12 17:06:31 2013 +0530
Committer: SrikanteswaraRao Talluri <talluri@apache.org>
Committed: Thu Dec 12 17:09:31 2013 +0530

----------------------------------------------------------------------
 tools/marvin/marvin/marvinInit.py   |  8 +++++---
 tools/marvin/marvin/marvinPlugin.py | 21 ++++++++++++++-------
 2 files changed, 19 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b52a641d/tools/marvin/marvin/marvinInit.py
----------------------------------------------------------------------
diff --git a/tools/marvin/marvin/marvinInit.py b/tools/marvin/marvin/marvinInit.py
index 2bbf119..f722058 100644
--- a/tools/marvin/marvin/marvinInit.py
+++ b/tools/marvin/marvin/marvinInit.py
@@ -118,7 +118,8 @@ class MarvinInit:
                 log_config = self.__parsedConfig.logger
                 if log_config is not None:
                     if log_config.LogFolderPath is not None:
-                        self.logFolderPath = log_config.LogFolderPath + '/' + temp_path
+                        self.logFolderPath = log_config.LogFolderPath + '/' \
+                            + temp_path
                     else:
                         self.logFolderPath = temp_path
                 else:
@@ -126,8 +127,9 @@ class MarvinInit:
             else:
                 self.logFolderPath = self.__logFolderPath + '/' + temp_path
             if os.path.exists(self.logFolderPath):
-                self.logFolderPath = self.logFolderPath \
-                                     + ''.join(random.choice(string.ascii_uppercase + string.digits)
for x in range(3))
+                self.logFolderPath += ''.join(random.choice(
+                    string.ascii_uppercase +
+                    string.digits for x in range(3)))
             os.makedirs(self.logFolderPath)
             '''
             Log File Paths

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b52a641d/tools/marvin/marvin/marvinPlugin.py
----------------------------------------------------------------------
diff --git a/tools/marvin/marvin/marvinPlugin.py b/tools/marvin/marvin/marvinPlugin.py
index 1353352..df7d7a3 100644
--- a/tools/marvin/marvin/marvinPlugin.py
+++ b/tools/marvin/marvin/marvinPlugin.py
@@ -80,17 +80,22 @@ class MarvinPlugin(Plugin):
         Register command line options
         """
         parser.add_option("--marvin-config", action="store",
-                          default=env.get('MARVIN_CONFIG', './datacenter.cfg'),
+                          default=env.get('MARVIN_CONFIG',
+                                          './datacenter.cfg'),
                           dest="config_file",
                           help="Marvin's configuration file where the " +
-                               "datacenter information is specified " +
-                               "[MARVIN_CONFIG]")
-        parser.add_option("--load", action="store_true", default=False,
+                               "datacenter information is specified" +
+                               " [MARVIN_CONFIG]")
+        parser.add_option("--load", action="store_true",
+                          default=False,
                           dest="load",
                           help="Only load the deployment configuration given")
-        parser.add_option("--log-folder-path", action="store", default=None,
+        parser.add_option("--log-folder-path",
+                          action="store",
+                          default=None,
                           dest="log_folder_path",
-                          help="Path to the folder where log files will be stored")
+                          help="Path to the folder "
+                               "where log files will be stored")
         Plugin.options(self, parser, env)
 
     def wantClass(self, cls):
@@ -169,7 +174,9 @@ class MarvinPlugin(Plugin):
         Creates a debugstream for tc debug log
         '''
         try:
-            obj_marvininit = MarvinInit(self.configFile, self.loadFlag, self.logFolderPath)
+            obj_marvininit = MarvinInit(self.configFile,
+                                        self.loadFlag,
+                                        self.logFolderPath)
             if obj_marvininit.init() == SUCCESS:
                 self.testClient = obj_marvininit.getTestClient()
                 self.tcRunLogger = obj_marvininit.getLogger()


Mime
View raw message