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

[3.8] Backports 4th of September #1971

Merged
merged 18 commits into from
Sep 6, 2024
Merged

Conversation

jedla97
Copy link
Member

@jedla97 jedla97 commented Sep 3, 2024

fedinskiy and others added 13 commits September 3, 2024 21:01
In KRaft mode Zookeper is not used, since brokers communicate to each
other directly.

(cherry picked from commit f4aefae)
* Now that Red Hat Serverless is shipped for OCP on aarch64, the tests
  should be re-enabled.

(cherry picked from commit 9418079)
* The operator is now delivered for OCP on aarch64, so the tests need to
  be enabled.

(cherry picked from commit 2645e96)
* Setting AMQ broker container to 7.12 which supports all Quarkus
  supported platforms, as opposed to the 7.1 that we used first released
  from 2019, for openshift-arm profile.

(cherry picked from commit 5179068)
* Enabling AMQ stream tests on aarch64 as the test container is now
  multi-arch.

(cherry picked from commit 32e88e6)
* Since library/mongodb is multiarch image, pinning it to tag is not
  needed or useful - this way we won't get fixes for Mongo container. So
  this commit removes the tag.

(cherry picked from commit 5e92d60)
* Reproducer for leaking transactions in XA mode

https://issues.redhat.com/browse/QUARKUS-4185

* Disabled long-running tests in PR CI

https://issues.redhat.com/browse/QQE-876
(cherry picked from commit 6587c08)
(cherry picked from commit 25a8612)
Copy link
Member

@rsvoboda rsvoboda left a comment

Choose a reason for hiding this comment

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

GitHub CI is green, approving

@rsvoboda
Copy link
Member

rsvoboda commented Sep 4, 2024

@jedla97 what about OpenShift runs? Are they needed?

@jedla97
Copy link
Member Author

jedla97 commented Sep 4, 2024

@rsvoboda I will run them just to check. Also I want to add other backports for 3.8.6, there were some recent merge.

@jedla97
Copy link
Member Author

jedla97 commented Sep 5, 2024

Waiting for #1972 to be merged, after that I will rebase, cherry pick and push

@gtroitsk
Copy link
Member

gtroitsk commented Sep 5, 2024

Waiting for #1972 to be merged, after that I will rebase, cherry pick and push

#1972 was merged, you can rebase PR

@jedla97
Copy link
Member Author

jedla97 commented Sep 5, 2024

run tests

@jedla97
Copy link
Member Author

jedla97 commented Sep 5, 2024

The #1963 and #1970 are not part of this backport as they are failing and fixes are not yet backported to 3.8 branch

@jedla97
Copy link
Member Author

jedla97 commented Sep 5, 2024

run tests

@rsvoboda
Copy link
Member

rsvoboda commented Sep 5, 2024

DownloadResourceIT failed because oidcdisable.properties was not in 3.8 branch. I think FW should complain about such scenario so the root cause is more obvious.

@jedla97 jedla97 marked this pull request as ready for review September 5, 2024 20:18
@jedla97
Copy link
Member Author

jedla97 commented Sep 6, 2024

run tests

@jedla97 jedla97 requested a review from rsvoboda September 6, 2024 08:56
Copy link
Member

@rsvoboda rsvoboda left a comment

Choose a reason for hiding this comment

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

Thanks for the back-port effort @jedla97. It's good to see green CI for all these changes!

@rsvoboda rsvoboda merged commit 8f044d5 into quarkus-qe:3.8 Sep 6, 2024
9 checks passed
@jedla97 jedla97 deleted the 3.8-backports branch December 10, 2024 12:17
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.

7 participants