Skip to content

Conversation

@sberyozkin
Copy link
Member

@sberyozkin sberyozkin commented Apr 13, 2025

Fixes #47102.

The current Keycloak container memory limit was designed to support running it in constrained environments, but unfortunately it causes side-effects, with no obvious logic.

For example, on my laptop I've never heard any issues with the 750 MB limit.
On my new laptop, which has twice as much memory and disk storage compared to the old laptop, it fails.

Users who have reported the side-effect before, confirmed 1.2 GB is sufficient for a case with the custom realm, and it works on my laptop.

I propose to increase it now and for Keycloak experts, @mabartos, and others, to investigate further.
I'm not sure about setting JAVA_OPTS_KC_HEAP="-XX:MaxRAMPercentage=50 -XX:MinRAMPercentage=50" by default without further analysis

Copy link
Member

@michalvavrik michalvavrik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I have also experienced OOM when running unit tests in OIDC extension. +1 but 1250M is a lot, let's hope someone finds a time and investigate this.

@quarkus-bot
Copy link

quarkus-bot bot commented Apr 13, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit cc81959.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@github-actions
Copy link

🎊 PR Preview 75b67ec has been successfully built and deployed to https://quarkus-pr-main-47340-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

@quarkus-bot
Copy link

quarkus-bot bot commented Apr 13, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit cc81959.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
✔️ JVM Integration Tests - JDK 17 Logs Raw logs 🔍
JVM Integration Tests - JDK 17 Windows Build Failures Logs Raw logs 🔍
✔️ JVM Integration Tests - JDK 21 Logs Raw logs 🔍

You can consult the Develocity build scans.

Failures

⚙️ JVM Integration Tests - JDK 17 Windows #

- Failing: integration-tests/oidc-wiremock-logout 

📦 integration-tests/oidc-wiremock-logout

Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.5.2:test (default-test) on project quarkus-integration-test-oidc-wiremock-logout:

See D:\a\quarkus\quarkus\integration-tests\oidc-wiremock-logout\target\surefire-reports for the individual test results.
See dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
There was an error in the forked process

Copy link
Contributor

@mabartos mabartos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sberyozkin I'm ok to increase the limit for now, and I'll check it further. Thanks!

@sberyozkin
Copy link
Member Author

Thanks, one of the integration tests fails on Windows during the docker environment checks, I'll tune it separately

@sberyozkin sberyozkin merged commit 55ff02e into quarkusio:main Apr 14, 2025
28 of 29 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.22 - main milestone Apr 14, 2025
@sberyozkin sberyozkin deleted the keycloak_dev_service_increase__memory_limit branch April 14, 2025 09:42
@gsmet gsmet modified the milestones: 3.22 - main, 3.21.3 Apr 15, 2025
@jmartisk jmartisk modified the milestones: 3.21.3, 3.20.1 May 6, 2025
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.

Keycloak Dev service does not start with OIDC client with 3.20 and 3.21

6 participants