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.17] 3.17.0 backports 1 #44581

Merged
merged 39 commits into from
Nov 20, 2024
Merged

[3.17] 3.17.0 backports 1 #44581

merged 39 commits into from
Nov 20, 2024

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Nov 19, 2024

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

mkouba and others added 30 commits November 19, 2024 19:47
- also add tests for programmatic lookup of connectors
- related to quarkusio#44465

(cherry picked from commit c59a6dc)
(cherry picked from commit e579dac)
Signed-off-by: Michael Edgar <[email protected]>
(cherry picked from commit 5c34e9e)
Bumps com.gradle.develocity from 3.18.1 to 3.18.2.

---
updated-dependencies:
- dependency-name: com.gradle.develocity
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
(cherry picked from commit 8111351)
Before these changes, we were only considering the "href" field for the HAL links. Example:

```json
{
  "_links": {
    "subject": {
      "href": "/subject"
    }
  }
}
```

After these changes, users that populate also the title and/or the type like:

```java
@get
    @path("/with-rest-link-with-all-fields")
    @produces({ MediaType.APPLICATION_JSON, RestMediaType.APPLICATION_HAL_JSON })
    public HalEntityWrapper<TestRecordWithIdAndPersistenceIdAndRestLinkId> get() {

        var entity = // ...
        return new HalEntityWrapper<>(entity,
                Link.fromUri(URI.create("/path/to/100"))
                        .rel("all")
                        .title("The title link") // the link title
                        .type(MediaType.APPLICATION_JSON) // the link type
                        .build());
    }
```

Or using the annotation like:

```java
@get
    @produces({ MediaType.APPLICATION_JSON, RestMediaType.APPLICATION_HAL_JSON })
    @RestLink(entityType = TestRecordWithRestLinkId.class, title = "The with rest link title", type = MediaType.APPLICATION_JSON)
    @InjectRestLinks
    public TestRecordWithRestLinkId get() {
        return // ...
    }
```

Then, the links will have the title and/or type fields populated:
```json
{
  "_links": {
    "subject": {
      "href": "/subject",
      "title": "The with rest link title",
      "type": "application/json"
    }
  }
}
```

(cherry picked from commit 679c1ed)
- for binary messages a custom codec is always needed

(cherry picked from commit d90b5f9)
(cherry picked from commit 73b1e5c)
Co-Authored-By: Max Rydahl Andersen <[email protected]>
Co-Authored-By: Guillaume Smet <[email protected]>
(cherry picked from commit 8180289)
…n Kubernetes resources

Because the original property is a build-time configuration property

(cherry picked from commit ef97403)
(cherry picked from commit 1069847)
…POM property;

* suport resolving JAR artifacts with classifiers from the classes dir;
* support the JAR and Surefire plugin configurations in the pluginManagement when initializing artifact source sets.

(cherry picked from commit 405455a)
(cherry picked from commit 1e8cad0)
(cherry picked from commit f841486)
(cherry picked from commit 4e38659)
(cherry picked from commit 8abb30d)
- Bump to Netty 4.1.115.Final and fix SSL-related substitutions due to internal Netty breaking changes
- Bump to Vert.x 4.5.11
- Bump Mutiny Vert.x bindings 3.16.0
- Re-aligned the Vert.x versions across Quarkus modules

Fixes CVE-2024-47535 with Netty 4.1.115.Final

(cherry picked from commit 9fd8dcb)
…o be generated in JacksonCodeGenerator

(cherry picked from commit ee68fdb)
This reverts commit 693ddaf and updates 'model' to matrix

Co-Authored-By: Guillaume Smet <[email protected]>
(cherry picked from commit 14946be)
(cherry picked from commit ea35c4d)
Copy link

github-actions bot commented Nov 19, 2024

🙈 The PR is closed and the preview is expired.

Copy link

quarkus-bot bot commented Nov 19, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 9ea9a9f.

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

Copy link

quarkus-bot bot commented Nov 20, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 9ea9a9f.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
MicroProfile TCKs Tests Verify Failures Logs Raw logs 🔍

You can consult the Develocity build scans.

Failures

⚙️ MicroProfile TCKs Tests #

- Failing: tcks/microprofile-opentelemetry 

📦 tcks/microprofile-opentelemetry

Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.5.0:test (tracing) on project quarkus-tck-microprofile-opentelemetry: There are test failures.

Please refer to /home/runner/work/quarkus/quarkus/tcks/microprofile-opentelemetry/target/surefire-reports-tracing for the individual test results.
Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.


Flaky tests - Develocity

⚙️ JVM Tests - JDK 21

📦 integration-tests/opentelemetry-vertx-exporter

io.quarkus.it.opentelemetry.vertx.exporter.grpc.SimpleGrpcWithTLSNoCompressionTest.test - History

  • Assertion condition defined as a Lambda expression in io.quarkus.it.opentelemetry.vertx.exporter.AbstractExporterTest Expecting Optional to contain a value but it was empty within 30 seconds. - org.awaitility.core.ConditionTimeoutException
org.awaitility.core.ConditionTimeoutException: 
Assertion condition defined as a Lambda expression in io.quarkus.it.opentelemetry.vertx.exporter.AbstractExporterTest 
Expecting Optional to contain a value but it was empty within 30 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:31)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1006)
	at org.awaitility.core.ConditionFactory.untilAsserted(ConditionFactory.java:790)

@gsmet gsmet merged commit 4c3b5f6 into quarkusio:3.17 Nov 20, 2024
54 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.