Skip to content

feat: orchestrate CI workflows to prevent duplicate runs when autofix makes changes - #24758

Merged
laipz8200 merged 1 commit into
mainfrom
broken
Aug 29, 2025
Merged

feat: orchestrate CI workflows to prevent duplicate runs when autofix makes changes#24758
laipz8200 merged 1 commit into
mainfrom
broken

Conversation

@laipz8200

@laipz8200 laipz8200 commented Aug 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Summary

  • Adds orchestrated CI workflow to coordinate autofix and test execution
  • Prevents redundant workflow runs when autofix makes changes
  • Maintains parallel test execution for efficiency

Changes

  1. New main CI orchestrator ():
    • Runs autofix as the first job
    • Commits any fixes back to the PR
    • Triggers test workflows only after autofix completes
    • Includes intelligent path filtering to run only relevant tests

Benefits

  • No duplicate runs: Tests wait for autofix to complete, preventing cancellation and re-runs
  • Resource efficiency: Avoids wasted CI compute from cancelled workflows
  • Faster overall CI: Eliminates the delay from cancelled and restarted workflows
  • Cleaner workflow history: No more cluttered history with cancelled runs
  • Smart execution: Tests still run in parallel after autofix, and only run for changed components

Fixes #24757

@laipz8200
feat: orchestrate CI workflows to prevent duplicate runs when autofix…
8daba83 
… makes changes

- Add main-ci.yml as orchestrator workflow that runs autofix first
- Make existing test workflows callable via workflow_call
- Ensure tests only run after autofix completes to avoid cancellations
- Add intelligent path filtering to run only relevant tests
- Maintain parallel execution of test suites for efficiency
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. 🌊 feat:workflow Workflow related stuff. labels Aug 29, 2025
@laipz8200
laipz8200 marked this pull request as draft August 29, 2025 04:50
@laipz8200
laipz8200 marked this pull request as ready for review August 29, 2025 04:59
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 29, 2025
@laipz8200
laipz8200 merged commit e47bfd2 into main Aug 29, 2025
13 checks passed
@laipz8200
laipz8200 deleted the broken branch August 29, 2025 05:23
zyileven pushed a commit to zyileven/dify that referenced this pull request Aug 29, 2025
@laipz8200 @zyileven
feat: orchestrate CI workflows to prevent duplicate runs when autofix…
333cba6 
… makes changes (langgenius#24758)
Nov1c444 pushed a commit that referenced this pull request Aug 29, 2025
@laipz8200 @Nov1c444
feat: orchestrate CI workflows to prevent duplicate runs when autofix…
de212da 
… makes changes (#24758)
Cluas pushed a commit to Cluas/dify that referenced this pull request Sep 1, 2025
@laipz8200 @Cluas
feat: orchestrate CI workflows to prevent duplicate runs when autofix…
1323bbe 
… makes changes (langgenius#24758)
@dosubot dosubot Bot mentioned this pull request Feb 23, 2026
Closed
6 tasks done
Ssazurov pushed a commit to Ssazurov/dify that referenced this pull request Aug 8, 2026
@laipz8200
feat: orchestrate CI workflows to prevent duplicate runs when autofix…
d5f0061 
… makes changes (langgenius#24758)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crazywoola crazywoola crazywoola approved these changes

Assignees

No one assigned

Labels

🌊 feat:workflow Workflow related stuff. lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CI workflows run redundantly when autofix makes changes

2 participants

@laipz8200 @crazywoola