feat: make hostNetwork configurable (#107)

* disable it by default (fixes port conflict issue on the node if another port is trying to acquire the same port)
* chore: update `VERSION` in the `Makefile`
* chore: replace `hostNetwork: true` with `hostNetwork: false`
* fix: template substituion for `hostNetwork`

Author: vadasambar

Makefile

@@ -1,4 +1,4 @@
-VERSION ?= v1.0.0
+VERSION ?= v1.0.1
 
 IMAGE_BUILDER ?= docker
 IMAGE_BUILD_CMD ?= buildx

charts/warm-metal-csi-driver/Chart.yaml

@@ -15,9 +15,9 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 1.0.0
+version: 1.0.1
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
-appVersion: v1.0.0
+appVersion: v1.0.1

charts/warm-metal-csi-driver/templates/nodeplugin.yaml

@@ -122,7 +122,7 @@ spec:
             - mountPath: {{ .Values.crioMountProgram }}
               name: crio-mount-program
             {{- end }}
-      hostNetwork: true
+      hostNetwork: {{.Values.csiPlugin.hostNetwork}}
       serviceAccountName: {{ include "warm-metal-csi-driver.fullname" . }}-nodeplugin
       volumes:
         - hostPath:

charts/warm-metal-csi-driver/values.yaml

@@ -9,6 +9,7 @@ enableAsyncPullMount: false
 pullImageSecretForDaemonset:
 
 csiPlugin:
+  hostNetwork: false
   resources: {}
   image:
     tag: ""

cmd/install/main.go

@@ -278,7 +278,7 @@ spec:
       labels:
         app: csi-image-warm-metal
     spec:
-      hostNetwork: true
+      hostNetwork: false
       serviceAccountName: csi-image-warm-metal
       containers:
         - name: node-driver-registrar

pkg/pullexecutor/pullexecutor.go

@@ -102,7 +102,8 @@ func (m *PullExecutor) StartPulling(o *PullOptions) error {
 			c, cancel := context.WithTimeout(context.Background(), pullCtxTimeout)
 			defer cancel()
 
-			if pullstatus.Get(o.NamedRef) == pullstatus.StillPulling {
+			if pullstatus.Get(o.NamedRef) == pullstatus.StillPulling ||
+				pullstatus.Get(o.NamedRef) == pullstatus.Pulled {
 				return
 			}
 

sample/install/cri-o.yaml

@@ -126,7 +126,7 @@ spec:
             - mountPath: /run/containers/storage
               mountPropagation: Bidirectional
               name: crio-run-root
-      hostNetwork: true
+      hostNetwork: false
       serviceAccountName: csi-image-warm-metal
       volumes:
         - hostPath:

sample/install/k3s-containerd.yaml

@@ -117,7 +117,7 @@ spec:
             - mountPath: /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs
               mountPropagation: Bidirectional
               name: snapshot-root-0
-      hostNetwork: true
+      hostNetwork: false
       serviceAccountName: csi-image-warm-metal
       volumes:
         - hostPath:

sample/install/minikube-containerd.yaml

@@ -123,7 +123,7 @@ spec:
             - mountPath: /mnt/vda1/var/lib/containerd/io.containerd.snapshotter.v1.overlayfs
               mountPropagation: Bidirectional
               name: snapshot-root-0
-      hostNetwork: true
+      hostNetwork: false
       serviceAccountName: csi-image-warm-metal
       volumes:
         - hostPath:

sample/install/snap-microk8s-containerd.yaml

@@ -117,7 +117,7 @@ spec:
             - mountPath: /var/snap/microk8s/common/var/lib/containerd/io.containerd.snapshotter.v1.overlayfs
               mountPropagation: Bidirectional
               name: snapshot-root-0
-      hostNetwork: true
+      hostNetwork: false
       serviceAccountName: csi-image-warm-metal
       volumes:
         - hostPath: