Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: add help on fixing IPv6 test failures #24372

Closed
wants to merge 7 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
doc: add help on fixing IPv6 test failures
It took me a while to figure out the problem and
then some googling to find the right answer.  I think
it is worth adding this to help other people in the
future and to have an easy place to point people to
for the solution if their test run fails with IPv6
failures.
  • Loading branch information
mhdawson committed Nov 14, 2018
commit db56b46aa7c9c138152d8cbdf3c52c81753c0cef
8 changes: 8 additions & 0 deletions BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -253,6 +253,14 @@ $ ./node ./test/parallel/test-stream2-transform.js
Remember to recompile with `make -j4` in between test runs if you change code in
the `lib` or `src` directories.

The tests attempt to detect support for IPv6 and exclude IPv6 tests if
appropriate. However, if your main interface has IPv6 addresess then your
mhdawson marked this conversation as resolved.
Show resolved Hide resolved
loopback interface must also have '::1' enabled. For some default installations
on ubuntu that does not seem to be the case. '::1' can be enabled on the
mhdawson marked this conversation as resolved.
Show resolved Hide resolved
loopback interface on ubuntu using:
mhdawson marked this conversation as resolved.
Show resolved Hide resolved

`sudo sysctl -w net.ipv6.conf.lo.disable_ipv6=0`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: It would be better if we formatted this as console text, instead of inline code.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


#### Running Coverage

It's good practice to ensure any code you add or change is covered by tests.
Expand Down