Add filesystem dependency to kubernetes worker init class
There is a race condition that can lead to kubeadm join
being called before the '/var/lib/kubelet' mount point
is created.
An early run of the join executes ok leaving expected
config and otherwise content in the local /var/lib/kubelet
directory.
Then when the filesystem manifests creates the
/var/lib/kubelet mountpoint the kubernetes config content
created earlier gets hidden by the mount thereby causing
a non-recoverable kubelet process failure due to missing
config.yaml
This update adds a filesystem class dependency in the
kubernetes::worker:init class to ensure the
/var/lib/kubelet mountpoint is created before the
kubeadm join is run.
Test Plan:
PASS: Verify Normal System Install
Change-Id: Ibc110589260c23f86beb5a6eaf1008b3d4f387b3
Closes-Bug: 1843344
Signed-off-by: Eric MacDonald <email address hidden>
Reviewed: https:/ /review. opendev. org/685773 /git.openstack. org/cgit/ starlingx/ stx-puppet/ commit/ ?id=79525f2edd5 6df142de22f6f2e d7fd9ff8577840
Committed: https:/
Submitter: Zuul
Branch: master
commit 79525f2edd56df1 42de22f6f2ed7fd 9ff8577840
Author: Eric MacDonald <email address hidden>
Date: Mon Sep 30 14:34:38 2019 -0400
Add filesystem dependency to kubernetes worker init class
There is a race condition that can lead to kubeadm join
being called before the '/var/lib/kubelet' mount point
is created.
An early run of the join executes ok leaving expected
config and otherwise content in the local /var/lib/kubelet
directory.
Then when the filesystem manifests creates the lib/kubelet mountpoint the kubernetes config content
/var/
created earlier gets hidden by the mount thereby causing
a non-recoverable kubelet process failure due to missing
config.yaml
This update adds a filesystem class dependency in the :worker: init class to ensure the lib/kubelet mountpoint is created before the
kubernetes:
/var/
kubeadm join is run.
Test Plan:
PASS: Verify Normal System Install
Change-Id: Ibc110589260c23 f86beb5a6eaf100 8b3d4f387b3
Closes-Bug: 1843344
Signed-off-by: Eric MacDonald <email address hidden>