-
Notifications
You must be signed in to change notification settings - Fork 356
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
Feat/make all masters equal and make join cmd more robust #2218
Conversation
Signed-off-by: huaiyou <huaiyou.cyz@alibaba-inc.com>
Signed-off-by: huaiyou <huaiyou.cyz@alibaba-inc.com>
Signed-off-by: huaiyou <huaiyou.cyz@alibaba-inc.com>
175d68d
to
116f29a
Compare
Codecov ReportPatch coverage has no change and project coverage change:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #2218 +/- ##
==========================================
- Coverage 19.93% 13.47% -6.46%
==========================================
Files 98 263 +165
Lines 9220 22757 +13537
==========================================
+ Hits 1838 3067 +1229
- Misses 7128 19280 +12152
- Partials 254 410 +156
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
if err := k.infra.Copy(host, k.getPKIPath(), clustercert.KubeDefaultCertPath); err != nil { | ||
return fmt.Errorf("failed to copy cluster cert: %v", err) | ||
} | ||
if err := k.infra.Copy(host, k.getPKIPath(), k.getPKIPath()); err != nil { |
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.
What does this mean?
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.
transform pki to all master's rootfs, so when master0 down, other master can scale up cluster.
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
This PR support the follow abilities:
Does this pull request fix one issue?
Describe how you did it
Describe how to verify it
Special notes for reviews