8000 Backport: Fix various issues holding up CI (22) by rmartinc · Pull Request #33562 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Backport: Fix various issues holding up CI (22) - #33562

Merged
mposolda merged 1 commit into
keycloak:release/22.0from
rmartinc:saml-test-hang-22
Oct 4, 2024
Merged

mposolda merged 1 commit into
keycloak:release/22.0from
rmartinc:saml-test-hang-22

Conversation

@rmartinc
@rmartinc rmartinc commented Oct 4, 2024
Copy link
Copy Markdown
Contributor

Closes #33064

PR: #33086
Commit: 42cde0c
Target branch: https://github.com/keycloak/keycloak/tree/release/22.0

Fix for the saml hang in ArtifactBindingWithResolutionServiceTest. In upstream the PR fixed several things. Just the needed commit is backported.

- Disables the remote operator tests, which will have to be fixed later.
- Fixes the action expired error which occurs when accessing regular registration page with Organizations enabled.
- Fixes a race condition in the test suite causing sporadic failures.

Closes keycloak#33064

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>
Signed-off-by: rmartinc <rmartinc@redhat.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Co-authored-by: rmartinc <rmartinc@redhat.com>
Co-authored-by: Jon Koops <jonkoops@gmail.com>
@mposolda mposolda self-assigned this Oct 4, 2024
@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

@keycloak-github-bot
8000 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.ui.account2.SessionTest#welcomeScreenAsyncLogoutTest

Keycloak CI - Account Console IT (firefox)

org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a org.keycloak.testsuite.util.URLAssert URL expected to begin with: https://localhost:8543/auth/realms/test/protocol/openid-connect/auth ; actual URL: https://localhost:8543/auth/realms/test/account/#/personal-info within 10 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:31)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:985)
...

Report flaky test

org.keycloak.testsuite.forms.BrowserFlowTest#testAlternativeNonInteractiveExecutorInSubflow

Keycloak CI - Forms IT (firefox)

org.openqa.selenium.TimeoutException: 
Navigation timed out after 10000 ms
Build info: version: '3.14.0', revision: 'aacccce0', time: '2018-08-02T20:19:58.91Z'
System info: host: 'fv-az1205-970', ip: '10.1.0.187', os.name: 'Linux', os.arch: 'amd64', os.version: '6.8.0-1014-azure', java.version: '17.0.12'
Driver info: org.openqa.selenium.firefox.FirefoxDriver
...

Report flaky test

@mposolda
mposolda merged commit dbf5bc6 into keycloak:release/22.0 Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
51D1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0