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.

Sign up for GitHub

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

Jump to bottom

[3.18] 3.18.0 backports 1 #45759

Merged
merged 50 commits into from
Jan 22, 2025
Merged

[3.18] 3.18.0 backports 1 #45759

merged 50 commits into from
Jan 22, 2025

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Jan 21, 2025

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

LarsSven and others added 30 commits January 21, 2025 18:07
@LarsSven @gsmet
Allow null for query parameters in Kotlin panache
337a8c7 
(cherry picked from commit dbdc588)
@rsvoboda @gsmet
Replace JavaVersion.VERSION_11 with JavaVersion.VERSION_21
ec991d2 
(cherry picked from commit d21a1da)
@metacosm @gsmet
Fix typo
d47763e 
(cherry picked from commit a9bb751)
@alesj @gsmet
Fix missing addChannel flag usage
02fe8a7 
(cherry picked from commit 8e09eb0)
@geoand @gsmet
Fix file handling in REST Client
b897387 
Fixes: quarkusio#45627
(cherry picked from commit 30ee339)
@geoand @gsmet
Use VertxLogDelegateFactory for internal Vert.x logging
2e71d98 
This is done because I assume that was the original intent,
but also because it avoids the non-zero cost
incurred by Vert.x to look up the various
supported options

(cherry picked from commit cf6c2cd)
@michalvavrik @gsmet
docs,tests(oidc): Use OidcTestClient with Dev Svc for OIDC
92b08ab 
(cherry picked from commit e227b9c)
@sberyozkin @gsmet
Enable public access to OidcProviderClientImpl#getWebClient
da51b6a 
(cherry picked from commit 930d0f2)
@brunobat @gsmet
Update grafana image and remove dashboard hack
1dbb706 
(cherry picked from commit 9acf79d)
@sberyozkin @gsmet
Getting Started with Security updates
4ad5844 
(cherry picked from commit 9e79265)
@brunobat @gsmet
Fix flaky otel quickstart test
d5a946d 
(cherry picked from commit d605c31)
@brunobat @gsmet
Update grafana image and remove dashboard hack
2effce0 
Fix Otel logging message formating

(cherry picked from commit 81f3e20)
@gsmet
Mark quarkus-oidc-client-registration as experimental in doc
60bc7f9 
It's marked as experimental in the metadata of the extension but not in
the documentation.

(cherry picked from commit 7f6a046)
@sberyozkin @gsmet
Update to HTTP error outputs in getting Started with Security
562e3fb 
(cherry picked from commit 5aa963b)
@gsmet
Gradle - Correctly pass platform.quarkus.* properties
16c0ed7 
Fixes quarkusio#45472

(cherry picked from commit 2ed6293)
@holly-cummins @gsmet
Make sure to delegate to instance variable for releasing config of a …
581f2f7 
…classloader

(cherry picked from commit c1c3bdf)
@dcdh @gsmet
Update drools.adoc
8240476 
An IT is based on RestAssured only. The name should end by Test.

(cherry picked from commit 7aa12cb)
@gsmet
Add missing @configitem to FilterConfig
648777c 
Fixes quarkusio#45676

(cherry picked from commit 6adfaff)
@gsmet
Do not prefix URL when URL is already absolute
b4fdc47 
Management URLs were prefixed twice when absolute:
http://localhost:9000http://localhost:9000/q/health

Which was defeating the logic removing the host when collecting
suppressed URIs.

Fixes quarkusio#36510

(cherry picked from commit 5da616b)
@gsmet
Identify non application routes with a name
0214f2d 
The name is used in the metrics if defined and thus we can pass a fully
qualifier path for the route.

(cherry picked from commit c377d4a)
@brunobat @gsmet
Test for non app uris
64156ec 
(cherry picked from commit 9f806c0)
@gsmet
Avoid duplicates in FrameworkEndpointsBuildItem
dbb78f3 
(cherry picked from commit ac983ac)
@gsmet
Simplify OpenTelemetrySuppressNonAppUriTest a bit
aec727a 
(cherry picked from commit 7c3e93a)
@gsmet
Use an HashSet contract for DropTargetsSamplerTest
51095df 
It's a detail but it's the contract we want and matching will be faster.

(cherry picked from commit 9cd2b13)
@gsmet
Let TestSpanExporter when we have at least X elements included
c0d81ef 
The current condition is weird because we end up saying we want at least
1, and it only returns when there are 2.

(cherry picked from commit 081c7b1)
@gsmet
Fix wildcard dropping in DropTargetsSampler
8586817 
While /q/dev-ui/* would exclude:
- /q/dev-ui/
- /q/dev-ui/whatever
it wouldn't exclude /q/dev-ui/whatever/whenever/, which was problematic.

See additional tests.

Note that I think this should be improved with a proper matcher that
handles all our matching rules efficiently but for now, it will have to
do.

(cherry picked from commit 7615750)
@gsmet
Add a SuppressNonAppUri test when management interface is enabled
f1d1740 
(cherry picked from commit d76ade7)
@gsmet
Gradle - Resolve platform properties when possible
1892b91 
We were always passing a <<ignored>> value for the native builder which
is a problem.
We now correctly get the values from the platform properties when possible.

(cherry picked from commit 17d805a)
@gsmet
Pass ApplicationModel to build configuration
2dde199 
(cherry picked from commit be3edfa)
@michalvavrik @gsmet
docs(security): CDI request context with HTTP perms and proactive auth
ca23957 
(cherry picked from commit a94d3ca)
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 21, 2025 •
edited
Loading

🙈 The PR is closed and the preview is expired.

DavideD and others added 3 commits January 21, 2025 22:14
@DavideD @gsmet
Bump hibernate-reactive.version from 2.4.3.Final to 2.4.4.Final
63463cb 
(cherry picked from commit 051d47d)
@dependabot @gsmet
Bump hibernate-orm.version from 6.6.4.Final to 6.6.5.Final
d8f56cb 
Bumps `hibernate-orm.version` from 6.6.4.Final to 6.6.5.Final.

Updates `org.hibernate.orm:hibernate-core` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

Updates `org.hibernate.orm:hibernate-graalvm` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

Updates `org.hibernate.orm:hibernate-envers` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

Updates `org.hibernate.orm:hibernate-jpamodelgen` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

Updates `org.hibernate:hibernate-jpamodelgen` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

Updates `org.hibernate.orm:hibernate-community-dialects` from 6.6.4.Final to 6.6.5.Final
- [Release notes](https://github.com/hibernate/hibernate-orm/releases)
- [Changelog](https://github.com/hibernate/hibernate-orm/blob/6.6.5/changelog.txt)
- [Commits](hibernate/hibernate-orm@6.6.4...6.6.5)

---
updated-dependencies:
- dependency-name: org.hibernate.orm:hibernate-core
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.hibernate.orm:hibernate-graalvm
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.hibernate.orm:hibernate-envers
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.hibernate.orm:hibernate-jpamodelgen
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.hibernate:hibernate-jpamodelgen
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.hibernate.orm:hibernate-community-dialects
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
(cherry picked from commit 37d0453)
@yrodiere @gsmet
Upgrade to ByteBuddy 1.15.11
b6bd029 
(cherry picked from commit 46bb4cd)
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@gsmet
Remove quarkus-extension-processor from cache-runtime-spi
3a4cf41 
It is not an extension so it shouldn't be treated as such.

I have no idea why but it causes some issues when building with an empty
repository so let's get rid of it given it's useless.
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Jan 22, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 3a4cf41.

✅ 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.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Jan 22, 2025 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 3a4cf41.

✅ 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.

You can consult the Develocity build scans.

Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 integration-tests/observability-lgtm

✖ io.quarkus.observability.test.LgtmReloadTest.testReload - History

  • io.quarkus.builder.BuildException: Build failure: Build failed due to errors [error]: Build step io.quarkus.observability.deployment.ObservabilityDevServiceProcessor\#startContainers threw an exception: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image docker.io/grafana/otel-lgtm:0.8.2 at io.quarkus.observability.deployment.ObservabilityDevServiceProcessor.lambda$startContainers$3(ObservabilityDevServiceProcessor.java:170) at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1708) at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:762) - java.lang.RuntimeException
java.lang.RuntimeException: 
io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkus.observability.deployment.ObservabilityDevServiceProcessor#startContainers threw an exception: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image docker.io/grafana/otel-lgtm:0.8.2
	at io.quarkus.observability.deployment.ObservabilityDevServiceProcessor.lambda$startContainers$3(ObservabilityDevServiceProcessor.java:170)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1708)
	at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:762)
	at io.quarkus.observability.deployment.ObservabilityDevServiceProcessor.startContainers(ObservabilityDevServiceProcessor.java:113)
	at java.base/java.lang.invoke.MethodHandle.invokeWithArguments(MethodHandle.java:733)

✖ io.quarkus.observability.test.LgtmResourcesTest.testTracing - History

  • java.lang.RuntimeException: Failed to start quarkus - java.lang.RuntimeException
java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.test.junit.QuarkusTestExtension.throwBootFailureException(QuarkusTestExtension.java:611)
	at io.quarkus.test.junit.QuarkusTestExtension.interceptTestClassConstructor(QuarkusTestExtension.java:695)
	at java.base/java.util.Optional.orElseGet(Optional.java:364)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
Caused by: java.lang.RuntimeException: Failed to start quarkus
	at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)

@gsmet gsmet merged commit 3a27b80 into quarkusio:3.18 Jan 22, 2025
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gsmet gsmet

Labels
area/cache area/core area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/docstyle issues related for manual docstyle review area/documentation area/gradle Gradle area/grpc gRPC area/infra internal and infrastructure related issues area/logging area/metrics area/oidc area/panache area/platform Issues related to definition and interaction with Quarkus Platform area/redis area/rest area/rest-client area/resteasy-classic area/swagger-ui area/testing area/tracing area/vertx triage/flaky-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.