-
Notifications
You must be signed in to change notification settings - Fork 713
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
Make kubeadm upgrade HA ready #706
Comments
Current issue described by this comment: |
@fabriziopandini I'm interested in picking this up but I'm a bit confused about how it would work / what exactly is needed.
|
@mattkelly thanks for helping on this issue! I happily share my personal opinion on how it would work / what exactly is needed, but please consider that it is necessary to get a wider consensus before starting to write a PR for this issue. The key elements of my idea are:
Detail are still TBD but let's discuss them in the breakout session or in slack if there is consensus on the approach 😉 |
I'm not yet qualified to really comment on whether that general approach would be acceptable, but it does seem reasonable to me. We already require a unique MAC address and I agree, let's discuss more at the next breakout session (and people can continue to comment here) before I go off and start implementing. |
@mattkelly feel free to add a new section for next week with an agenda item 🙂 |
Is there any overlap between this and dynamic kubelet config? |
@Kargakis I don't think there is an overlap. Kubeadm applies the same dynamic kubelet config to all nodes, so IMO for the scope of this discussion the dynamic kubelet config is not a node-specific configuration. |
We had a long discussion on this during last weeks call, I think the path forward was a simple proposal which can be linked here as well as a prototype which could help to determine whether a single of multiple config maps makes more sense. |
@timothysc yup, sounds good to me. I wasn't sure if you would have further comments after reviewing the ticket more in-depth. I'll have something out for review within a few days. |
/cc @liztio this will be added as one of the requirements on the config KEP. It's also listed in the kubeadm office hour notes for 20180418 |
@timothysc IMO this issue should be closed as soon as kubernetes/kubernetes#67944 merges |
pinging @fabriziopandini and @timothysc for status. |
Current implementation of the
kubeadm upgrade
relies on thekubeadm-config
configMap created atkubeadm init
time.Such configMap - that is the serialization of the master configuration file -, contains two kind of information:
To make
kubeadm upgrade
working in an HA scenario, with more than one master, the management of the second group of information should be improved, by adding the capability to track information specific to each master node (e.g. more than one nodeName)The text was updated successfully, but these errors were encountered: