diff options
author | Andy Doan <andy.doan@linaro.org> | 2015-03-16 17:32:46 -0500 |
---|---|---|
committer | Linaro Code Review <review@review.linaro.org> | 2015-03-17 18:55:06 +0000 |
commit | 3d8873d47d040bd3597d1249d4977f9a83740b67 (patch) | |
tree | 542aeff0a1e086ec407a62e5ff22c9d10ef81ad8 /grok-cron.py | |
parent | 99543a5c13d1d2807dad0d66543ff9f91a76f3fb (diff) |
grok: fix logging to be friendly with cron
This script was always logging to stdout a single INFO message.
The best logging level to use for this from cron (so there's only
output upon an error) is WARN.
Change-Id: I065495b895277ea5166c9ff7f44f07cd718df687
Diffstat (limited to 'grok-cron.py')
-rwxr-xr-x | grok-cron.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/grok-cron.py b/grok-cron.py index cc0b464..997eb5a 100755 --- a/grok-cron.py +++ b/grok-cron.py @@ -47,13 +47,12 @@ if __name__ == '__main__': help='location of repositories') parser.add_argument('-n', '--dryrun', action='store_true', help='Make no changes to manifest') - parser.add_argument('-v', '--verbose', action='store_true', - help='Verbose output') + parser.add_argument('--log', default='WARN', + choices=('WARN', 'INFO', 'DEBUG'), + help='Logging level to use. Default=%(default)s') args = parser.parse_args() - if args.verbose: - log.setLevel(logging.DEBUG) - else: - log.setLevel(logging.INFO) + for l in logging.getLogger().manager.loggerDict.keys(): + logging.getLogger(l).setLevel(getattr(logging, args.log)) main(args.manifest, args.toplevel, args.dryrun) |