Add --ignoreErrors argument to be able to bypass some errors
This commit is contained in:
parent
f3685701ac
commit
fa32770e96
|
@ -45,6 +45,10 @@ parser.add_argument('-e',
|
|||
'--expecting',
|
||||
type=int,
|
||||
help='the expected number of artifacts. If omitted, no check will be done.')
|
||||
parser.add_argument('-i',
|
||||
'--ignoreErrors',
|
||||
help='Ignore errors that can be ignored. Build state and number of artifacts.',
|
||||
action="store_true")
|
||||
parser.add_argument('-d',
|
||||
'--directory',
|
||||
default="",
|
||||
|
@ -91,8 +95,13 @@ print(" git commit : \"%s\"" % data0.get('commit'))
|
|||
print(" git commit message : \"%s\"" % data0.get('message'))
|
||||
print(" build state : %s" % data0.get('state'))
|
||||
|
||||
error = False
|
||||
|
||||
if data0.get('state') != 'passed':
|
||||
print("❌ Error, the build is in state '%s', and not 'passed'" % data0.get('state'))
|
||||
if args.ignoreErrors:
|
||||
error = True
|
||||
else:
|
||||
exit(1)
|
||||
|
||||
### Fetch artifacts list
|
||||
|
@ -110,7 +119,10 @@ data = json.loads(r.content.decode())
|
|||
print(" %d artifact(s) found." % len(data))
|
||||
|
||||
if args.expecting is not None and args.expecting != len(data):
|
||||
print("Error, expecting %d artifacts and found %d." % (args.expecting, len(data)))
|
||||
print("❌ Error, expecting %d artifacts and found %d." % (args.expecting, len(data)))
|
||||
if args.ignoreErrors:
|
||||
error = True
|
||||
else:
|
||||
exit(1)
|
||||
|
||||
if args.verbose:
|
||||
|
@ -128,8 +140,6 @@ else:
|
|||
if not args.simulate:
|
||||
os.mkdir(targetDir)
|
||||
|
||||
error = False
|
||||
|
||||
for elt in data:
|
||||
if args.verbose:
|
||||
print()
|
||||
|
@ -157,7 +167,7 @@ for elt in data:
|
|||
print("❌ Checksum mismatch: expecting %s and get %s" % (elt.get("sha1sum"), hash))
|
||||
|
||||
if error:
|
||||
print("❌ Error(s) occurred, check the log")
|
||||
print("❌ Error(s) occurred, please check the log")
|
||||
exit(1)
|
||||
else:
|
||||
print("Done!")
|
||||
|
|
Loading…
Reference in New Issue