Uh oh!
There was an error while loading. Please reload this page.
perf(supervisor): inspect Docker resources concurrently - #4471
perf(supervisor): inspect Docker resources concurrently#4471RealBhupesh wants to merge 1 commit into
Conversation
|
Hi @RealBhupesh, thanks for your interest in contributing! This project requires that pull request authors are vouched, and you are not in the list of vouched users. This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details. |
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
WalkthroughSupervisor resource monitoring now filters for running containers before inspection. It inspects the containers concurrently with ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Root cause
The supervisor awaited each Docker container inspection inside a sequential loop, delaying resource availability updates as the number of running containers increased.
Validation
Please keep this PR in draft until vouching and CI review are complete.
fixes#3727