summaryrefslogtreecommitdiffstats
path: root/test/tools
diff options
context:
space:
mode:
authorAnton Yuzhaninov <citrin+git@citrin.ru>2019-08-13 15:47:00 +0100
committerAnton Yuzhaninov <citrin+git@citrin.ru>2019-08-13 15:47:00 +0100
commit46727d273491d5ce2b7e74cd1b1f4501607db28f (patch)
treefc11d5608c2ca7ea7054a39853edf34dd475d14f /test/tools
parent6597e51e5a56e7285a41041bb4b80b98ef4b8773 (diff)
downloadrspamd-46727d273491d5ce2b7e74cd1b1f4501607db28f.tar.gz
rspamd-46727d273491d5ce2b7e74cd1b1f4501607db28f.zip
Ignore coveralls API errors
Exiting with zero exit code in case of errors in bad style, but we do not want to fail CI test run if coveralls.io is down. For other error (e. g. internal scrip errors) exit code still be non zero.
Diffstat (limited to 'test/tools')
-rwxr-xr-xtest/tools/merge_coveralls.py23
1 files changed, 14 insertions, 9 deletions
diff --git a/test/tools/merge_coveralls.py b/test/tools/merge_coveralls.py
index 8fad0f55b..09380a386 100755
--- a/test/tools/merge_coveralls.py
+++ b/test/tools/merge_coveralls.py
@@ -160,12 +160,17 @@ if __name__ == '__main__':
if args.token:
j1['repo_token'] = args.token
- print("sending data to coveralls...")
- r = requests.post('https://coveralls.io/api/v1/jobs', files={"json_file": json.dumps(j1)})
- response = r.json()
- print("[coveralls] %s" % response['message'])
- if 'url' in response:
- print("[coveralls] Uploaded to %s" % response['url'])
-
- # post https://coveralls.io/api/v1/jobs
- # print args
+ try:
+ r = requests.post('https://coveralls.io/api/v1/jobs', files={"json_file": json.dumps(j1)})
+ r.raise_for_status()
+ except requests.exceptions.RequestException as e:
+ print("Failed to send data to coveralls: %s" % e)
+ sys.exit()
+
+ try:
+ response = r.json()
+ print("[coveralls] %s" % response['message'])
+ if 'url' in response:
+ print("[coveralls] Uploaded to %s" % response['url'])
+ except json.decoder.JSONDecodeError:
+ print("Bad resonse: '%s'" % r.text)