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

Bump smallrye-open-api from 4.0.2 to 4.0.3 #44481

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

MikeEdgar
Copy link
Contributor

Update smallrye-open-api to 4.0.3 release

Signed-off-by: Michael Edgar <michael@xlate.io>
@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/smallrye labels Nov 13, 2024
@MikeEdgar
Copy link
Contributor Author

cc: @phillip-kruger

Copy link

quarkus-bot bot commented Nov 14, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 5c34e9e.

✅ 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 17 Windows

📦 integration-tests/smallrye-graphql

io.quarkus.it.smallrye.graphql.MicrometerMetricsTest.shouldCreateMultipleMetrics - History

  • 1 expectation failed. Response body doesn't match expectation. Expected: a string containing "mp_graphql_seconds_count{name=\"asyncSuperMetricFoo\",source=\"false\",type=\"QUERY\"} 200" Actual: \# TYPE http_server_requests_seconds summary \# HELP http_server_requests_seconds HTTP server request processing time - java.lang.AssertionError
java.lang.AssertionError: 
1 expectation failed.
Response body doesn't match expectation.
Expected: a string containing "mp_graphql_seconds_count{name=\"asyncSuperMetricFoo\",source=\"false\",type=\"QUERY\"} 200"
  Actual: # TYPE http_server_requests_seconds summary
# HELP http_server_requests_seconds HTTP server request processing time
http_server_requests_seconds_count{method="POST",outcome="SUCCESS",status="200",uri="/graphql"} 201.0
http_server_requests_seconds_sum{method="POST",outcome="SUCCESS",status="200",uri="/graphql"} 50.957801998

@phillip-kruger phillip-kruger merged commit 90b6e9b into quarkusio:main Nov 14, 2024
52 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.18 - main milestone Nov 14, 2024
@phillip-kruger
Copy link
Member

@MikeEdgar should this be baackported to 3.17 ?

@MikeEdgar MikeEdgar deleted the smallrye-open-api-4.0.3 branch November 14, 2024 09:48
@MikeEdgar
Copy link
Contributor Author

@MikeEdgar should this be baackported to 3.17 ?

Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/smallrye triage/flaky-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants