Skip to content

Synchronize workflow scheduled settings using cluster events#48422

Merged
pedroigor merged 2 commits into
keycloak:mainfrom
sguilhen:48185-scheduled-workflow-cluster-sync
Apr 27, 2026
Merged

Synchronize workflow scheduled settings using cluster events#48422
pedroigor merged 2 commits into
keycloak:mainfrom
sguilhen:48185-scheduled-workflow-cluster-sync

Conversation

@sguilhen

@sguilhen sguilhen commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Closes #48185

This was referenced Apr 23, 2026
Closed
Closed
@keycloak-github-bot

keycloak-github-bot Bot commented Apr 23, 2026

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.authz.PolicyEvaluationTest#

Keycloak CI - Base IT (3)

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

Report flaky test

keycloak-github-bot[bot]
keycloak-github-bot Bot reviewed Apr 23, 2026
View reviewed changes

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

Copy link
Copy Markdown

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

@sguilhen sguilhen force-pushed the 48185-scheduled-workflow-cluster-sync branch from 694b1c8 to 67c93b1 Compare April 23, 2026 17:52
@sguilhen sguilhen marked this pull request as ready for review April 23, 2026 18:57
@sguilhen sguilhen requested review from a team as code owners April 23, 2026 18:57
@keycloak-github-bot

keycloak-github-bot Bot commented Apr 23, 2026

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.authz.PolicyEvaluationTest#

Keycloak CI - Base IT (3)

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

org.opentest4j.AssertionFailedError: expected: <PERMIT> but was: <null>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
...

Report flaky test

keycloak-github-bot[bot]
keycloak-github-bot Bot reviewed Apr 23, 2026
View reviewed changes

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

Copy link
Copy Markdown

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

@sguilhen sguilhen force-pushed the 48185-scheduled-workflow-cluster-sync branch from 67c93b1 to 659a3c5 Compare April 24, 2026 12:15
This was referenced Apr 24, 2026
Closed
Closed
Closed
Closed
pedroigor
pedroigor previously approved these changes Apr 24, 2026
View reviewed changes
@sguilhen sguilhen force-pushed the 48185-scheduled-workflow-cluster-sync branch from 659a3c5 to 815d3ee Compare April 24, 2026 20:55
This was referenced Apr 24, 2026
Closed
Closed
@sguilhen sguilhen requested review from a team as code owners April 24, 2026 23:53
@keycloak-github-bot keycloak-github-bot Bot mentioned this pull request Apr 25, 2026
Closed
sguilhen added 2 commits April 27, 2026 08:23
@sguilhen
Synchronize workflow scheduled settings using cluster events
596baba 
Closes keycloak#48185

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>
@sguilhen
Prevent concurrent execution of workflow steps across cluster nodes
dd5fc3c 
- Use per-execution distributed locking in RunWorkflowTask to ensure only one node processes a given workflow execution at a time.

Closes keycloak#48185

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>
@sguilhen sguilhen force-pushed the 48185-scheduled-workflow-cluster-sync branch from 2d9aef5 to dd5fc3c Compare April 27, 2026 11:24
@keycloak-github-bot keycloak-github-bot Bot mentioned this pull request Apr 27, 2026
Closed
@pedroigor pedroigor merged commit 33651e4 into keycloak:main Apr 27, 2026
86 checks passed
sguilhen added a commit to sguilhen/keycloak that referenced this pull request Apr 27, 2026
@sguilhen
Synchronize workflow scheduled settings using cluster events (keycloa…
ed7ec51 
…k#48422)

Closes keycloak#48185

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>

(cherry picked from commit 33651e4)
pedroigor pushed a commit that referenced this pull request Apr 27, 2026
@sguilhen @pedroigor
Synchronize workflow scheduled settings using cluster events (#48422)
41ff67b 
Closes #48185

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>

(cherry picked from commit 33651e4)
@sguilhen sguilhen deleted the 48185-scheduled-workflow-cluster-sync branch April 28, 2026 16:02
@stianst stianst mentioned this pull request Jun 1, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedroigor pedroigor pedroigor 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

No one assigned

Labels

flaky-test team/cloud-native team/core-clients team/core-iam

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deleted workflow still attempting to run

2 participants

@sguilhen @pedroigor