Skip to content

Quarkus IT that use Oracle DB don't work with -Dproduct - #23084

Merged
vmuzikar merged 1 commit into
keycloak:mainfrom
mabartos:KC-23058-oracle-test-product
Sep 8, 2023
Merged

Quarkus IT that use Oracle DB don't work with -Dproduct#23084
vmuzikar merged 1 commit into
keycloak:mainfrom
mabartos:KC-23058-oracle-test-product

Conversation

@mabartos

@mabartos mabartos commented Sep 8, 2023

Copy link
Copy Markdown
Member

Fixes #23058

We can improve the process around the JDBC drivers' copies to providers in a follow-up task.

@ahus1 ahus1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, although I don't have a strong opinion here / not sure how to test this as there is no OracleDistTest at the moment

@vmuzikar vmuzikar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Changes LGTM, does what it's supposed to do but I was unable to run the OracleTest in the end. We don't test it in the CI and even when trying to get it working locally I was unable to manually run the DB container as the prerequisite. @mabartos Were you able to run the test?

In any case, this PR correctly addresses the issue and even if the test is broken, it's out of the scope of this PR and we'd address it separately.

@vmuzikar
vmuzikar merged commit a6ae1b9 into keycloak:main Sep 8, 2023
@cypress

cypress Bot commented Sep 10, 2023

Copy link
Copy Markdown

1 flaky test on run #8879 ↗︎

0 1054 96 0 Flakiness 1

Details:

Quarkus IT that use Oracle DB don't work with -Dproduct (#23084)
Project: Keycloak Admin UI Commit: a6ae1b9db7
Status: Passed Duration: 13:03 💡
Started: Sep 11, 2023 12:23 AM Ended: Sep 11, 2023 12:36 AM
Flakiness  cypress/e2e/realm_settings_client_profiles_test.spec.ts • 1 flaky test • firefox

View Output Video

Test Artifacts
Realm settings client profiles tab tests > Should not create duplicate client profile Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

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.

Quarkus IT that use Oracle DB don't work with -Dproduct

3 participants