Skip to content

Commit b67578a

Browse files
KFLUXINFRA-3694 Add CI to validate tekton-kueue CEL expressions
Add a GitHub Actions workflow and Python test script that validates the tekton-kueue CEL configuration for internal-staging and internal-production. Tests run the actual tekton-kueue container via Podman to verify mutations produce correct queue-name labels, priority classes, and resource annotations for signing pipeline rate-limiting. Addresses review feedback: - Use actions/checkout@v6 (filariow) - Tolerate missing config environments gracefully (qodo) - Add 300s timeout to podman subprocess (qodo) - Validate YAML output type before accessing fields (qodo) - Add job-level timeout-minutes: 15 as safety net Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 92ab92a commit b67578a

2 files changed

Lines changed: 561 additions & 0 deletions

File tree

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
name: Test Tekton-Kueue Configuration
2+
3+
on:
4+
pull_request:
5+
branches: [ main ]
6+
paths:
7+
- 'components/kueue/**'
8+
- 'hack/test-tekton-kueue-config.py'
9+
- '.github/workflows/test-tekton-kueue-config.yaml'
10+
11+
jobs:
12+
test-tekton-kueue-config:
13+
name: Test Tekton-Kueue CEL expressions
14+
runs-on: ubuntu-latest
15+
timeout-minutes: 15
16+
steps:
17+
- name: Checkout code
18+
uses: actions/checkout@v6
19+
20+
- name: Set up Python 3
21+
uses: actions/setup-python@v4
22+
with:
23+
python-version: '3.x'
24+
25+
- name: Install Python dependencies
26+
run: |
27+
python -m pip install --upgrade pip
28+
pip install PyYAML
29+
30+
- name: Install Podman
31+
run: |
32+
sudo apt-get update
33+
sudo apt-get install -y podman
34+
35+
- name: Check prerequisites
36+
run: |
37+
python hack/test-tekton-kueue-config.py --check-setup
38+
39+
- name: Run Tekton-Kueue configuration tests
40+
run: |
41+
python hack/test-tekton-kueue-config.py --verbose
42+
43+
- name: Report success
44+
if: success()
45+
run: |
46+
echo "✅ All Tekton-Kueue CEL expression tests passed successfully"

0 commit comments

Comments
 (0)