fix(telegram): bound stale-client cleanup and add Windows CLOSE-WAIT live probes (#87057)
Follow-ups on top of the salvaged commits from PR #87111 (@HexLab98) and PR #87265 (@JoaoMarcos44): - keep main's #92991 stall watchdog (150s progress-based) as the single steady-state liveness probe instead of adding a second overlapping one - orphaned-client aclose() cleanup uses the wall-clock thread deadline and is tracked in _background_tasks so a wedged close can neither hang nor leak one task per reconnect attempt (from #87265's review findings) - merge #87265's no-keepalive getUpdates pool (max_keepalive_connections=0) with #87111's TCP-keepalive socket options on all transports - add tests/gateway/test_telegram_closewait_windows_live.py: live probes against a real half-closing HTTP server, skipif non-win32, wired into the on-demand windows-venv-e2e lane (wine2e/**)
This commit is contained in:
@@ -50,7 +50,7 @@ jobs:
|
||||
- name: Install dependencies
|
||||
uses: ./.github/actions/retry
|
||||
with:
|
||||
command: uv sync --locked --python 3.11 --extra dev
|
||||
command: uv sync --locked --python 3.11 --extra dev --extra messaging
|
||||
|
||||
- name: Run venv-holder live E2E
|
||||
shell: bash
|
||||
@@ -61,3 +61,11 @@ jobs:
|
||||
tests/hermes_cli/test_taskkill_identity_windows_live.py \
|
||||
tests/hermes_cli/test_git_trampoline_windows_live.py \
|
||||
-o addopts= -v -p no:cacheprovider
|
||||
|
||||
- name: Run Telegram CLOSE-WAIT reconnect live E2E (#87057)
|
||||
shell: bash
|
||||
run: |
|
||||
set -uo pipefail
|
||||
uv run --no-sync python -m pytest \
|
||||
tests/gateway/test_telegram_closewait_windows_live.py \
|
||||
-o addopts= -v -p no:cacheprovider
|
||||
|
||||
Reference in New Issue
Block a user