Closed rohantmp closed 16 hours ago
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | 81fa5e42845a62d400126b0ed9be993e3b4c2cfd |
Latest deploy log | https://app.netlify.com/sites/vcluster-docs/deploys/673f09a5de23b90008bed865 |
I'll move it into a destroy folder and move some of the common options out into clihelper
~As I was looking into refactoring, I'm starting to think it makes sense to have start and destroy together. It's is like having install and uninstall together. There are lots of common things and cli argument sanity checking.~
~Feels a bit awkward to put sanity checking of args that hold the same set of requirements and clients in a destroy package.~ That said, I could leave that bit in start and just import it from destroy.
Ready to review
Oh wait, there's more capitlization pointed out I missed
Done, fixed capitalization and "loft" mentions in the touched files
What issue type does this pull request address? (keep at least one, remove the others) /kind feature
What does this pull request do? Which issues does it resolve? (use
resolves #<issue_number>
if possible) resolves ENG-3980Please provide a short message that should be published in the vcluster release notes Add a new cli command: vcluster platform destroy
What else do we need to know?
The command will wait for each resource type to finish cleaning up before proceeding to the next, so that resource types that depend on each other will not prevent cleanup. (Remove a cluster object before a virtualclusterinstance running on it, etc)
vcluster platform destroy
(no args) on v4.0.0:vcluster platform destroy
(no args) on v3.4.9: