-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test Extended Architecture (XA) connection #971
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jsmrcka
suggested changes
Jan 2, 2023
sql-db/sql-app/src/test/java/io/quarkus/ts/sqldb/sqlapp/XAPostgresIT.java
Outdated
Show resolved
Hide resolved
e012572
to
3575565
Compare
jsmrcka
approved these changes
Jan 2, 2023
The linux native CI build did not really execute anything due to the module detection issue, but I ran it locally and it works. |
@fedinskiy Could you pls check the failures in the JVM build - |
3575565
to
0f9df68
Compare
Provides coverage for https://issues.redhat.com/browse/QUARKUS-2742
0f9df68
to
ce15b26
Compare
fedinskiy
commented
Jan 3, 2023
CI fails due to DB2DatabaseIT failure. I can reproduce this failure on main branch, so it is not connected to these changes |
3 tasks
1 task
jsmrcka
added a commit
to jsmrcka/quarkus-test-suite
that referenced
this pull request
Jan 30, 2023
Removes workaround for a failure reported in quarkus-qe/quarkus-test-framework#641 and observed in quarkus-qe#971.
9 tasks
jsmrcka
added a commit
to jsmrcka/quarkus-test-suite
that referenced
this pull request
Feb 15, 2023
Removes workaround for a failure reported in quarkus-qe/quarkus-test-framework#641 and observed in quarkus-qe#971.
jsmrcka
added a commit
to jsmrcka/quarkus-test-suite
that referenced
this pull request
Feb 16, 2023
Removes workaround for a failure reported in quarkus-qe/quarkus-test-framework#641 and observed in quarkus-qe#971.
jsmrcka
added a commit
to jsmrcka/quarkus-test-suite
that referenced
this pull request
Mar 1, 2023
Removes workaround for a failure reported in quarkus-qe/quarkus-test-framework#641 and observed in quarkus-qe#971.
pjgg
pushed a commit
that referenced
this pull request
Mar 2, 2023
* Remove PostgreSQL image workaround Removes workaround for a failure reported in quarkus-qe/quarkus-test-framework#641 and observed in #971. * Due to a test framework issue we are going to reuse postgres docker container image --------- Co-authored-by: pablo gonzalez granados <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Provides coverage for https://issues.redhat.com/browse/QUARKUS-2742
Summary
(Summarize the problem solved by this PR, and how to verify it manually)
Please select the relevant options.
run tests
phrase in comment)Checklist: