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.7.2 backports 2 #38661

Merged
merged 11 commits into from
Feb 8, 2024
Merged

3.7.2 backports 2 #38661

merged 11 commits into from
Feb 8, 2024

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Feb 7, 2024

Please don't merge, I will merge it myself.

mcruzdev and others added 11 commits February 7, 2024 19:02
…s HalCollectionWrapper

(cherry picked from commit 9855040)
Signed-off-by: Phillip Kruger <[email protected]>
(cherry picked from commit 627bfa8)
The only detected breaking changes are related to the vertx-mutiny-http-proxy module which is not used in Quarkus.

Full release notes are there: https://github.com/vert-x3/wiki/wiki/4.5.3-Release-Notes.

Important:
 - Fix CVE-2024-1300 io.vertx:vertx-core: memory leak when a TCP server is configured with TLS and SNI support
 - Fix the classloading issue for NVMPN and WebJars in dev mode (quarkusio#38576)
(cherry picked from commit 7bcbb99)
We use the same trick as used in native-image
building

Fixes: quarkusio#38616
(cherry picked from commit 692a640)
(cherry picked from commit 89bd338)
And use the same version everywhere.
This might help with the CI issues.

(cherry picked from commit df6b11f)
(cherry picked from commit a309b30)
Per doc team request.

(cherry picked from commit 279a889)
@gsmet gsmet self-assigned this Feb 7, 2024
Copy link

quarkus-bot bot commented Feb 7, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 7f77c9b.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

⚠️ There are other check runs running, make sure you don't need to wait for their status before merging.

Copy link

github-actions bot commented Feb 7, 2024

🙈 The PR is closed and the preview is expired.

Copy link

quarkus-bot bot commented Feb 7, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 7f77c9b.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 17 Build Failures Logs Raw logs 🔍
✔️ JVM Tests - JDK 21 Logs Raw logs 🔍

Full information is available in the Build summary check run.
You can consult the Develocity build scans.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: integration-tests/opentelemetry 

📦 integration-tests/opentelemetry

io.quarkus.it.opentelemetry.EndUserEnabledTest.baseTest line 23 - History - More details - Source on GitHub

org.opentest4j.AssertionFailedError: expected: <null> but was: <testUser>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177)
	at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:1145)
	at io.quarkus.it.opentelemetry.EndUserEnabledTest.evaluateAttributes(EndUserEnabledTest.java:23)

@gsmet gsmet merged commit 14d7d51 into quarkusio:3.7 Feb 8, 2024
53 of 54 checks passed
@gsmet gsmet added area/infra internal and infrastructure related issues and removed triage/backport-3.8 labels Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core area/dependencies Pull requests that update a dependency file area/docstyle issues related for manual docstyle review area/documentation area/infra internal and infrastructure related issues area/oidc area/reactive-messaging area/rest area/smallrye
Projects
Development

Successfully merging this pull request may close these issues.

7 participants