Skip to content

Commit 5a825d4

Browse files
authored
ci: use valid pipelineruns in tekton-kueue tests (#9531)
Tekton-kueue expects all pipelineruns it processes to be valid (as defined by upstream tekton). Unfortunately, some of the pipelineruns used in the tests in this repository don't meet that requirement. Update them to be valid. This will enable #9393 to merge, since it's blocked on this issue. Fixes: KFLUXINFRA-2652 Signed-off-by: Andy Sadler <[email protected]>
1 parent a6a91e7 commit 5a825d4

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

hack/test-tekton-kueue-config.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,6 +276,7 @@ def check_prerequisites(should_print: bool = True) -> Dict[str, TestConfig]:
276276
},
277277
"spec": {
278278
"pipelineSpec": {
279+
"description": "foo", # a completely empty pipelineSpec is not allowed
279280
"tasks": None,
280281
},
281282
"workspaces": [{"name": "shared-workspace", "emptyDir": {}}]
@@ -304,6 +305,7 @@ def check_prerequisites(should_print: bool = True) -> Dict[str, TestConfig]:
304305
},
305306
"spec": {
306307
"pipelineSpec": {
308+
"description": "foo", # a completely empty pipelineSpec is not allowed
307309
"tasks": [],
308310
},
309311
"workspaces": [{"name": "shared-workspace", "emptyDir": {}}]
@@ -603,7 +605,6 @@ def check_prerequisites(should_print: bool = True) -> Dict[str, TestConfig]:
603605
}
604606
},
605607
"spec": {
606-
"pipelineRef": {"name": "build-pipeline"},
607608
"params": [
608609
{
609610
"name": "build-platforms",

0 commit comments

Comments
 (0)