diff options
author | Rich Trott <rtrott@gmail.com> | 2015-11-20 10:15:31 -0800 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2015-12-23 08:38:33 -0800 |
commit | ff2266a3deb59ca65c4c6ddaae99e6940d009325 (patch) | |
tree | 705e94632fd2e8d8a9c5fa18eb42571c654e5f3c /tools | |
parent | 9b41e7ef758ac4a9269b7b8bec223b20ecb313ce (diff) | |
download | nodejs-ff2266a3deb59ca65c4c6ddaae99e6940d009325.tar.gz nodejs-ff2266a3deb59ca65c4c6ddaae99e6940d009325.tar.bz2 nodejs-ff2266a3deb59ca65c4c6ddaae99e6940d009325.zip |
test: retry on smartos if ECONNREFUSED
SmartOS has a bug that causes unexpected ECONNREFUSED errors.
See https://smartos.org/bugview/OS-2767
If ECONNREFUSED on SmartOS, retry the test one time.
Fixes: https://github.com/nodejs/node/issues/3864
Fixes: https://github.com/nodejs/node/issues/2815
PR-URL: https://github.com/nodejs/node/pull/3941
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/test.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tools/test.py b/tools/test.py index 26b70233d..49db74727 100755 --- a/tools/test.py +++ b/tools/test.py @@ -138,6 +138,13 @@ class ProgressIndicator(object): try: start = datetime.now() output = case.Run() + # SmartOS has a bug that causes unexpected ECONNREFUSED errors. + # See https://smartos.org/bugview/OS-2767 + # If ECONNREFUSED on SmartOS, retry the test one time. + if (output.UnexpectedOutput() and + sys.platform == 'sunos5' and + 'ECONNREFUSED' in output.output.stderr): + output = case.Run() case.duration = (datetime.now() - start) except IOError, e: return |