Skip to content

Commit d2dffda

Browse files
authored
Merge pull request #34984 from pohly/dra-version-skew
DRA canary: invert the kubelet version filter
2 parents e4f4356 + 4bdf5e4 commit d2dffda

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

config/jobs/kubernetes/sig-node/dra-canary.yaml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -262,10 +262,9 @@ presubmits:
262262
# We need support for disabling tests which need a recent kubelet.
263263
# If a test is labeled with `KubeletMinVersion:1.34`, then it cannot run
264264
# when the deployed kubelet is 1.32. This is enforced by
265-
# generating `KubeletMinVersion: isSubsetOf { 1.33, 1.34 }`, i.e.
266-
# not including the actual kubelet version and anything older
267-
# in an allow list.
268-
kubelet_label_filter="&& KubeletMinVersion: isSubsetOf { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
265+
# generating `! KubeletMinVersion: containsAny { 1.33, 1.34 }`, i.e.
266+
# including all unsupportd kubelet versions in a deny list.
267+
kubelet_label_filter=" && ! KubeletMinVersion: containsAny { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
269268
270269
KUBECONFIG=${HOME}/.kube/config ${ginkgo} run --nodes=8 --timeout=24h --silence-skips --force-newlines --no-color --label-filter="DRA && Feature: isSubsetOf { OffByDefault, DynamicResourceAllocation }$kubelet_label_filter && !Alpha && !Flaky && !Slow" ${e2e_test} -- -provider=local -report-dir="${ARTIFACTS}" -report-complete-ginkgo -report-complete-junit &
271270
GINKGO_E2E_PID=$!
@@ -370,10 +369,9 @@ presubmits:
370369
# We need support for disabling tests which need a recent kubelet.
371370
# If a test is labeled with `KubeletMinVersion:1.34`, then it cannot run
372371
# when the deployed kubelet is 1.32. This is enforced by
373-
# generating `KubeletMinVersion: isSubsetOf { 1.33, 1.34 }`, i.e.
374-
# not including the actual kubelet version and anything older
375-
# in an allow list.
376-
kubelet_label_filter="&& KubeletMinVersion: isSubsetOf { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
372+
# generating `! KubeletMinVersion: containsAny { 1.33, 1.34 }`, i.e.
373+
# including all unsupportd kubelet versions in a deny list.
374+
kubelet_label_filter=" && ! KubeletMinVersion: containsAny { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
377375
378376
KUBECONFIG=${HOME}/.kube/config ${ginkgo} run --nodes=8 --timeout=24h --silence-skips --force-newlines --no-color --label-filter="DRA && Feature: isSubsetOf { OffByDefault, DynamicResourceAllocation }$kubelet_label_filter && !Alpha && !Flaky && !Slow" ${e2e_test} -- -provider=local -report-dir="${ARTIFACTS}" -report-complete-ginkgo -report-complete-junit &
379377
GINKGO_E2E_PID=$!

config/jobs/kubernetes/sig-node/dra.jinja

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -210,10 +210,9 @@ presubmits:
210210
# We need support for disabling tests which need a recent kubelet.
211211
# If a test is labeled with `KubeletMinVersion:1.34`, then it cannot run
212212
# when the deployed kubelet is 1.32. This is enforced by
213-
# generating `KubeletMinVersion: isSubsetOf { 1.33, 1.34 }`, i.e.
214-
# not including the actual kubelet version and anything older
215-
# in an allow list.
216-
kubelet_label_filter="&& KubeletMinVersion: isSubsetOf { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
213+
# generating `! KubeletMinVersion: containsAny { 1.33, 1.34 }`, i.e.
214+
# including all unsupportd kubelet versions in a deny list.
215+
kubelet_label_filter=" && ! KubeletMinVersion: containsAny { $( for v in $(seq $((previous_minor + 1)) $((minor - 1))); do echo "1.$v, "; done)1.$minor }"
217216
{%- endif %}
218217

219218
KUBECONFIG=${HOME}/.kube/config ${ginkgo} run --nodes=8 --timeout=24h --silence-skips --force-newlines --no-color --label-filter="DRA && Feature: isSubsetOf { OffByDefault, DynamicResourceAllocation } {%- if kubelet_skew|int > 0 %}$kubelet_label_filter{%- endif %} {%- if not all_features %} && !Alpha {%- endif %} && !Flaky {%- if not ci %} && !Slow {%- endif %}" ${e2e_test} -- -provider=local -report-dir="${ARTIFACTS}" -report-complete-ginkgo -report-complete-junit &

0 commit comments

Comments
 (0)