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

Enable overriding the grace periods for cleanup steps in shoot deletion #4212

Merged
merged 3 commits into from Jun 22, 2021

Conversation

rfranzke
Copy link
Member

How to categorize this PR?

/area usability ops-productivity
/kind enhancement

What this PR does / why we need it:
It's now possible to override the grace periods for the cleanup steps in the shoot deletion by specifying the following annotations on the Shoot:

  • shoot.gardener.cloud/cleanup-webhooks-finalize-grace-period-seconds (default behaviour: "300")
  • shoot.gardener.cloud/cleanup-extended-apis-finalize-grace-period-seconds (default behaviour: "3600")
  • shoot.gardener.cloud/cleanup-kubernetes-resources-finalize-grace-period-seconds (default behaviour: "300")
  • shoot.gardener.cloud/cleanup-namespaces-finalize-grace-period-seconds (default behaviour: "300")

If "0" is provided then all resources are finalized immediately without waiting for any graceful deletion. Please be aware that this might lead to orphaned infrastructure artefacts.

Which issue(s) this PR fixes:
Part 2 of #3110 (comment)

Release note:

It's now possible to override the grace periods for the cleanup steps in the shoot deletion by specifying the following annotations on the `Shoot`:

- `shoot.gardener.cloud/cleanup-webhooks-finalize-grace-period-seconds` (default behaviour: `"300"`)
- `shoot.gardener.cloud/cleanup-extended-apis-finalize-grace-period-seconds` (default behaviour: `"3600"`)
- `shoot.gardener.cloud/cleanup-kubernetes-resources-finalize-grace-period-seconds` (default behaviour: `"300"`)
- `shoot.gardener.cloud/cleanup-namespaces-finalize-grace-period-seconds` (default behaviour: `"300"`)

If `"0"` is provided then all resources are finalized immediately without waiting for any graceful deletion. Please be aware that this might lead to orphaned infrastructure artefacts.

@rfranzke rfranzke requested a review from a team as a code owner June 14, 2021 12:17
@gardener-robot gardener-robot added kind/api-change API change with impact on API users needs/second-opinion area/ops-productivity Operator productivity related (how to improve operations) area/usability Usability related kind/enhancement Enhancement, improvement, extension size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 14, 2021
@timebertt
Copy link
Member

/assign

Copy link
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

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

Nice! Generally lgtm, just one comment :)

pkg/operation/botanist/cleanup.go Show resolved Hide resolved
@rfranzke
Copy link
Member Author

/squash

@rfranzke
Copy link
Member Author

cc @pbochynski - you might be interested in this PR

Copy link
Member

@timebertt timebertt left a comment

Choose a reason for hiding this comment

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

Thanks!
/lgtm

@rfranzke rfranzke merged commit d01615c into gardener:master Jun 22, 2021
@rfranzke rfranzke deleted the enh/grace-period-configuration branch June 22, 2021 11:16
krgostev pushed a commit to krgostev/gardener that referenced this pull request Apr 21, 2022
…on (gardener#4212)

* Enable overriding the grace periods for cleanup steps in shoot deletion

* Improve signature of getCleanOptions

* Ignore grace periods larger than the defaults
krgostev pushed a commit to krgostev/gardener that referenced this pull request Jul 5, 2022
…on (gardener#4212)

* Enable overriding the grace periods for cleanup steps in shoot deletion

* Improve signature of getCleanOptions

* Ignore grace periods larger than the defaults
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ops-productivity Operator productivity related (how to improve operations) area/usability Usability related kind/api-change API change with impact on API users kind/enhancement Enhancement, improvement, extension size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants