Skip to content
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 1 commit into from
Jan 4, 2023

Conversation

fedinskiy
Copy link
Contributor

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.

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@fedinskiy fedinskiy added the triage/backport-2.13? It needs to backport changes to branch 2.13 label Jan 2, 2023
@fedinskiy fedinskiy requested a review from jsmrcka January 2, 2023 14:06
@fedinskiy fedinskiy force-pushed the feature/xa-coverage branch from e012572 to 3575565 Compare January 2, 2023 14:35
@fedinskiy fedinskiy requested a review from jsmrcka January 2, 2023 14:55
@jsmrcka
Copy link
Contributor

jsmrcka commented 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.

@jsmrcka
Copy link
Contributor

jsmrcka commented Jan 3, 2023

@fedinskiy Could you pls check the failures in the JVM build - sql-db/sql-app module?

@fedinskiy fedinskiy force-pushed the feature/xa-coverage branch from 3575565 to 0f9df68 Compare January 3, 2023 15:46
@fedinskiy fedinskiy force-pushed the feature/xa-coverage branch from 0f9df68 to ce15b26 Compare January 3, 2023 16:32
@fedinskiy
Copy link
Contributor Author

CI fails due to DB2DatabaseIT failure. I can reproduce this failure on main branch, so it is not connected to these changes

@fedinskiy fedinskiy merged commit 67f1eef into quarkus-qe:main Jan 4, 2023
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.
@jsmrcka jsmrcka mentioned this pull request Jan 30, 2023
9 tasks
@pjgg pjgg added this to the 2.7 milestone Feb 2, 2023
@pjgg pjgg removed the triage/backport-2.13? It needs to backport changes to branch 2.13 label Feb 2, 2023
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants