Merge pull request #120 from andyzhangx/format-yaml-files

doc: format yaml files

charts/latest/csi-driver-nfs/values.yaml

@@ -1,7 +1,7 @@
 image:
     nfs:
-        repository: gcr.io/k8s-staging-sig-storage/nfsplugin
+        repository: mcr.microsoft.com/k8s/csi/nfs-csi
-        tag: amd64-linux-canary
+        tag: latest
         pullPolicy: IfNotPresent
     csiProvisioner:
         repository: k8s.gcr.io/sig-storage/csi-provisioner

deploy/csi-nfs-controller.yaml

@@ -61,7 +61,7 @@ spec:
               cpu: 10m
               memory: 20Mi
         - name: nfs
-          image: gcr.io/k8s-staging-sig-storage/nfsplugin:amd64-linux-canary
+          image: mcr.microsoft.com/k8s/csi/nfs-csi:latest
           securityContext:
             privileged: true
             capabilities:

deploy/csi-nfs-node.yaml

@@ -63,7 +63,7 @@ spec:
             capabilities:
               add: ["SYS_ADMIN"]
             allowPrivilegeEscalation: true
-          image: gcr.io/k8s-staging-sig-storage/nfsplugin:amd64-linux-canary
+          image: mcr.microsoft.com/k8s/csi/nfs-csi:latest
           args:
             - "-v=5"
             - "--nodeid=$(NODE_ID)"

deploy/example/deployment.yaml

@@ -16,29 +16,29 @@ kind: Deployment
 metadata:
   name: deployment-nfs
 spec:
-  replicas: 3
+  replicas: 1
   selector:
     matchLabels:
-        name: deployment-nfs
+      name: deployment-nfs
   template:
     metadata:
+      name: deployment-nfs
+      labels:
         name: deployment-nfs
-        labels:
-            name: deployment-nfs
     spec:
       nodeSelector:
         "kubernetes.io/os": linux
       containers:
-      - name: deployment-nfs
+        - name: deployment-nfs
-        image: mcr.microsoft.com/oss/nginx/nginx:1.17.3-alpine
+          image: mcr.icrosoft.com/oss/nginx/nginx:1.19.5
-        command:
+          command:
-          - "/bin/sh"
+            - "/bin/bash"
-          - "-c"
+            - "-c"
-          - while true; do echo $(hostname) $(date) >> /mnt/nfs/outfile; sleep 1; done
+            - set -euo pipefail; while true; do echo $(hostname) $(date) >> /mnt/nfs/outfile; sleep 1; done
-        volumeMounts:
+          volumeMounts:
-          - name: nfs
+            - name: nfs
-            mountPath: "/mnt/nfs"
+              mountPath: "/mnt/nfs"
       volumes:
-      - name: nfs
+        - name: nfs
-        persistentVolumeClaim:
+          persistentVolumeClaim:
-          claimName: pvc-deployment-nfs
+            claimName: pvc-deployment-nfs

deploy/example/nfs-provisioner/nginx-pod.yaml

@@ -39,15 +39,15 @@ metadata:
   name: nginx-nfs-example
 spec:
   containers:
-  - image: nginx
+    - image: nginx
-    name: nginx
+      name: nginx
-    ports:
+      ports:
-    - containerPort: 80
+        - containerPort: 80
-      protocol: TCP
+          protocol: TCP
-    volumeMounts:
+      volumeMounts:
-      - mountPath: /var/www
+        - mountPath: /var/www
-        name: pvc-nginx
+          name: pvc-nginx
   volumes:
-  - name: pvc-nginx
+    - name: pvc-nginx
-    persistentVolumeClaim:
+      persistentVolumeClaim:
-      claimName: pvc-nginx
+        claimName: pvc-nginx

deploy/example/statefulset.yaml

@@ -7,7 +7,7 @@ metadata:
     app: nginx
 spec:
   serviceName: statefulset-nfs
-  replicas: 3
+  replicas: 1
   template:
     metadata:
       labels:
@@ -17,11 +17,11 @@ spec:
         "kubernetes.io/os": linux
       containers:
         - name: statefulset-nfs
-          image: mcr.microsoft.com/oss/nginx/nginx:1.17.3-alpine
+          image: mcr.microsoft.com/oss/nginx/nginx:1.19.5
           command:
-            - "/bin/sh"
+            - "/bin/bash"
             - "-c"
-            - while true; do echo $(date) >> /mnt/nfs/outfile; sleep 1; done
+            - set -euo pipefail; while true; do echo $(date) >> /mnt/nfs/outfile; sleep 1; done
           volumeMounts:
             - name: persistent-storage
               mountPath: /mnt/nfs

deploy/example/storageclass-nfs.yaml

@@ -7,7 +7,7 @@ provisioner: nfs.csi.k8s.io
 parameters:
   server: nfs-server.default.svc.cluster.local
   share: /
-reclaimPolicy: Retain  # only retain is supported
+reclaimPolicy: Retain
 volumeBindingMode: Immediate
 mountOptions:
   - hard

hack/verify-yamllint.sh

@@ -29,7 +29,7 @@ if [[ "${deployDirNum}" != "${helmDirNum}" ]]; then
   exit 1
 fi
 
-for path in "deploy/*.yaml"
+for path in "deploy/*.yaml" "deploy/example/*.yaml" "deploy/example/nfs-provisioner/*.yaml"
 do
     echo "checking yamllint under path: $path ..."
     yamllint -f parsable $path | grep -v "line too long" > $LOG