Merge pull request #157 from andyzhangx/loglevel

set 200Mi limit for csi-provisioner

Makefile

@@ -115,6 +115,8 @@ e2e-bootstrap: install-helm
 	--set image.nfs.repository=$(REGISTRY)/$(IMAGENAME) \
 	--set image.nfs.tag=$(IMAGE_VERSION) \
 	--set image.nfs.pullPolicy=Always
+	--set controller.logLevel=8
+	--set node.logLevel=8
 
 .PHONY: e2e-teardown
 e2e-teardown:

charts/README.md

@@ -47,6 +47,8 @@ The following table lists the configurable parameters of the latest NFS CSI Driv
 | `rbac.create`                                     | whether create rbac of csi-nfs-controller                  | true                                                              |
 | `controller.replicas`                             | the replicas of csi-nfs-controller                         | 2                                                                 |
 | `controller.runOnMaster`                          | run controller on master node                              | false                                                             |
+| `controller.logLevel`                             | controller driver log level                                                          |`5`                                                           |
+| `node.logLevel`                                   | node driver log level                                                          |`5`                                                           |
 
 ## troubleshooting
  - Add `--wait -v=5 --debug` in `helm install` command to get detailed error

charts/latest/csi-driver-nfs/templates/csi-nfs-controller.yaml

@@ -44,7 +44,7 @@ spec:
           resources:
             limits:
               cpu: 100m
-              memory: 100Mi
+              memory: 200Mi
             requests:
               cpu: 10m
               memory: 20Mi
@@ -75,7 +75,7 @@ spec:
             allowPrivilegeEscalation: true
           imagePullPolicy: {{ .Values.image.nfs.pullPolicy }}
           args:
-            - "-v=5"
+            - "--v={{ .Values.controller.logLevel }}"
             - "--nodeid=$(NODE_ID)"
             - "--endpoint=$(CSI_ENDPOINT)"
           env:

charts/latest/csi-driver-nfs/templates/csi-nfs-node.yaml

@@ -67,7 +67,7 @@ spec:
             allowPrivilegeEscalation: true
           image: "{{ .Values.image.nfs.repository }}:{{ .Values.image.nfs.tag }}"
           args :
-            - "-v=5"
+            - "--v={{ .Values.node.logLevel }}"
             - "--nodeid=$(NODE_ID)"
             - "--endpoint=$(CSI_ENDPOINT)"
           env:

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

@@ -22,3 +22,7 @@ rbac:
 controller:
   replicas: 2
   runOnMaster: false
+  logLevel: 5
+
+node:
+  logLevel: 5

deploy/csi-nfs-controller.yaml

@@ -39,7 +39,7 @@ spec:
           resources:
             limits:
               cpu: 100m
-              memory: 100Mi
+              memory: 200Mi
             requests:
               cpu: 10m
               memory: 20Mi

pkg/nfs/utils.go

@@ -74,7 +74,7 @@ func getLogLevel(method string) int32 {
 	if method == "/csi.v1.Identity/Probe" ||
 		method == "/csi.v1.Node/NodeGetCapabilities" ||
 		method == "/csi.v1.Node/NodeGetVolumeStats" {
-		return 10
+		return 8
 	}
 	return 2
 }

pkg/nfs/utils_test.go

@@ -91,15 +91,15 @@ func TestGetLogLevel(t *testing.T) {
 	}{
 		{
 			method: "/csi.v1.Identity/Probe",
-			level:  10,
+			level:  8,
 		},
 		{
 			method: "/csi.v1.Node/NodeGetCapabilities",
-			level:  10,
+			level:  8,
 		},
 		{
 			method: "/csi.v1.Node/NodeGetVolumeStats",
-			level:  10,
+			level:  8,
 		},
 		{
 			method: "",