Return gardener user error for unmanaged resource on cluster delete#13
Return gardener user error for unmanaged resource on cluster delete#13aniruddha2000 wants to merge 10 commits intomainfrom
Conversation
# Conflicts: # pkg/apis/stackit/helper/error_codes.go
nschad
left a comment
There was a problem hiding this comment.
Add the error cases like SubnetInUse to dependenciesRegexp. Don't change the the infraflow part. Don't call DetermineError multiple times.
It's already called here:
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
3deafb8 to
749384b
Compare
There was a problem hiding this comment.
Please do it like this
gardener-extension-provider-stackit/pkg/openstack/client/client.go
Lines 216 to 226 in 281fccb
Just add a util function within the same file.
How to categorize this PR?
/kind enhancement
What this PR does / why we need it:
this PR handles graceful deletion for 409 error code returned by STACKIT IaaS API and converts it to relevent gardener error.
Special notes for your reviewer:
Breaking changes: