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

test: test-http-dns-fail should be made non-flaky #25673

Closed
joaocgreis opened this issue Jul 13, 2015 · 7 comments
Closed

test: test-http-dns-fail should be made non-flaky #25673

joaocgreis opened this issue Jul 13, 2015 · 7 comments
Assignees
Milestone

Comments

@joaocgreis
Copy link
Member

simple/test-http-dns-fail is currently marked as flaky on Solaris. It should be investigated and made non-flaky.

Quick command for testing:

time (OK=0; NOK=0; for i in `seq 10000`; do ./out/Release/node test/internet/test-http-dns-fail.js && OK=$(($OK+1)) || NOK=$(($NOK+1)); echo "$i   OK: $OK   NOT OK: $NOK"; done)

Links:

/cc @joyent/node-collaborators

@joaocgreis
Copy link
Member Author

I cannot see this test fail on SmartOS x86 with the current v0.12. More results would be welcome.

@misterdjules
Copy link

@joaocgreis Thanks for creating this issue. I had created #8056 a while ago about the same problem, let me check if our SmartOS Jenkins node have actually been switched to Google DNS.

@misterdjules
Copy link

@joaocgreis The DNS configuration has indeed been changed to use DNS servers as mentioned in that issue a few months after the test was marked flaky. If you weren't able to reproduce this failure, I would just remove the test from the flaky list.

@joaocgreis
Copy link
Member Author

@misterdjules Thanks! I'm preparing a PR, I will include this.

@joaocgreis
Copy link
Member Author

@misterdjules For test-dns you marked #9034 as a duplicate of the issue you mentioned above. I cannot get it to fail on Linux, Windows or SmartOS. Can I assume it was also fixed by the DNS configuration?

@misterdjules
Copy link

@joaocgreis Yes, it is fair to assume this was fixed by the DNS configuration. You could even change the DNS configuration on the SmartOS agents to point to 209.244.0.3 (the level3 DNS server mentioned in my original description of the problem) in addition to 8.8.8.8 and see if you can reproduce the problem, but I would not bother.

The worst thing that can happen is that the test will fail again at some point, and we'll add it to the flaky list again, and start the investigation from where we left it now.

joaocgreis added a commit that referenced this issue Jul 16, 2015
- `test-crypto-domains` was fixed by
2afa3d8

- All tests under linux appear to be fixed and have not failed recently
on Jenkins

- `test-http-dns-fail` was fixed by the DNS configuration change
mentioned in #8056

Fixes #25656
Fixes #25673

Reviewed-By: Alexis Campailla <[email protected]>
PR-URL: #25676
@joaocgreis
Copy link
Member Author

Unmarked flaky in 78d256e.

jBarz pushed a commit to ibmruntimes/node that referenced this issue Nov 4, 2016
- `test-crypto-domains` was fixed by
nodejs/node-v0.x-archive@2afa3d8

- All tests under linux appear to be fixed and have not failed recently
on Jenkins

- `test-http-dns-fail` was fixed by the DNS configuration change
mentioned in nodejs#8056

Fixes nodejs#25656
Fixes nodejs#25673

Reviewed-By: Alexis Campailla <[email protected]>
PR-URL: nodejs#25676
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants