Merge pull request #337 from raif-ahmed/master

add helm parameter to enable 'ENFORCER_STATS' env variable

charts/monitor/README.md

@@ -17,7 +17,7 @@ Parameter | Description | Default | Notes
 `exporter.ctrlSecretName` | existing secret that have CTRL_USERNAME and CTRL_PASSWORD fields to login to the controller.  | `nil` | if parameter exists then `exporter.CTRL_USERNAME` & `exporter.CTRL_PASSWORD` will be skipped
 `exporter.CTRL_USERNAME` | Username to login to the controller. Suggest to replace the default admin user to a read-only user | `admin` |
 `exporter.CTRL_PASSWORD` | Password to login to the controller. | `admin` |
-
+`exporter.enforcerStats.enabled` | If true, enable the Enforcers stats | `false` | For the performance reason, by default the exporter does NOT pull CPU/memory usage from enforcers.
 ---
 Contact <[email protected]> for access to Docker Hub and docs.
 

charts/monitor/templates/exporter-deployment.yaml

@@ -49,6 +49,10 @@ spec:
               value: {{ .Values.exporter.apiSvc }}
             - name: EXPORTER_PORT
               value: "8068"
+            {{- if .Values.exporter.enforcerStats }}
+            - name: ENFORCER_STATS
+              value: "{{.Values.exporter.enforcerStats.enabled | default "false"}}"
+            {{- end }}
           envFrom:
             - secretRef:
             {{- if .Values.exporter.ctrlSecretName }}

charts/monitor/values.yaml

@@ -15,8 +15,10 @@ exporter:
   # changes this to a readonly user !
   CTRL_USERNAME: admin
   CTRL_PASSWORD: admin
+  ctrlSercretName: ''
+  enforcerStats:
+    enabled: false
   ctrlSecretName: ''
-
   apiSvc: neuvector-svc-controller-api:10443
 
   svc: