Skip to content

fix: restore four Project sync workers - #10946

Merged
zouyonghe merged 1 commit into
AstrBotDevs:masterfrom
zouyonghe:fix/project-sync-restore-4-workers
Nov 2, 2026
Merged

zouyonghe merged 1 commit into
AstrBotDevs:masterfrom
zouyonghe:fix/project-sync-restore-4-workers

Conversation

@zouyonghe

@zouyonghe zouyonghe commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Reduce Project apply concurrency from eight workers back to four.
  • Update the concurrency regression test and operator documentation.
  • Keep the 120-minute job timeout unchanged.

Verification

  • All 52 focused offline tests pass.
  • Repository Ruff checks, Actionlint, and diff checks pass.
  • No live Project writes or workflow reruns performed.

This only reduces concurrency; it does not claim to diagnose or fix the API errors from run 41367884956.

Summary by Sourcery

Restore Project synchronization to four concurrent apply workers and align its documentation and regression coverage.

Enhancements:

  • Restore Project synchronization apply concurrency to four workers while preserving sequential mutations within each item.

Documentation:

  • Update Project synchronization documentation to reflect the four-worker apply limit.

Tests:

  • Update the concurrency regression test to verify a maximum of four active workers.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Needs a human reviewer. With fewer concurrent workers, a sync could take longer and hit its execution budget after applying only some updates. Those partial Project changes remain after reverting, but the sync can be rerun to reconcile them.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@zouyonghe
zouyonghe merged commit 4f60422 into AstrBotDevs:master Nov 2, 2026
26 checks passed
@Soulter Soulter moved this to Done in AstrBot Roadmap Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant