starrocks icon indicating copy to clipboard operation
starrocks copied to clipboard

[Tool] CI split commit filter

Open andyziye opened this issue 3 weeks ago • 9 comments

Why I'm doing:

What I'm doing:

Fixes #issue

What type of PR is this:

  • [ ] BugFix
  • [ ] Feature
  • [ ] Enhancement
  • [ ] Refactor
  • [ ] UT
  • [ ] Doc
  • [x] Tool

Does this PR entail a change in behavior?

  • [ ] Yes, this PR will result in a change in behavior.
  • [x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

  • [ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
  • [ ] Parameter changes: default values, similar parameters but with different default values
  • [ ] Policy changes: use new policy to replace old one, functionality automatically enabled
  • [ ] Feature removed
  • [ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

  • [ ] I have added test cases for my bug fix or my new feature
  • [ ] This pr needs user documentation (for new or modified features or behaviors)
    • [ ] I have added documentation for my new feature or new function
  • [ ] This is a backport pr

Bugfix cherry-pick branch check:

  • [x] I have checked the version labels which the pr will be auto-backported to the target branch
    • [ ] 4.0
    • [ ] 3.5
    • [ ] 3.4
    • [ ] 3.3

[!NOTE] Refactors CI to use pre-commit-driven path filters and conditional job gating, updates filters/dependencies/artifacts, and makes minor comment tweaks in BE/FE files.

  • CI pipeline:
    • Pre-commit integration: basic-checker now outputs pre_commit*, ONLY_*_TEST, and *CHANGE flags; be-checker/fe-checker parse pre-commit job conclusions to gate downstream jobs.
    • Conditional execution: Added guards across clang-format, thirdparty-update, be-ut, clang-tidy, fe-codestyle-check, sonarcloud-fe-checker, fe-ut, and build to skip when unchanged or pre-commit succeeded; adjusted build necessity and dependencies; reuse latest OSS package when no code changes.
    • Path filters: Moved run-be-ut.sh under BE ut; FE includes **/pom.xml in fe and pom filters.
    • Artifacts/actions: Increased artifact retention to 7 (or 3 for logs); added Test Case Info artifact; switched actions/download-artifact to v6 where used.
    • Workflow plumbing: Propagate/consume new outputs (e.g., MYSQL_ECI_IP via restart-fe to admit); added git checkout commit-pipeline in CI-tool steps; added extra cleanup/eci handling.
    • Jobs adjusted: build dependencies simplified; SQL-Tester clean step exits early; Teardown now also needs fe-ut and be-ut.
  • Code:
    • Minor comment additions in be/test/agent/agent_task_test.cpp and AlterCancelException.java.

Written by Cursor Bugbot for commit cbaeef92148c0b8a92f3fd46cd1bcdf6145faabe. This will update automatically on new commits. Configure here.

andyziye avatar Dec 01 '25 11:12 andyziye

🧪 CI Insights

Here's what we observed from your CI run for 8a8b3ef7.

🟢 All jobs passed!

But CI Insights is watching 👀

mergify[bot] avatar Dec 08 '25 08:12 mergify[bot]

@cursor review

alvin-celerdata avatar Dec 08 '25 17:12 alvin-celerdata

@cursor review

alvin-celerdata avatar Dec 09 '25 17:12 alvin-celerdata

@cursor review

alvin-celerdata avatar Dec 10 '25 17:12 alvin-celerdata

@cursor review

alvin-celerdata avatar Dec 11 '25 15:12 alvin-celerdata

@cursor review

alvin-celerdata avatar Dec 12 '25 17:12 alvin-celerdata

[Java-Extensions Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] avatar Dec 13 '25 13:12 github-actions[bot]

[FE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] avatar Dec 13 '25 13:12 github-actions[bot]

[BE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] avatar Dec 13 '25 13:12 github-actions[bot]