Skip to content

Commit 1c27e58

Browse files
committed
Remove the WORKERS parameter
This task is being used by the release service which overrides the WORKERS parameter. While this change can be made there, it will take time for the parameter modification to make it to production. In order to unblock some users with large Snapshots, I am proposing to remove the configuration altogether. We already increased the default resource requirements for the task, so this worker count should suffice. If this number of workers is too many in the future, users can just override the resources to further increase them. Once the release pipeline users can customize the worker count in the RPA (https://issues.redhat.com/browse/KONFLUX-7850), we can re-enable this parameter. Signed-off-by: arewm <arewm@users.noreply.github.com>
1 parent dad5d39 commit 1c27e58

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

docs/modules/ROOT/pages/verify-conforma-konflux-ta.adoc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,8 @@ paths can be provided by using the `:` separator.
5555
*EXTRA_RULE_DATA* (`string`):: Merge additional Rego variables into the policy data. Use syntax "key=value,key2=value2..."
5656
*TIMEOUT* (`string`):: This param is deprecated and will be removed in future. Its value is ignored. EC will be run without a timeout. (If you do want to apply a timeout use the Tekton task timeout.)
5757

58-
*WORKERS* (`string`):: Number of parallel workers to use for policy evaluation.
58+
*WORKERS* (`string`):: Number of parallel workers to use for policy evaluation. This parameter is currently not used. All policy evaluations are run with 35 workers.
59+
5960
+
6061
*Default*: `35`
6162
*SINGLE_COMPONENT* (`string`):: Reduce the Snapshot to only the component whose build caused the Snapshot to be created

tasks/verify-conforma-konflux-ta/0.1/verify-conforma-konflux-ta.yaml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,9 @@ spec:
127127

128128
- name: WORKERS
129129
type: string
130-
description: Number of parallel workers to use for policy evaluation.
130+
description: >
131+
Number of parallel workers to use for policy evaluation. This parameter is currently not used. All
132+
policy evaluations are run with 35 workers.
131133
default: "35"
132134

133135
- name: SINGLE_COMPONENT
@@ -248,7 +250,8 @@ spec:
248250
- "$(params.REKOR_HOST)"
249251
- "--ignore-rekor=$(params.IGNORE_REKOR)"
250252
- "--workers"
251-
- "$(params.WORKERS)"
253+
# NOTE: This value is temporarily hardcoded instead of using the WORKERS parameter.
254+
- "35"
252255
# NOTE: The syntax below is required to negate boolean parameters
253256
- "--info=$(params.INFO)"
254257
# Fresh versions of ec support "--timeout=0" to indicate no timeout, but this would break

0 commit comments

Comments
 (0)