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

@WithKubernetesMockServer not cleaning up pods on delete #20039

Closed
kenfinnigan opened this issue Sep 9, 2021 · 6 comments
Closed

@WithKubernetesMockServer not cleaning up pods on delete #20039

kenfinnigan opened this issue Sep 9, 2021 · 6 comments
Labels
area/kubernetes kind/bug Something isn't working
Milestone

Comments

@kenfinnigan
Copy link
Member

Describe the bug

When creating pods in the mock server with mockServer.getClient().pods().create(pod1), deleting them at the end of a test with mockServer.getClient().pods().delete() does not result in the pods being removed in subsequent tests

Expected behavior

Calling mockServer.getClient().pods().delete() removes all pods from the mocked server

Actual behavior

mockServer.getClient().pods() still returns the pods that were created in a previous test

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

2.1.3.Final

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

@kenfinnigan kenfinnigan added the kind/bug Something isn't working label Sep 9, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 9, 2021

/cc @geoand, @iocanel

@geoand
Copy link
Contributor

geoand commented Sep 10, 2021

Seems like an upstream issue as we pass do pass the crud=true flag to the io.fabric8.kubernetes.client.server.mock.KubernetesServer

@kenfinnigan
Copy link
Member Author

Ok @geoand, I'll open an upstream issue

@kenfinnigan
Copy link
Member Author

@geoand
Copy link
Contributor

geoand commented Sep 10, 2021

Great, thanks

@geoand
Copy link
Contributor

geoand commented Sep 29, 2021

Closing as the fix was delivered in Kubernetes Client 5.8.0 which is now included in Quarkus

@geoand geoand closed this as completed Sep 29, 2021
@geoand geoand added this to the 2.4 - main milestone Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubernetes kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants