From b40453a7cac02b6d2faaf6eae2155e44d05717c3 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Tue, 19 Dec 2017 14:41:31 +0900 Subject: [PATCH] abandon_stale: Only log change owner in verbose mode Change-Id: I211d1e39e6d4081a83f9770d1d06c6d4ea56b313 --- contrib/abandon_stale.py | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/contrib/abandon_stale.py b/contrib/abandon_stale.py index cb1cc23e6d..c542e7af07 100755 --- a/contrib/abandon_stale.py +++ b/contrib/abandon_stale.py @@ -186,15 +186,18 @@ def _main(): abandon_message += "\n\n" + options.message for change in stale_changes: number = change["_number"] - try: - owner = change["owner"]["name"] - except: - owner = "Unknown" + owner = "" + if options.verbose: + try: + o = change["owner"]["name"] + except KeyError: + o = "Unknown" + owner = " (%s)" % o subject = change["subject"] if len(subject) > 70: subject = subject[:65] + " [...]" change_id = change["id"] - logging.info("%s (%s): %s", number, owner, subject) + logging.info("%s%s: %s", number, owner, subject) if options.dry_run: continue