kcp
kcp copied to clipboard
:bug: tenancy/workspace-deletion: try harder with the admin client to delete the LogicalCluster
Summary
The front-proxy might return errors when looking for the LogicalCluster
on workspace deletion,
especially when the LogicalCluster
is already gone, and it returns a 403. This PR adds another attempt through the workspace admin client.
Related issue(s)
Fixes #
Release Notes
Make workspace deletion more reliable, trying harder to not leak `LogicalClusters`.
@yastij: changing LGTM is restricted to collaborators
In response to this:
/lgtm
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
Issues go stale after 90d of inactivity.
After a furter 30 days, they will turn rotten.
Mark the issue as fresh with /remove-lifecycle stale
.
If this issue is safe to close now please do so with /close
.
/lifecycle stale
Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten
.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close
.
/lifecycle rotten
Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen
.
Mark the issue as fresh with /remove-lifecycle rotten
.
/close
@kcp-ci-bot: Closed this PR.
In response to this:
Rotten issues close after 30d of inactivity. Reopen the issue with
/reopen
. Mark the issue as fresh with/remove-lifecycle rotten
./close
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
/reopen
@sttts: Reopened this PR.
In response to this:
/reopen
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sttts
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [sttts]
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment