-
Notifications
You must be signed in to change notification settings - Fork 362
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
support cluster scale up via sealer apply cli #1818
support cluster scale up via sealer apply cli #1818
Conversation
Codecov ReportBase: 21.75% // Head: 21.75% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #1818 +/- ##
=======================================
Coverage 21.75% 21.75%
=======================================
Files 72 72
Lines 6554 6554
=======================================
Hits 1426 1426
Misses 4942 4942
Partials 186 186 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
fb8a5f9
to
1332474
Compare
@kakaZhou719 Resolve conflict. |
1332474
to
2f35650
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice work~
cmd/sealer/cmd/cluster/apply.go
Outdated
infraDriver, err := infradriver.NewInfraDriver(&cluster) | ||
if err != nil { | ||
return err | ||
if !osi.IsFileExist(common.DefaultKubeConfigFile()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can check if k8s exists here, more security.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
init k8s client to check ?
9c36d33
to
e02ba08
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Describe what this PR does / why we need it
Does this pull request fix one issue?
Describe how you did it
Describe how to verify it
Special notes for reviews