-
Notifications
You must be signed in to change notification settings - Fork 459
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
Add error code for cluster cleanup #2090
Conversation
cc @gardener/dashboard-maintainers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :) Just a very minor nit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
feee8ac
to
3719a90
Compare
3719a90
to
0b0bb44
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, /approve
What this PR does / why we need it:
This PR adds a new error code
ERR_CLEANUP_CLUSTER_RESOURCES
which indicates that resources in the shoot cluster are stuck in deletion.This allows Gardener users to react accordingly, e.g. the Gardener-Dashboard can show this kind of error as an user related error.
Special notes for your reviewer:
extensions.gardener.cloud
already contains a known error code then this one is reused rather than re-matching the Regex. I see that we need a clear error code contract between Gardener and related extensions but don't want to introduce it in this PR. I'll create an issue and will implement it in further PRs.Release note: