-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Tweak kubeadm release notes #221
Tweak kubeadm release notes #221
Conversation
Signed-off-by: Joe Beda <joe.github@bedafamily.com>
I didn't list every changed command-line flag as I think it would create more noise than be helpful. |
Also /cc @luxas @pires @errordeveloper |
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.
external etcd settings and the option for setting the cloud provider on the | ||
API server. The [kubeadm reference | ||
documentation](https://kubernetes.io/docs/admin/kubeadm/) is up to date with | ||
the new flags. | ||
- A new label and taint is used for marking the master. The label is | ||
`node-role.kubernetes.io/master=""` and the taint has the effect |
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.
We should remove and the taint has the effect NoSchedule
, it doesn't make sense in this sentence.
[redact comment, can't read good] |
Add some more detail and references.
This is a continuation of #211.
Astute readers will notice that I accidentally checked this in directly and then reverted it. Sorry about that.