Skip to content

Commit 02a2647

Browse files
committed
jobs: avoid blocking job which doesn't always run
As bentheelder said on Slack (https://kubernetes.slack.com/archives/C2C40FMNF/p1734418617113169?thread_ts=1734417601.687079&cid=C2C40FMNF), "adding blocking tests that only sometimes run is confusing and doesn't work well". That's because a regression might get merged without running the job. Then any PR where the job gets run is blocked, whether it is related to the failure or not. Therefore pull-kubernetes-e2e-gce-network-proxy-http-connect should not be blocking.
1 parent d4a63b0 commit 02a2647

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

Diff for: config/jobs/kubernetes/sig-api-machinery/sig-api-machinery-config.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ presubmits:
8080
cluster: k8s-infra-prow-build
8181
always_run: false
8282
run_if_changed: '^(cluster/gce/manifests/konnectivity-server.yaml$|cluster/gce/addons/konnectivity-agent)'
83+
optional: true # Doesn't always run and regressions shouldn't block merging PRs where it gets run.
8384
labels:
8485
preset-service-account: "true"
8586
preset-k8s-ssh: "true"

0 commit comments

Comments
 (0)