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

Add basic IPv6 support to i386 docker runners #224

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abrender
Copy link

@abrender abrender commented Mar 20, 2025

Github runners do not natively support IPv6 [1], which makes it hard to
test IPv6 code, even when external IPv6 connectivity is not needed.

This simple change adds IPv6 support to the docker containers along
with an IPv6 address assigned from a ULA (Unique Local Address) subnet.

There is still no external IPv6 connectivity, but local IPv6 tests can
now be run in the docker containers.

This new IPv6 functionality will be used to re-enable IPv6 tests in
the mdns repo, which are currently broken due to lack of IPv6 in
docker.

https://github.com/abrender/mdns/actions/runs/13980622847 shows an
example of how IPv6 tests can now pass with i386 docker containers.

[1] actions/runner-images#668

@abrender abrender marked this pull request as ready for review March 20, 2025 22:36
Github runners do not natively support IPv6 [1], which makes it hard to
test IPv6 code, even when external IPv6 connectivity is not needed.

This simple change adds IPv6 support to the docker containers along with
an IPv6 address assigned from a ULA (Unique Local Address) subnet.

There is still no external IPv6 connectivity, but local IPv6 tests can
now be run in the docker containers.

This new IPv6 functionality will be used to re-enable IPv6 tests in the
`mdns` repo, which are currently broken due to lack of IPv6 in docker.

[1] actions/runner-images#668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant