Skip to content

CI: Select Flink PR matrix by changed version #16947

Open
ajantha-bhat wants to merge 2 commits into
apache:mainfrom
ajantha-bhat:codex/ci-flink-pr-version-matrix
Open

CI: Select Flink PR matrix by changed version #16947
ajantha-bhat wants to merge 2 commits into
apache:mainfrom
ajantha-bhat:codex/ci-flink-pr-version-matrix

Conversation

@ajantha-bhat

Copy link
Copy Markdown
Member

depends on #16945

Summary

This updates Flink CI to build an incremental PR matrix based on the changed Flink version path.

For ordinary PRs, Flink CI now uses the JVM plan from the earlier Java CI PR and then narrows the Flink matrix when changes are scoped to a known Flink version path:

  • flink/v1.20/** runs Flink 1.20 only
  • flink/v2.0/** runs Flink 2.0 only
  • flink/v2.1/** runs Flink 2.1 only
  • unknown Flink version paths, unknown Flink paths, missing changed-file detection, full-ci, and push/main/release/tag runs fall back to the full Flink matrix

Known Flink versions are read from knownFlinkVersions in gradle.properties, so adding/removing Flink versions does not require hard-coding them in the planner.

This keeps the PR path conservative: stale or unknown planner inputs run more CI, not less.

Testing

  • actionlint .github/workflows/flink-ci.yml
  • bash -n .github/scripts/ci-plan-common.sh .github/scripts/plan-flink-ci.sh
  • YAML parse for jvm-ci-plan.yml and flink-ci.yml
  • git diff --check
  • Local planner checks:
    • flink/v2.0/** -> Flink 2.0 only
    • flink/v1.20/** -> Flink 1.20 only
    • unknown Flink version/path -> all Flink versions
    • full-ci / push -> all Flink versions with the full JVM matrix

@github-actions github-actions Bot added the INFRA label Jun 24, 2026
@ajantha-bhat ajantha-bhat force-pushed the codex/ci-flink-pr-version-matrix branch from a3e713a to 4ffe7cc Compare June 24, 2026 08:50
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.

1 participant