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

tests: k8s: avoid deleting unrelated pods #9070

Merged
merged 1 commit into from Feb 12, 2024

Conversation

danmihai1
Copy link
Contributor

Delete the debugger pod created during the test, rather than already existing debugger pods.

Fixes: #9069

@katacontainersbot katacontainersbot added the size/small Small and simple task label Feb 9, 2024
@danmihai1 danmihai1 added no-backport-needed ok-to-test do-not-merge PR has problems or depends on another wip Work in Progress (PR incomplete - needs more work or rework) labels Feb 9, 2024
Delete the debugger pod created during the test, rather than already
existing debugger pods.

Also, send the output of "kubectl delete" to stderr, just in case it's
useful for debugging.

Fixes: kata-containers#9069

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
@danmihai1 danmihai1 removed do-not-merge PR has problems or depends on another wip Work in Progress (PR incomplete - needs more work or rework) labels Feb 10, 2024
@danmihai1
Copy link
Contributor Author

/test

Copy link
Contributor

@wainersm wainersm left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @danmihai1 !

@GabyCT GabyCT merged commit 00be9ae into kata-containers:main Feb 12, 2024
291 of 298 checks passed
@danmihai1 danmihai1 deleted the danmihai1/debug-containers branch March 29, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tests: k8s: exec_host() deletes pods that were not created by the test
4 participants