-
Notifications
You must be signed in to change notification settings - Fork 299
KeyCloak integration tests failing on some OSX Dev machines #2505
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
base: main
Are you sure you want to change the base?
Conversation
…ssing port property
@@ -27,6 +27,8 @@ public class KeycloakProfile implements QuarkusTestProfile { | |||
@Override | |||
public Map<String, String> getConfigOverrides() { | |||
return Map.of( | |||
"quarkus.http.test-port", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property is already set here:
quarkus.http.test-port=0 |
I'm not sure adding it here will solve the original issue tbh.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know the property is in there but when I debugged the test that property is gone, so I assumed the override was removing it, even though it shouldn't be the case. Adding it back fixed the test on my machine. Can someone else check if they get the same result?
On my machine this at least gets us to the real exception
But does not fix the issue |
is this just running the integration test? Mine passes with no issues |
It seems it depends of the Docker "machine" you are using. The problem only occurs with Docker Desktop (I bet related to the Docker Desktop network stack). If you are using "native" Docker daemon, or podman, etc, there's no issue. |
See my notes on the issue, I now can pass without this change or any others. I just ran the full suite on the RC candidate and everything passed. |
Now that i'm running I still get failures, and on this branch as well. The same 403 error I posted above. |
Added missing port property in Keycloack profile for integration testing
#2501