Skip to content

Fix dashboard SSE shutdown during autoupdate - #172

Merged
ecarreras merged 1 commit into
mainfrom
issue-171-sse-shutdown
Aug 3, 2026
Merged

Fix dashboard SSE shutdown during autoupdate#172
ecarreras merged 1 commit into
mainfrom
issue-171-sse-shutdown

Conversation

@pilipilisbot

Copy link
Copy Markdown
Owner

Summary

  • Signal dashboard SSE generators through the FastAPI lifespan shutdown event.
  • Let job session streams exit while sleeping and close journal tail streams when shutdown begins.
  • Document the shutdown behavior for dashboard autoupdates.

Fixes #171

Tests

  • . .venv/bin/activate && pytest -q

Result: 341 passed, 1 existing Starlette/httpx deprecation warning.

Co-authored-by: giscebot <286264155+giscebot@users.noreply.github.com>
@ecarreras
ecarreras merged commit ba27a1a into main Aug 3, 2026
3 checks passed
@ecarreras
ecarreras deleted the issue-171-sse-shutdown branch August 3, 2026 11:03
@pilipilisbot

Copy link
Copy Markdown
Owner Author

Post-merge verification: PR #172 is merged into main as ba27a1ae54d1578fea53194a3e2f053f9152ebd6 (head 3b6dcc048fe1d02efd489183038484b63e9597ac). The linked issue #171 is closed, and the PR checks completed successfully: dashboard, pytest (3.11), and pytest (3.12). No follow-up code changes are needed for this merge event.

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.

Dashboard update causes ~90 seconds of HTTP 502 while SSE streams block shutdown

2 participants