Skip to content

Block shutdown if the Infinispan cluster is not stable#48538

Merged
ahus1 merged 1 commit intokeycloak:mainfrom
pruivo:t_44620_npe
Apr 28, 2026
Merged

Block shutdown if the Infinispan cluster is not stable#48538
ahus1 merged 1 commit intokeycloak:mainfrom
pruivo:t_44620_npe

Conversation

@pruivo
Copy link
Copy Markdown
Member

@pruivo pruivo commented Apr 28, 2026

Prevent NullPointerException in DefaultInfinispanConnectionProviderFactory

Closes #44620

@keycloak-github-bot
Copy link
Copy Markdown

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.model.singleUseObject.SingleUseObjectModelTest#testRevokedTokenIsPresentAfterRestartAndEventuallyExpires

Keycloak CI - Store Model Tests

org.infinispan.remoting.RemoteException: ISPN000217: Received exception from node-70, see cause for remote stack trace
	at org.infinispan.remoting.transport.ResponseCollectors.wrapRemoteException(ResponseCollectors.java:26)
	at org.infinispan.remoting.transport.ValidSingleResponseCollector.withException(ValidSingleResponseCollector.java:37)
	at org.infinispan.remoting.transport.ValidSingleResponseCollector.addResponse(ValidSingleResponseCollector.java:21)
	at org.infinispan.remoting.transport.impl.SingleTargetRequest.addResponse(SingleTargetRequest.java:69)
...

Report flaky test

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

@pruivo pruivo marked this pull request as ready for review April 28, 2026 15:25
@pruivo pruivo requested review from a team as code owners April 28, 2026 15:25
Prevent NullPointerException in DefaultInfinispanConnectionProviderFactory

Closes keycloak#44620

Signed-off-by: Pedro Ruivo <[email protected]>
@ahus1 ahus1 enabled auto-merge (squash) April 28, 2026 16:34
@ahus1 ahus1 merged commit b3fbe0b into keycloak:main Apr 28, 2026
188 of 194 checks passed
@pruivo pruivo deleted the t_44620_npe branch April 29, 2026 07:27
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.

Possible data loss when scaling Keycloak StatefulSet down (on Kubernetes)

3 participants