feature: Add a "kubectl kcp workspace delete" command

[manoelmarques]

Feature Description

Even though one can delete a workspace with kubectl delete workspace myworkspace, I believe there should be also a kubectl kcp workspace delete myworkspace command since there is already a create command.

Proposed Solution

I already have the code implementing it, just need to create formal unit tests now.

In case this issue is approved, I could immediately create a WIP PR with the implementation while I work on the unit tests.

Alternative Solutions

No response

Want to contribute?

• I would like to work on this issue.

Additional Context

No response

[ncdc]

We used to have one, because it required extra logic beyond what kubectl delete workspace does. Once we removed the need for that extra logic (it was in #778), we removed the subcommand, favoring having the users use kubectl delete.

I do, however, understand the lack of symmetry between kubectl ws create and kubectl delete workspace... @sttts I believe you did the PR to remove kubectl ws delete ... have any revised thoughts at this time? cc @stevekuznetsov @davidfestal

[embik]

A possible alternative to explore is looking into a custom kubectl create subcommand, which might be available by now.

[sttts]

A possible alternative to explore is looking into a custom kubectl create subcommand, which might be available by now.

Background: in the past kubectl did not call out on kubectl create foo to kubectl-create-foo, only on top-level commands. Afaik this has changed like few months ago. So you can now place a custom create func and kubectl will use it.

[embik]

A possible alternative to explore is looking into a custom kubectl create subcommand, which might be available by now.

Background: in the past kubectl did not call out on kubectl create foo to kubectl-create-foo, only on top-level commands. Afaik this has changed like few months ago. So you can now place a custom create func and kubectl will use it.

FYI, this is still not possible according to documentation.

[sttts]

Works with kubectl 1.29.0.

[embik]

Ohhh so upstream documentation is outdated. Interesting, thank you for checking.