Merge pull request #379 from RaunakShah/add-flag-2

Provide option to enable volume mode conversion flag to HostPath driver deployment script

Author: k8s-ci-robot

deploy/kubernetes-1.22/hostpath/csi-hostpath-plugin.yaml

@@ -328,6 +328,7 @@ spec:
             - -v=5
             - --csi-address=/csi/csi.sock
             - --feature-gates=Topology=true
+            # end csi-provisioner args
           securityContext:
             # This is necessary only for systems with SELinux, where
             # non-privileged sidecar containers cannot access unix domain socket

deploy/util/deploy-hostpath.sh

@@ -70,7 +70,7 @@ default_kubelet_data_dir=/var/lib/kubelet
 # implies that refreshing that image has to be done manually.
 #
 # As a special case, 'none' as registry removes the registry name.
-
+# Set VOLUME_MODE_CONVERSION_TESTS to "true" to enable the feature in external-provisioner.
 
 # The default is to use the RBAC rules that match the image that is
 # being used, also in the case that the image gets overridden. This
@@ -138,6 +138,10 @@ function version_gt() {
     test "$(printf '%s' "$versions" | sort -V | head -n 1)" != "$greaterVersion"
 }
 
+function volume_mode_conversion () {
+    [ "${VOLUME_MODE_CONVERSION_TESTS}" == "true" ]
+}
+
 # In addition, the RBAC rules can be overridden separately.
 # For snapshotter 2.0+, the directory has changed.
 SNAPSHOTTER_RBAC_RELATIVE_PATH="rbac.yaml"
@@ -210,6 +214,9 @@ done
 echo "deploying hostpath components"
 for i in $(ls ${BASE_DIR}/hostpath/*.yaml | sort); do
     echo "   $i"
+    if volume_mode_conversion; then
+      sed -i -e 's/# end csi-provisioner args/- \"--prevent-volume-mode-conversion=true\"\n            # end csi-provisioner args/' $i
+    fi
     modified="$(cat "$i" | sed -e "s;${default_kubelet_data_dir}/;${KUBELET_DATA_DIR}/;" | while IFS= read -r line; do
         nocomments="$(echo "$line" | sed -e 's/ *#.*$//')"
         if echo "$nocomments" | grep -q '^[[:space:]]*image:[[:space:]]*'; then