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 kubernetes-client-bom from 6.7.1 to 6.7.2 #34068

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

manusa
Copy link
Contributor

@manusa manusa commented Jun 15, 2023

Kubernetes Client 6.7.2 was just released: https://github.com/fabric8io/kubernetes-client/releases/tag/v6.7.2

Just speeding up the dependabot process to see if CI reports any issue.

No breaking changes.

/cc @metacosm

@gastaldi
Copy link
Contributor

/cc @metacosm

@quarkus-bot
Copy link

quarkus-bot bot commented Jun 16, 2023

Failing Jobs - Building 0c52b6f

Status Name Step Failures Logs Raw logs
JVM Tests - JDK 11 Build Failures Logs Raw logs
JVM Tests - JDK 17 Build Failures Logs Raw logs
JVM Tests - JDK 17 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 19
✔️ Maven Tests - JDK 11
Maven Tests - JDK 11 Windows Build ⚠️ Check → Logs Raw logs
Native Tests - Windows - RESTEasy Jackson Setup GraalVM ⚠️ Check → Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: extensions/resteasy-reactive/rest-client-reactive/deployment 
! Skipped: extensions/keycloak-admin-client-reactive/deployment extensions/oidc-client-reactive-filter/deployment extensions/oidc-token-propagation-reactive/deployment and 22 more

📦 extensions/resteasy-reactive/rest-client-reactive/deployment

io.quarkus.rest.client.reactive.StreamJsonTest.shouldReadUpToThreeTicks line 129 - More details - Source on GitHub

java.lang.AssertionError: Expected 3 items.  Only 2 items have been received.
	at io.smallrye.mutiny.helpers.test.AssertSubscriber.awaitItemEvents(AssertSubscriber.java:687)
	at io.smallrye.mutiny.helpers.test.AssertSubscriber.awaitItems(AssertSubscriber.java:441)

⚙️ JVM Tests - JDK 17 #

- Failing: extensions/amazon-lambda/deployment 
! Skipped: extensions/amazon-lambda-http/deployment extensions/amazon-lambda-rest/deployment integration-tests/amazon-lambda and 6 more

📦 extensions/amazon-lambda/deployment

io.quarkus.amazon.lambda.deployment.testing.LambdaDevServicesContinuousTestingTestCase.testLambda line 48 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: expected: <1> but was: <0>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)

@metacosm
Copy link
Contributor

Should be OK on our side.

@manusa
Copy link
Contributor Author

manusa commented Jun 20, 2023

Hi, this bump contains a fix that solves a memory leak for applications that are intensive in requests to the Kube API server.
Is there something else needed? shall I rebase to the latest changes in main?

Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. CI errors are unrelated. Let's merge it

@gastaldi gastaldi merged commit 3bb7820 into quarkusio:main Jun 20, 2023
@quarkus-bot quarkus-bot bot added this to the 3.2 - main milestone Jun 20, 2023
@manusa manusa deleted the deps/kubernetes-client branch June 20, 2023 10:37
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.

3 participants