task_runner: always use run_isolated

client/utils/logging_utils.py

 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 """Utility relating to logging."""
 
 import argparse
 import codecs
 import ctypes
 import logging
 import logging.handlers
 import optparse
 import os
 import sys
 import tempfile
 import time
 
+from utils import file_path
 
 # This works around file locking issue on Windows specifically in the case of
 # long lived child processes.
 #
 # Python opens files with inheritable handle and without file sharing by
 # default. This causes the RotatingFileHandler file handle to be duplicated in
 # the subprocesses even if the log file is not used in it. Because of this
 # handle in the child process, when the RotatingFileHandler tries to os.rename()
 # the file in the parent process, it fails with:
 #     WindowsError: [Error 32] The process cannot access the file because
@@ skipping 164 matching lines @@
   stderr = logging.StreamHandler()
   stderr.setFormatter(formatter)
   stderr.addFilter(Filter())
   # Default to ERROR.
   stderr.setLevel(logging.ERROR)
   logger.addHandler(stderr)
 
   # Setup up logging to a constant file so we can debug issues where
   # the results aren't properly sent to the result URL.
   if filename:
+    file_path.ensure_tree(os.path.dirname(os.path.abspath(unicode(filename))))
     try:
       rotating_file = NoInheritRotatingFileHandler(
           filename, maxBytes=10 * 1024 * 1024, backupCount=5,
           encoding='utf-8')
       rotating_file.setLevel(logging.DEBUG)
       rotating_file.setFormatter(formatter)
       rotating_file.addFilter(Filter())
       logger.addHandler(rotating_file)
     except Exception:
       # May happen on cygwin. Do not crash.
@@ skipping 79 matching lines @@
 
   def parse_args(self, *args, **kwargs):
     # Make sure this group is always the last one.
     self._add_logging_group()
 
     args = super(ArgumentParserWithLogging, self).parse_args(*args, **kwargs)
     prepare_logging(self.enable_log_file and args.log_file, self.logger_root)
     set_console_level(
         LEVELS[min(len(LEVELS) - 1, args.verbose)], self.logger_root)
     return args