Skip to content

test: force IPv4 on Docker 26+ to fix Envoy int tests#20986

Merged
zalimeni merged 1 commit intomainfrom
zalimeni/fix-envoy-int-tests-docker-26-force-ipv4
Apr 17, 2024
Merged

test: force IPv4 on Docker 26+ to fix Envoy int tests#20986
zalimeni merged 1 commit intomainfrom
zalimeni/fix-envoy-int-tests-docker-26-force-ipv4

Conversation

@zalimeni
Copy link
Member

As of Docker Engine 26.0.0 (moby/moby#47062), IPv6 is enabled by default where supported. This causes issues for our tests attempting to resolve requests to other containers over localhost, since on Linux IPv6 will be preferred over IPv4 when available when applying the default behavior defined in RFC3484.

As a workaround, force IPv4 with a flag passed to docker run.

Testing & Reproduction steps

CI continues to pass (Docker Engine version is still <26 on runners used for CE builds).

Links

moby/moby#47062

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

As of Docker Engine 26.0.0 (moby/moby#47062),
IPv6 is enabled by default where supported. This causes issues for our
tests attempting to resolve requests to other containers over
localhost, since on Linux IPv6 will be preferred over IPv4 when
available when applying the default behavior defined in RFC3484.

As a workaround, force IPv4 with a flag passed to `docker run`.
@zalimeni zalimeni added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. backport/1.18 labels Apr 17, 2024
@zalimeni zalimeni enabled auto-merge (squash) April 17, 2024 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants