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

set option to cascade delete job resources #635

Merged
merged 1 commit into from
Oct 9, 2019

Conversation

houqp
Copy link
Contributor

@houqp houqp commented Oct 8, 2019

This is also the default behavior from kubectl.

Currently, when a job is deleted from Terraform, all pods are left behind as orphan resources.

Relates to #534.

This is also the default behavior from kubectl.
@ghost ghost added the size/XS label Oct 8, 2019
@alexsomesan
Copy link
Member

Thank you, this is really useful!
The necessity for such cleanups becomes apparent as users manage K8s more with Terraform.

@alexsomesan
Copy link
Member

I'm testing it and will merge when green.

Copy link
Member

@alexsomesan alexsomesan 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 passes

@houqp
Copy link
Contributor Author

houqp commented Oct 9, 2019

@alexsomesan I noticed other resource types like deployment also have this issue, do mind if I work on a PR to address all of them?

@alexsomesan
Copy link
Member

That would be really nice! By all means, please do.
Much appreciated!

@ghost ghost locked and limited conversation to collaborators Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants