[GH-ISSUE #266] Enable IPv6-specific tests by default, disable explicitly when running on Travis CI #425

New issue

Open

opened 2026-03-15 22:26:45 +03:00 by kerem · 2 comments

kerem commented

2026-03-15 22:26:45 +03:00

Owner

Originally created by @briansmith on GitHub (Oct 25, 2017).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/266

Currently many IPv6-related tests are disabled because Travis CI doesn't support IPv6 (see https://github.com/travis-ci/travis-ci/issues/5200 and related issues in their issue tracker). However, it should be made possible to enable IPv6 testing, and it should be enabled by default. Only on Travis CI should we disable IPv6 testing.

Search for // ignored until Travis-CI fixes IPv6 to find the relevant tests.

I suggest that we have a "disable-ipv6-tests" feature that, when enabled, disables the tests. I know that's not how features are supposed to work, but that's really the better semantics for this workaround. However, I'm not wedded to doing things this way; we can do things the recommended way (have a "ipv6-tests" default feature) if people prefer.

Originally created by @briansmith on GitHub (Oct 25, 2017). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/266 Currently many IPv6-related tests are disabled because Travis CI doesn't support IPv6 (see https://github.com/travis-ci/travis-ci/issues/5200 and related issues in their issue tracker). However, it should be made possible to enable IPv6 testing, and it should be enabled by default. Only on Travis CI should we disable IPv6 testing. Search for `// ignored until Travis-CI fixes IPv6` to find the relevant tests. I suggest that we have a "disable-ipv6-tests" feature that, when enabled, disables the tests. I know that's not how features are supposed to work, but that's really the better semantics for this workaround. However, I'm not wedded to doing things this way; we can do things the recommended way (have a "ipv6-tests" default feature) if people prefer.

kerem added the

crate:all

tools

labels

2026-03-15 22:26:45 +03:00

kerem commented

2026-03-15 22:26:56 +03:00

Author

Owner

@bluejekyll commented on GitHub (Oct 25, 2017):

btw, if I remember correctly, this may only be ipv6 bound local addresses, ipv6 local -> ipv6 local trust-dns server, and ipv6 local -> ipv6 remote.

IPv6 lookups, are currently enabled by default.

@bluejekyll commented on GitHub (Oct 25, 2017): btw, if I remember correctly, this may only be ipv6 bound local addresses, ipv6 local -> ipv6 local trust-dns server, and ipv6 local -> ipv6 remote. IPv6 lookups, are currently enabled by default.

kerem commented

2026-03-15 22:27:01 +03:00

Author

Owner

@bluejekyll commented on GitHub (Apr 20, 2021):

Travis is no longer in use, we should review if ipv6 tests are enabled on github actions.

@bluejekyll commented on GitHub (Apr 20, 2021): Travis is no longer in use, we should review if ipv6 tests are enabled on github actions.

kerem referenced this issue

2026-03-16 02:05:57 +03:00

[PR #425] [MERGED] Cleanup travis #1429