k8s-daemonset-installer breaks in scenarios with multiple kubelet systemd drop-in files
Open
@rodnymolina is already working on this.
Since Apr 5, 2022.
bug
- Dominant language
- Shell
- Stars
- 3.9k
- Forks
- 230
- Avg merge
- 7h 48m
- Merged PRs (30d)
- 3
Description
I have no idea where the /usr/local/bin/kubelet-config-helper.sh file comes from (can't find it via searches in nestybox organization?), but it contains this little gem:
# Let's look first at the drop-in file first as it has more preference.
#
# TODO: What about scenarios with multiple dropin-files?
In the situation where /etc/systemd/system/kubelet.service.d/ contains more than one file, this setup script will later attempt to move tmp.txt to /etc/systemd/system/kubelet.service.d/dropin1 /etc/systemd/system/kubelet.service.d/dropin2.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.