Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

test: Extend timeout due to failure on AIX #25138

Closed
wants to merge 1 commit into from
Closed

test: Extend timeout due to failure on AIX #25138

wants to merge 1 commit into from

Conversation

mhdawson
Copy link
Member

@mhdawson mhdawson commented May 6, 2015

The loopback on AIX is slower by default than on other platforms
and we've seen a number of tests fail on AIX for this reason. This
looks to be another instance. Changing the test to bind to the
host ip instead of the loopback makes it pass reliably.

This change extends the timeout so that it passes reliably on AIX
even with the slower loopback behaviour

modified:   test/simple/test-tls-wrap-timeout.js

The loopback on AIX is slower by default than on other platforms
and we've seen a number of tests fail on AIX for this reason. This
looks to be another instance.  Changing the test to bind to the
host ip instead of the loopback makes it pass reliably.

This change extends the timeout so that it passes reliably on AIX
even with the slower loopback behaviour

	modified:   test/simple/test-tls-wrap-timeout.js
@cjihrig
Copy link

cjihrig commented May 6, 2015

LGTM

mhdawson added a commit that referenced this pull request May 6, 2015
The loopback on AIX is slower by default than on other platforms
and we've seen a number of tests fail on AIX for this reason. This
looks to be another instance.  Changing the test to bind to the
host ip instead of the loopback makes it pass reliably.

This change extends the timeout so that it passes reliably on AIX
even with the slower loopback behaviour

	modified:   test/simple/test-tls-wrap-timeout.js

Reviewed-By: coln Ihrig <[email protected]>
PR-URL: #25138
@mhdawson
Copy link
Member Author

mhdawson commented May 7, 2015

Landed as fcbf226

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants