Skip to content

Run health alert checks concurrently#388

Open
silentgeckoaudit3801 wants to merge 7 commits into
Quantarq:mainfrom
silentgeckoaudit3801:perf/concurrent-health-alerts-206
Open

Run health alert checks concurrently#388
silentgeckoaudit3801 wants to merge 7 commits into
Quantarq:mainfrom
silentgeckoaudit3801:perf/concurrent-health-alerts-206

Conversation

@silentgeckoaudit3801

Copy link
Copy Markdown

Closes #206.

Summary

  • check notification users concurrently with a configurable semaphore cap
  • apply a per-user timeout so one slow RPC does not block the alert sweep
  • record Prometheus success/failure counters for mixed alert sweep outcomes
  • add async tests for failure isolation, concurrency, and timeout continuation

Validation

  • Parsed the changed Python files with ast.parse using the bundled Codex Python runtime.
  • Checked changed lines stay within the project lint line-length convention.
  • GitHub compare reports this branch is 0 commits behind main.

Testing

  • Not run as a full local project suite because this workspace is avoiding dependency installation and project toolchain execution.

Comment thread quantara/web_app/tests/test_alert_mixin.py Fixed
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.

Eliminate N+1 in get_users_for_notifications

2 participants