Skip to content

chore(ci): filter unrelated builds with native paths - #3217

Merged
imbajin merged 4 commits into
apache:masterfrom
imbaGeek:chore/ci-path-gating
Sep 17, 2026
Merged

imbajin merged 4 commits into
apache:masterfrom
imbaGeek:chore/ci-path-gating

Conversation

@imbajin

@imbajin imbajin commented Sep 17, 2026

Copy link
Copy Markdown
Member

Problem and change

Documentation-only PRs trigger unrelated builds. Use native GitHub Actions pull_request.paths
filters for PD/Store, cluster, Docker and dependency workflows. Include module and shared
build inputs, and exclude named README/AGENTS/CLAUDE documents.

Required Server, Commons, CodeQL and license workflows remain unfiltered to avoid pending
required checks. Push coverage and job dependencies are unchanged.
Correct existing release/test branch glob syntax.

Validation

  • Independent review: no blocking findings.
  • actionlint passed for changed workflows, excluding pre-existing legacy-action warnings
    and unchanged shellcheck findings.
  • git diff --check passed.
  • No runtime code changed. Remote CI is pending.
  • Native path filtering retains GitHub's documented changed-file limits.

Documentation Status

  • Doc - No Need: internal CI trigger changes; required-check constraints are noted in .asf.yaml.

- share conservative PR path selection across build workflows
- retain required check names and full non-PR coverage
- add rename and API fallback regression checks
- document filtering and correct branch globs
- remove custom scope workflow and JavaScript tests
- use native PR path filters for non-required workflows
- retain required checks and original job dependencies
- document the narrower filtering scope
@imbajin imbajin changed the title chore(ci): skip unrelated PR build jobs chore(ci): filter unrelated builds with native paths Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.14%. Comparing base (fabe0f0) to head (d08f2e0).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3217   +/-   ##
=========================================
  Coverage     41.13%   41.14%           
  Complexity     7212     7212           
=========================================
  Files           802      802           
  Lines         69393    69393           
  Branches       9237     9237           
=========================================
+ Hits          28546    28549    +3     
+ Misses        37583    37582    -1     
+ Partials       3264     3262    -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- remove CI implementation details from the build guide
- explain path-filter constraints beside required checks
- keep the filtering rationale in the PR description
@imbajin
imbajin merged commit 2e2c02c into apache:master Sep 17, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants