Skip to content

Commit 97b0b69

Browse files
KFLUXINFRA-3694 Simplify to FIFO scheduling for signing pipelines (#340)
Remove the konflux-signing WorkloadPriorityClass and simplify the CEL priority expression to assign konflux-default to all pipelines. Kueue's BestEffortFIFO strategy handles ordering by creation time, which is the desired scheduling behavior for the signing use case (ISV-6622). The signing-server-request resource quota remains for independent concurrency control. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent d8cc0ad commit 97b0b69

2 files changed

Lines changed: 0 additions & 13 deletions

File tree

components/kueue/internal-staging/queue-config/workload-priority-class.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,6 @@
11
---
22
apiVersion: kueue.x-k8s.io/v1beta1
33
kind: WorkloadPriorityClass
4-
metadata:
5-
name: konflux-signing
6-
value: 950
7-
description: "High priority for signing pipelines rate-limited via ISV-6622"
8-
---
9-
apiVersion: kueue.x-k8s.io/v1beta1
10-
kind: WorkloadPriorityClass
114
metadata:
125
name: konflux-default
136
value: 400

components/kueue/internal-staging/tekton-kueue/config.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,6 @@ queueName: pipelines-queue
22
cel:
33
expressions:
44
- |
5-
has(pipelineRun.metadata.labels) &&
6-
'internal-services.appstudio.openshift.io/rate-limited' in pipelineRun.metadata.labels &&
7-
pipelineRun.metadata.labels['internal-services.appstudio.openshift.io/rate-limited'] == 'true' &&
8-
'internal-services.appstudio.openshift.io/rate-limiting-group' in pipelineRun.metadata.labels &&
9-
pipelineRun.metadata.labels['internal-services.appstudio.openshift.io/rate-limiting-group'] == 'signing-server' ?
10-
priority('konflux-signing') :
115
priority('konflux-default')
126
137
- |

0 commit comments

Comments
 (0)