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

Make @QuarkusIntegrationTest work reliably with testcontainers launched services #18653

Merged
merged 1 commit into from
Jul 13, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Jul 13, 2021

As services launched with testcontainers (which are under user-control, not ours)
will use localhost as the host they present to the relevant Quarkus configuration,
the only way to reliably make the application container work with these, is to
use host networking.

See https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/testcontainers.20networking.20in.20docker.20mode for a thorough discussion

…ed services

As services launched with testcontainers (which are under user-control, not ours)
will use localhost as the host they present to the relevant Quarkus configuration,
the only way to reliably make the application container work with these, is to
use host networking
@geoand
Copy link
Contributor Author

geoand commented Jul 13, 2021

We will definitely want this in 2.1

@geoand geoand added triage/backport? triage/waiting-for-ci Ready to merge when CI successfully finishes and removed triage/backport? labels Jul 13, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Jul 13, 2021

Failing Jobs - Building ae770fb

Status Name Step Test failures Logs Raw logs
✔️ JVM Tests - JDK 11
JVM Tests - JDK 11 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 16

@geoand geoand merged commit c91510c into quarkusio:main Jul 13, 2021
@quarkus-bot quarkus-bot bot added this to the 2.1 - main milestone Jul 13, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 13, 2021
@geoand geoand deleted the qit-testresources branch December 15, 2021 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants