Using jq to parse the default CNI for improved script robustness

Author: cyclinder

charts/spiderpool/templates/configmap.yaml

@@ -172,13 +172,14 @@ data:
         cat /host/etc/cni/net.d/${DEFAULT_CNI_FILEPATH}
 
         echo ""
-        DEFAULT_CNI_NAME=$(grep '"name":' ${CNI_CONF_DIR}/${DEFAULT_CNI_FILEPATH} | awk '{print $2}' | sed -e 's/^[[:space:]]*//' -e 's/[[:space:]]*$//'  | tr -d ',' | tr -d '"')
+
+        DEFAULT_CNI_NAME=$(jq -r '.name' ${CNI_CONF_DIR}/${DEFAULT_CNI_FILEPATH})
         if [ -z "$DEFAULT_CNI_NAME" ] ; then
           error "The name fleid shouldn't be empty, please check the default cni: ${DEFAULT_CNI_FILEPATH}" && exit 1
         fi
 
         log "Updating the clusterNetwork of the multus-cni config to $DEFAULT_CNI_NAME"
-        sed -i "s?\"clusterNetwork\": \"\"?\"clusterNetwork\": \"${DEFAULT_CNI_NAME}\"?g" /tmp/00-multus.conf
+        jq ".clusterNetwork=\"${DEFAULT_CNI_NAME}\"" /tmp/00-multus.conf > /tmp/00-multus.conf.tmp && mv /tmp/00-multus.conf.tmp /tmp/00-multus.conf
     else
       log "User set multus ClusterNetwork: $MULTUS_CLUSTER_NETWORK"
     fi

images/spiderpool-base/install-others.sh

@@ -14,6 +14,7 @@ packages=(
   libmnl0
   bash-completion
   iptables
+  jq
 )