-
Notifications
You must be signed in to change notification settings - Fork 716
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
kubeadm join can not mix '--config' with arguments [experimental-control-plane] #1485
Comments
@marccarre in terms of:
and
what is your use case for requiring the this is the address : port you specify here: $CERT_KEY should be passed to so if you don't pass a
|
@neolit123, I currently pass the following via apiVersion: kubeadm.k8s.io/v1beta1
kind: JoinConfiguration
discovery:
bootstrapToken:
apiServerEndpoint: $SEED_MASTER_IP:$PORT
caCertHashes:
- sha256:$CERT_HASH
token: $TOKEN
nodeRegistration:
kubeletExtraArgs:
node-ip: $IP Is there a CLI equivalent to the below? nodeRegistration:
kubeletExtraArgs:
node-ip: $IP I couldn't find any in |
although under normal circumstances you don't need to specify the |
Many thanks for the pointers 👍
Unfortunately, I have abnormal circumstances 🙂 |
FWIW,
joining with the CLI flags [2]
[1]: where I run [2]:
|
Out of curiosity, what is the target release date for 1.15 / this improvement? |
1.14 was released 2 weeks ago so roughly 3 months minus 2 weeks. |
/assign |
a) Have you used $ cat /etc/sysconfig/kubelet
KUBELET_EXTRA_ARGS="--fail-swap-on=false --node-ip=$PRIVATE_IP" $ kubeadm join --token $TOKEN \
--discovery-token-ca-cert-hash sha256:$CERT_HASH \
--experimental-control-plane \
--apiserver-advertise-address=$PRIVATE_IP \
(--certificate-key $CERT_KEY) \
$SEED_MASTER_PRIVATE_IP:6443 Anyways, I think there is an issue in that the current code in apiVersion: kubeadm.k8s.io/v1beta1
kind: JoinConfiguration
discovery:
bootstrapToken:
apiServerEndpoint: $SEED_MASTER_IP:$PORT
caCertHashes:
- sha256:$CERT_HASH
token: $TOKEN
nodeRegistration:
kubeletExtraArgs:
node-ip: $IP
controlPlane:
localAPIEndpoint:
advertiseAddress: $IP That is a bug. Enabling the control plane mode via the config file (even by just doing |
@neolit123, @luxas thanks for your help & suggestions!
@luxas, for the record & to answer your questions:
|
thanks for the feedback @marccarre
these changes cannot be backported to <1.15 as they fall under for additional issues please open new tickets. |
What keywords did you search in kubeadm issues before filing this one?
can not mix '--config' with arguments [experimental-control-plane]
experimental
certificate
Is this a BUG REPORT or FEATURE REQUEST?
BUG REPORT or maybe neither and more a documentation issue?
Versions
kubeadm version:
Environment:
uname -a
):What happened?
When trying to setup a HA cluster with stacked control plane and etcd nodes, one needs to provide the following options to
kubeadm join
for master nodes:--experimental-control-plane
--certificate-key $CERT_KEY
However if one is already passing configuration via
--config
, the following error is returned:According to #1251, and if I understood correctly, the below can be added to the
JoinConfiguration
object instead of passing--experimental-control-plane
:However, it is unclear where
$CERT_KEY
should be added.kubeadm
viaJoinConfiguration
too? If so, where?v1beta1
's types. MaybeExternalEtcd.KeyFile
?N.B.:
--certificate-key $CERT_KEY
, joining fails with:--certificate-key $CERT_KEY
anyway doesn't fail withcan not mix
but the value seems to be ignored as it eventually also times out with:What you expected to happen?
The machine to join the cluster as a master node in order to form a HA cluster with stacked control plane and etcd nodes.
How to reproduce it (as minimally and precisely as possible)?
The text was updated successfully, but these errors were encountered: