opendev/elastic-recheck
Code Issues Proposed changes

The output file is optional. Default to stdout if not specified.

Browse Source 
Avoid python stacktrace when omitting the output file.
Write to stdout by default.

Change-Id: I7e8bc7b2c49402786ddc20dca353bff6269ed594
This commit is contained in:
Ramy Asselin 2015-11-11 16:36:35 -08:00 committed by Matt Riedemann
parent b9946fded5
commit bce164eb66
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
elastic_recheck/cmd/graph.py
View File

@ -19,6 +19,7 @@ import ConfigParser
from datetime import datetime
import json
import os
import sys
from launchpadlib import launchpad
import pytz
@ -81,7 +82,7 @@ def main():
parser.add_argument(dest='queries',
help='path to query file')
parser.add_argument('-o', dest='output',
help='output filename')
help='output filename. Omit for stdout')
parser.add_argument('-q', dest='queue',
help='limit results to a specific build queue')
parser.add_argument('-c', '--conf', help="Elastic Recheck Configuration "
@ -197,9 +198,15 @@ def main():
key=lambda bug: -(bug['fails24'] * 100000 + bug['fails']))
jsondata['buglist'] = buglist
out = open(args.output, 'w')
out.write(json.dumps(jsondata))
out.close()
if args.output:
out = open(args.output, 'w')
else:
out = sys.stdout
try:
out.write(json.dumps(jsondata))
finally:
out.close()
if __name__ == "__main__":