Skip to content

Upgrade to Quarkus 3.20.6.1#48706

Open
vmuzikar wants to merge 1 commit intokeycloak:release/26.2from
vmuzikar:quarkus-3.20.6.1
Open

Upgrade to Quarkus 3.20.6.1#48706
vmuzikar wants to merge 1 commit intokeycloak:release/26.2from
vmuzikar:quarkus-3.20.6.1

Conversation

@vmuzikar
Copy link
Copy Markdown
Contributor

@vmuzikar vmuzikar commented May 5, 2026

Closes #48692
Closes #48275
Closes #48570

@vmuzikar vmuzikar requested a review from shawkins May 5, 2026 10:35
@vmuzikar vmuzikar requested a review from a team as a code owner May 5, 2026 10:35
@vmuzikar vmuzikar linked an issue May 5, 2026 that may be closed by this pull request
Upgrade to Quarkus 3.20.6.1 #48692
Open
shawkins
shawkins reviewed May 5, 2026
View reviewed changes
Comment thread pom.xml Outdated
@vmuzikar
Upgrade to Quarkus 3.20.6.1
255240b 
Closes keycloak#48692
Closes keycloak#48275
Closes keycloak#48570

Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
@vmuzikar vmuzikar force-pushed the quarkus-3.20.6.1 branch from 96e84d3 to 255240b Compare May 5, 2026 11:26
keycloak-github-bot[bot]
keycloak-github-bot Bot reviewed May 5, 2026
View reviewed changes
Copy link
Copy Markdown

@keycloak-github-bot keycloak-github-bot Bot left a comment

Choose a reason for hiding this comment

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

Unreported flaky test detected, please review

@keycloak-github-bot
Copy link
Copy Markdown

keycloak-github-bot Bot commented May 5, 2026

Unreported flaky test detected

If the flaky tests below are affected by the changes, please review and update the changes accordingly. Otherwise, a maintainer should report the flaky tests prior to merging the PR.

org.keycloak.testsuite.adapter.servlet.SAMLServletAdapterTest#testAdapterRoleMappings

Keycloak CI - Base IT

java.lang.AssertionError
	at org.junit.Assert.fail(Assert.java:87)
	at org.junit.Assert.assertTrue(Assert.java:42)
	at org.junit.Assert.assertTrue(Assert.java:53)
	at org.keycloak.testsuite.adapter.servlet.SAMLServletAdapterTest.assertForbiddenLogin(SAMLServletAdapterTest.java:518)
...

java.lang.NullPointerException: Cannot invoke "String.contains(java.lang.CharSequence)" because the return value of "org.openqa.selenium.WebDriver.getPageSource()" is null
	at org.keycloak.testsuite.adapter.servlet.SAMLServletAdapterTest.assertForbiddenLogin(SAMLServletAdapterTest.java:518)
	at org.keycloak.testsuite.adapter.servlet.SAMLServletAdapterTest.testAdapterRoleMappings(SAMLServletAdapterTest.java:1827)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
...

Report flaky test

shawkins
shawkins approved these changes May 5, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@shawkins shawkins left a comment

Choose a reason for hiding this comment

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

LGTM

@vmuzikar vmuzikar enabled auto-merge (squash) May 5, 2026 11:35
@stianst stianst self-assigned this May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shawkins shawkins shawkins approved these changes

+1 more reviewer

@keycloak-github-bot keycloak-github-bot[bot] keycloak-github-bot[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@stianst stianst

Labels

flaky-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Upgrade to Quarkus 3.20.6.1

3 participants

@vmuzikar @shawkins @stianst