Skip to content

Implement gevent monkey-patching - #1806

Draft
valyo wants to merge 3 commits into
devfrom
HMS-2644-gevent-patch
Draft

Implement gevent monkey-patching#1806
valyo wants to merge 3 commits into
devfrom
HMS-2644-gevent-patch

Conversation

@valyo

@valyo valyo commented Mar 13, 2026

Copy link
Copy Markdown
Member

Pull Request Template

Before Marking as Ready for Review

  • Add relevant information to the sections below (Summary etc)
  • Rebase or merge the latest dev (or other targeted branch)
  • Update documentation if needed
  • Add an entry to the SPRINTLOG.md if needed
  • Choose an appropriate label. See here for information on the labelling options
  • The code follows the style guidelines
  • Perform a self-review: read the diff as if reviewing someone else's code
  • I have commented the code, particularly in hard-to-understand areas
  • Verify that all checks and tests have passed

If the target branch is master:

Summary

Implementing gevent monkey-patching for the DDS backend. This is necessary for testing how the gevent worker class handles requests compared to the gthread class we currently use

Related Issue/Ticket

HMS-2644

Testing

If applicable: How did you verify the change? Include commands, data, or screenshots.

Reviewer Notes

Anything that helps reviewers (e.g. areas needing close attention).


Once all boxes are checked, mark the PR as Ready for Review and tag at least one team member as the initial reviewer.

@codecov

codecov Bot commented Mar 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.92%. Comparing base (2b87934) to head (a9905f7).
⚠️ Report is 6 commits behind head on dev.

Files with missing lines Patch % Lines
dds_web/run_app.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1806      +/-   ##
==========================================
- Coverage   92.95%   92.92%   -0.04%     
==========================================
  Files          30       30              
  Lines        5028     5030       +2     
==========================================
  Hits         4674     4674              
- Misses        354      356       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant