Merge pull request #1826 from mercedes-benz/feature-1825-pds-owaspzap…

…-one-scan-at-a-time workerThreadCount is pinned to 1 #1825
mercedes-benz · Dec 16, 2022 · 860977f · 860977f
2 parents d3a317f + 980492a
commit 860977f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/Chart.yaml b/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/Chart.yaml
@@ -13,4 +13,4 @@ maintainers:
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.12.0
+version: 0.13.0
diff --git a/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/templates/deployment.yaml b/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/templates/deployment.yaml
@@ -45,8 +45,9 @@ spec:
                       value: "{{ .Values.zap.apiKey }}"
                     - name: PDS_CONFIG_EXECUTE_QUEUE_MAX
                       value: "{{ .Values.pds.config.execute.queueMax }}"
+                    # Only one scan in parallel possible per pds-owaspzap instance
                     - name: PDS_CONFIG_EXECUTE_WORKER_THREAD_COUNT
-                      value: "{{ .Values.pds.config.execute.workerThreadCount }}"
+                      value: "1"
                     - name: ZAP_PROXY_HOST
                       value: "{{ .Values.zap.proxy.host }}"
                     - name: ZAP_PROXY_PORT

diff --git a/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/values.yaml b/sechub-pds-solutions/owasp-zap/helm/pds-owaspzap/values.yaml
@@ -19,7 +19,7 @@ pds:
     config:
         execute:
             queueMax: 10
-            workerThreadCount: 1
+            # workerThreadCount is pinned to 1
     heartbeatLogging: "true"
     logging:
         type: