Skip to content

Commit 4a41917

Browse files
committed
node dra: reduce the number of automatic pre-submit jobs
It's not necessary to cover all possible runtime variants during pre-submit for all PRs. One of them should be sufficient, the rest can be covered during periodic testing. This cuts down CI costs and noise in unrelated PRs that happen to trigger the run_if_changed.
1 parent 72c1f43 commit 4a41917

File tree

1 file changed

+13
-2
lines changed

1 file changed

+13
-2
lines changed

config/jobs/kubernetes/sig-node/sig-node-presubmit.yaml

+13-2
Original file line numberDiff line numberDiff line change
@@ -3201,6 +3201,7 @@ presubmits:
32013201
# Not relevant for most PRs.
32023202
always_run: false
32033203
# This covers most of the code related to dynamic resource allocation.
3204+
# Periodic variant: ci-kind-dra
32043205
run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/
32053206
# The tests might still be flaky or this job might get triggered accidentally for
32063207
# an unrelated PR.
@@ -3257,6 +3258,7 @@ presubmits:
32573258
# Not relevant for most PRs.
32583259
always_run: false
32593260
# This covers most of the code related to dynamic resource allocation.
3261+
# Periodic variant: ci-kind-dra-all
32603262
run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/
32613263
# The tests might still be flaky or this job might get triggered accidentally for
32623264
# an unrelated PR.
@@ -4276,6 +4278,9 @@ presubmits:
42764278
skip_branches:
42774279
- release-\d+\.\d+ # per-release image
42784280
always_run: false
4281+
# Automatically testing with one container runtime in one configuration is sufficient to detect basic problems in kubelet early.
4282+
# CRI-O was picked because it was solid for testing so far.
4283+
# Periodic variant: ci-node-e2e-crio-cgrpv1-dra-features
42794284
run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/|/test/e2e_node/dra_test.go
42804285
optional: true
42814286
skip_report: false
@@ -4381,7 +4386,10 @@ presubmits:
43814386
skip_branches:
43824387
- release-\d+\.\d+ # per-release image
43834388
always_run: false
4384-
run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/|/test/e2e_node/dra_test.go
4389+
# Automatically testing with one container runtime in one configuration is sufficient to detect basic problems in kubelet early.
4390+
# CRI-O was picked because it was solid for testing so far.
4391+
# Periodic variant: ci-node-e2e-cgrpv2-crio-dra
4392+
# run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/|/test/e2e_node/dra_test.go
43854393
optional: true
43864394
skip_report: false
43874395
labels:
@@ -4486,7 +4494,10 @@ presubmits:
44864494
skip_branches:
44874495
- release-\d+\.\d+ # per-release image
44884496
always_run: false
4489-
run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/|/test/e2e_node/dra_test.go
4497+
# Automatically testing with one container runtime in one configuration is sufficient to detect basic problems in kubelet early.
4498+
# CRI-O was picked because it was solid for testing so far.
4499+
# Periodic variant: ci-node-e2e-containerd-1-7-dra
4500+
# run_if_changed: /dra/|/dynamicresources/|/resourceclaim/|/resourceclass/|/podscheduling/|/resourceclaimtemplate/|/dynamic-resource-allocation/|/pkg/apis/resource/|/api/resource/|/test/e2e_node/dra_test.go
44904501
optional: true
44914502
skip_report: false
44924503
labels:

0 commit comments

Comments
 (0)