Skip to content
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

refactor: introduce vCluster manager #1710

Merged
merged 4 commits into from
Apr 23, 2024
Merged

Conversation

FabianKramm
Copy link
Member

@FabianKramm FabianKramm commented Apr 22, 2024

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
Resolves ENG-3491
Resolves ENG-3493

Changes:

  • New vcluster use manager helm|platform command
  • Added platform functionality for vcluster create, vcluster connect, vcluster list, vcluster pause, vcluster resume & vcluster delete
  • Renamed vcluster import -> vcluster activate
  • Renamed vcluster pro -> vcluster platform
  • Removed vcluster pro generate admin-kube-config
  • Renamed platform.apiKey -> platform.api
  • Moved cli functionality from cmd/vclusterctl/cmd to pkg/cli
  • Make e2e tests reusable for vcluster-pro (move actual tests into packages, so they can be imported separately)
  • Moved pkg/config/legacyconfig -> config/legacyconfig

Copy link

netlify bot commented Apr 22, 2024

Deploy Preview for vcluster-docs canceled.

Name Link
🔨 Latest commit 90fd3af
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/6627703ae1cd6b000763c894

@FabianKramm FabianKramm force-pushed the refactor branch 2 times, most recently from 5a2b38b to 2977ac4 Compare April 22, 2024 11:54
@FabianKramm FabianKramm force-pushed the refactor branch 3 times, most recently from b3d892e to ce015cf Compare April 22, 2024 12:22
@FabianKramm FabianKramm force-pushed the refactor branch 2 times, most recently from f10823a to c3f828d Compare April 22, 2024 12:32
@heiko-braun
Copy link
Contributor

Do we keep vcluster activate?

@FabianKramm
Copy link
Member Author

@heiko-braun yes, why shouldn't we?

@FabianKramm FabianKramm force-pushed the refactor branch 5 times, most recently from 88dc92d to ef8174c Compare April 22, 2024 14:36
Copy link
Contributor

@johannesfrey johannesfrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly comments. I'm sure I missed cases of vCluster vs. virtual cluster. vCluster being the product/CLI vs. an actual virtual cluster that is running/created/updated etc.

chart/values.yaml Outdated Show resolved Hide resolved
cmd/vclusterctl/cmd/activate.go Show resolved Hide resolved
pkg/platform/client.go Outdated Show resolved Hide resolved
pkg/platform/client.go Outdated Show resolved Hide resolved
pkg/platform/client.go Show resolved Hide resolved
pkg/cli/activate_platform.go Outdated Show resolved Hide resolved
cmd/vclusterctl/cmd/platform/pro.go Outdated Show resolved Hide resolved
cmd/vclusterctl/cmd/resume.go Outdated Show resolved Hide resolved
cmd/vclusterctl/cmd/use/manager.go Outdated Show resolved Hide resolved
cmd/vclusterctl/cmd/use/manager.go Outdated Show resolved Hide resolved
@FabianKramm FabianKramm force-pushed the refactor branch 8 times, most recently from 6deaf5e to d8a7fe8 Compare April 23, 2024 08:04
@FabianKramm FabianKramm merged commit 464ec88 into loft-sh:main Apr 23, 2024
71 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants