-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Disable Dev Services fully in oidc-wiremock-logout IT #47511
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
They are not required and it's failing on Windows as Docker is not available.
Status for workflow
|
@@ -1,4 +1,4 @@ | |||
quarkus.keycloak.devservices.enabled=false | |||
quarkus.devservices.enabled=false |
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.
That appears to disable it completely for all platforms
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.
Never mind, looks like they are indeed not required for this test - though a more general fix in the linked seems to work better in a sense where we want to check if a lightweight OIDC service should be used, so that PR is needed anyway.
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.
LGTM, please review the linked PR as well to support a case of actually running lightweight OIDC dev service on Windows
They are not required and it's failing on Windows as Docker is not available.