Skip to content

Let area:kubernetes-tests label force the Kubernetes tests job#69519

Merged
jason810496 merged 2 commits into
apache:mainfrom
jason810496:ci/selective-checks/kubernetes-tests-label
Jul 8, 2026
Merged

Let area:kubernetes-tests label force the Kubernetes tests job#69519
jason810496 merged 2 commits into
apache:mainfrom
jason810496:ci/selective-checks/kubernetes-tests-label

Conversation

@jason810496

@jason810496 jason810496 commented Jul 7, 2026

Copy link
Copy Markdown
Member

Why

Today the only label that can force the Kubernetes tests job is full tests needed, which also pulls in the entire test matrix. Respecting the existing area:kubernetes-tests label lets maintainers re-run just the Kubernetes tests when selective-checks' file-path heuristics don't catch a change's impact.

What

  • run_kubernetes_tests returns true when the area:kubernetes-tests label is set on the PR, independent of full_tests_needed (the tests-kubernetes job and prod image build already cascade from this output — no workflow changes needed).
  • Document the label in the labels table of 04_selective_checks.md.
  • Add a test asserting the label forces run_kubernetes_tests without tripping full_tests_needed.

Was generative AI tooling used to co-author this PR?

Comment thread dev/breeze/tests/test_selective_checks.py

@Lee-W Lee-W left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One nit. It probably works fine without modification. approve.

@jason810496
jason810496 requested a review from Lee-W July 8, 2026 02:12
Contributors validating a change whose impact selective-checks'
file-path heuristics miss previously had to reach for `full tests
needed`, which pulls in the entire test matrix. Reusing the existing
`area:kubernetes-tests` triage label lets them force just the
Kubernetes tests job instead.
Covers a label-only trigger (e.g. added after the fact to an existing
PR) to confirm the forced job doesn't depend on the file-based
heuristics also matching.
@jason810496
jason810496 force-pushed the ci/selective-checks/kubernetes-tests-label branch from 37e8836 to 7b1caa7 Compare July 8, 2026 06:02
@jason810496
jason810496 merged commit 70913ba into apache:main Jul 8, 2026
145 of 146 checks passed
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-3-test PR Link

Lee-W pushed a commit that referenced this pull request Jul 8, 2026
…s job (#69519) (#69593)

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
vatsrahul1001 pushed a commit that referenced this pull request Jul 9, 2026
…s job (#69519) (#69593)

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants