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

2.12.0 backports 1 without gradle #27476

Merged
merged 33 commits into from Aug 24, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Aug 24, 2022

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

Sanne and others added 30 commits August 23, 2022 22:55
In particular we need it to avoid loading ANTLR classes because it's
using versions which are not aligned with our BOM.
Alternatively we could have pursued alignment, but it would need ongoing
maintenance and ANTLR is being used by multiple significant projects.

(cherry picked from commit 52ca34b)
Signed-off-by: Michael Edgar <michael@xlate.io>
(cherry picked from commit 96e13a6)
Save method hashes based on resource implementation classes and both the
concrete and (potentially) abstract methods.

(cherry picked from commit 1e0bb5b)
Beyond us not actually needing this dependency, it was also
transitively pulling in a dependency on com.nimbusds:oauth2-oidc-sdk
which in turns enables additional Maven repositories on
the project. We really don't want that to happen.

(cherry picked from commit c3f2aaf)
The example in the guide isn't the same as the rest-json-quickstart code.

(cherry picked from commit 75c1c08)
Bumps [postgresql](https://github.com/pgjdbc/pgjdbc) from 42.4.1 to 42.4.2.
- [Release notes](https://github.com/pgjdbc/pgjdbc/releases)
- [Changelog](https://github.com/pgjdbc/pgjdbc/blob/master/CHANGELOG.md)
- [Commits](pgjdbc/pgjdbc@REL42.4.1...REL42.4.2)

---
updated-dependencies:
- dependency-name: org.postgresql:postgresql
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 1ac6498)
With the 'resteasy-jackson' extension it gives error, it must be the 'resteasy-reactive-jackson'.

(cherry picked from commit c28bb72)
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds.

The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job.

Fixes quarkusio#27346

(cherry picked from commit e4e0d4e)
The implementation is exactly the same as the one from
RESTEasy Classic

Fixes: quarkusio#27221
(cherry picked from commit 64315ae)
Bumps [kotlinx-serialization-json](https://github.com/Kotlin/kotlinx.serialization) from 1.3.3 to 1.4.0.
- [Release notes](https://github.com/Kotlin/kotlinx.serialization/releases)
- [Changelog](https://github.com/Kotlin/kotlinx.serialization/blob/master/CHANGELOG.md)
- [Commits](Kotlin/kotlinx.serialization@v1.3.3...v1.4.0)

---
updated-dependencies:
- dependency-name: org.jetbrains.kotlinx:kotlinx-serialization-json
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 947523f)
The new API is using the Redis 7 commands, so without this, some commands do not work (geosearch for example) as the server does not handle them.

(cherry picked from commit aece8a4)
And add a link to the execution model that explains smart dispath.

(cherry picked from commit 15f2c17)
(cherry picked from commit 7328ef4)
(cherry picked from commit 23c61b8)
Signed-off-by: Phillip Kruger <phillip.kruger@gmail.com>
(cherry picked from commit ff766fd)
@gsmet gsmet added the area/infra internal and infrastructure related issues label Aug 24, 2022
@gsmet gsmet self-assigned this Aug 24, 2022
@quarkus-bot quarkus-bot bot added 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/documentation area/graphql area/jackson Issues related to Jackson (JSON library) area/jakarta area/oidc area/openapi area/platform Issues related to definition and interaction with Quarkus Platform area/qute The template engine area/redis area/resteasy-reactive area/scheduler area/smallrye area/vertx labels Aug 24, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 24, 2022

Failing Jobs - Building d85cf96

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
JVM Tests - JDK 17 Build Failures Logs Raw logs
✔️ JVM Tests - JDK 18

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 17 #

- Failing: integration-tests/micrometer-prometheus 

📦 integration-tests/micrometer-prometheus

io.quarkus.it.micrometer.prometheus.ClientRequestTest.testClientRequests line 36 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Response body doesn't match expectation.

@gsmet
Copy link
Member Author

gsmet commented Aug 24, 2022

@glefloch so I removed the Gradle upgrade from the backport PR and it looks much better. So it seems there's something fishy with the Gradle upgrade.

@gsmet gsmet merged commit 22d2a08 into quarkusio:2.12 Aug 24, 2022
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/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/graphql area/infra internal and infrastructure related issues area/jackson Issues related to Jackson (JSON library) area/jakarta area/oidc area/openapi area/platform Issues related to definition and interaction with Quarkus Platform area/qute The template engine area/redis area/resteasy-reactive area/scheduler area/smallrye area/vertx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet