Skip to content

Remove asyncio usage in jobs monitoring#796

Draft
amaslenn wants to merge 3 commits intomainfrom
am/no-async
Draft

Remove asyncio usage in jobs monitoring#796
amaslenn wants to merge 3 commits intomainfrom
am/no-async

Conversation

@amaslenn
Copy link
Contributor

@amaslenn amaslenn commented Feb 6, 2026

Summary

Most of the calls inside the async wrappers are blocking: sync rest calls and system calls. So asyncio only adds a level of complexity without bringing any benefits.

Test Plan

  1. CI (updated)
  2. Manual runs of scenarios under conf/common + some private ones.

Additional Notes

Most of the calls inside the async wrappers are blocking: sync rest
calls and system calls. So asyncio only adds a level of complexity
without bringing any benefit.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch am/no-async

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amaslenn amaslenn requested a review from podkidyshev February 6, 2026 10:52
podkidyshev
podkidyshev previously approved these changes Feb 6, 2026
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