findmissing: Add timestamp to logging information A complete robot run now takes more than three hours. Add timestamp information to logging data to help tracking down the reason. To fix a new upload error, use "from" keyword when re-raising an exception. BUG=None TEST=Run robot and observe logging data Change-Id: Idaa34c73ec1d642c0c5943269175ac7ffe80d612 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/2618622 Reviewed-by: Curtis Malainey <cujomalainey@chromium.org> Commit-Queue: Curtis Malainey <cujomalainey@chromium.org> Tested-by: Guenter Roeck <groeck@chromium.org> Auto-Submit: Guenter Roeck <groeck@chromium.org>
diff --git a/contrib/findmissing/util.py b/contrib/findmissing/util.py index ab21496..27d4503 100755 --- a/contrib/findmissing/util.py +++ b/contrib/findmissing/util.py
@@ -42,8 +42,8 @@ try: subprocess.run(process_grep, check=True, stdout=subprocess.DEVNULL, encoding='utf-8', errors='ignore') - except subprocess.CalledProcessError: - raise ProcessLookupError('Service %s is not running.' % keyword) + except subprocess.CalledProcessError as e: + raise ProcessLookupError('Service %s is not running.' % keyword) from e def check_cloud_sql_proxy_running(): @@ -76,7 +76,8 @@ else: level = logging.WARNING - logging.basicConfig(format='%(levelname)s: %(message)s', level=level) + logging.basicConfig(format='%(asctime)s %(levelname)s: %(message)s', level=level, + datefmt='%Y-%m-%d %H:%M:%S') f(*args) return wrapped_preliminary_check