Skip to content

Commit d60d61f

Browse files
committed
fix(helm): use /host prefix for CNI config path in container
The Helm template was hardcoding configPath: /etc/cni/net.d which overrode the auto-detection logic in the Go code. Since the host filesystem is mounted at /host in the container, the CNI config path needs to be /host/etc/cni/net.d. This fixes CNI health checks showing False even when CNI is healthy.
1 parent b141cd8 commit d60d61f

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.version-rc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
v1.5.5-rc1
1+
v1.5.5-rc3

helm/node-doctor/templates/configmap.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,8 @@ data:
104104
minReachablePeers: 80
105105
cniHealth:
106106
enabled: true
107-
configPath: /etc/cni/net.d
107+
# Use /host prefix since host filesystem is mounted at /host in container
108+
configPath: /host/etc/cni/net.d
108109
checkInterfaces: true
109110
{{- end }}
110111

0 commit comments

Comments
 (0)