Make all tools log with their name
authorHelga Velroyen <helgav@google.com>
Tue, 15 Dec 2015 10:39:16 +0000 (11:39 +0100)
committerHelga Velroyen <helgav@google.com>
Thu, 7 Jan 2016 13:33:24 +0000 (14:33 +0100)
The first patch of this series has introduced the option
of naming the tool that is logging to tools.log. This
patch updates all calls to SetupToolsLogging to
provide a toolname.

Signed-off-by: Helga Velroyen <helgav@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>

lib/tools/ensure_dirs.py
lib/tools/node_cleanup.py
lib/tools/node_daemon_setup.py
lib/tools/prepare_node_join.py
lib/tools/ssh_update.py
lib/tools/ssl_update.py
tools/cluster-merge
tools/move-instance
tools/ovfconverter

index 0a197ba..66b37e8 100644 (file)
@@ -250,7 +250,9 @@ def Main():
   """
   (opts, args) = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   if args:
     logging.error("No arguments are expected")
index f8ec076..08a9548 100644 (file)
@@ -80,7 +80,9 @@ def Main():
   """
   opts = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   try:
     # List of files to delete. Contains tuples consisting of the absolute path
index 6ade4bd..c971d15 100644 (file)
@@ -118,7 +118,9 @@ def Main():
   """
   opts = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   try:
     getent = runtime.GetEnts()
index fa45a58..0a0e2c8 100644 (file)
@@ -195,7 +195,9 @@ def Main():
   """
   opts = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   try:
     data = common.LoadData(sys.stdin.read(), _DATA_CHECK)
index b37972e..23f5077 100644 (file)
@@ -210,7 +210,9 @@ def Main():
   """
   opts = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   try:
     data = common.LoadData(sys.stdin.read(), _DATA_CHECK)
index 56e8d6a..05be975 100644 (file)
@@ -114,7 +114,9 @@ def Main():
   """
   opts = ParseOptions()
 
-  utils.SetupToolLogging(opts.debug, opts.verbose)
+  utils.SetupToolLogging(
+      opts.debug, opts.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   try:
     data = common.LoadData(sys.stdin.read(), _DATA_CHECK)
index 926b705..8af20df 100755 (executable)
@@ -807,7 +807,9 @@ def main():
 
   (options, args) = parser.parse_args()
 
-  utils.SetupToolLogging(options.debug, options.verbose)
+  utils.SetupToolLogging(
+      options.debug, options.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   if not args:
     parser.error("No clusters specified")
index 8913f62..5507a86 100755 (executable)
@@ -1033,7 +1033,9 @@ def main():
   """
   (parser, options, args) = ParseOptions()
 
-  utils.SetupToolLogging(options.debug, options.verbose, threadname=True)
+  utils.SetupToolLogging(
+      options.debug, options.verbose, threadname=True,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   (src_cluster_name, dest_cluster_name, instance_names) = \
     CheckOptions(parser, options, args)
index ba437c7..f13a3a9 100755 (executable)
@@ -177,7 +177,9 @@ def main():
   """
   (mode, input_path, options) = ParseOptions()
 
-  utils.SetupToolLogging(options.debug, options.verbose)
+  utils.SetupToolLogging(
+      options.debug, options.verbose,
+      toolname=os.path.splitext(os.path.basename(__file__))[0])
 
   logging.info("Chosen %s mode, reading the %s file", mode, input_path)
   assert mode in (IMPORT_MODE, EXPORT_MODE)