Skip to content

Add public ingress notification drivers#987

Merged
cbusillo merged 1 commit into
mainfrom
feature/public-ingress-notification-drivers
May 29, 2026
Merged

Add public ingress notification drivers#987
cbusillo merged 1 commit into
mainfrom
feature/public-ingress-notification-drivers

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

Summary

  • add DB-backed public-ingress notification policy and delivery-attempt records
  • wire incident transition delivery attempts into the public ingress monitor
  • add GitHub issue, email, and Discord destination drivers with durable per-destination evidence
  • document policy/delivery ownership and the initial driver set

Validation

  • uv run python -m unittest
  • uv run --extra dev mypy control_plane tests
  • uv run --extra dev ruff check control_plane/contracts/public_ingress_monitoring.py control_plane/storage/filesystem.py control_plane/storage/postgres.py control_plane/storage/migrations/versions/f8a0b2c3d4e5_add_public_ingress_notifications.py control_plane/workflows/public_ingress_monitor.py tests/test_public_ingress_monitor.py tests/test_filesystem_store.py
  • uv run --extra dev ruff format --check control_plane/contracts/public_ingress_monitoring.py control_plane/storage/filesystem.py control_plane/storage/postgres.py control_plane/storage/migrations/versions/f8a0b2c3d4e5_add_public_ingress_notifications.py control_plane/workflows/public_ingress_monitor.py tests/test_public_ingress_monitor.py tests/test_filesystem_store.py
  • uv run alembic heads

Refs #982
Refs #983
Refs #929

@cbusillo cbusillo merged commit ffeffa0 into main May 29, 2026
12 checks passed
@cbusillo cbusillo deleted the feature/public-ingress-notification-drivers branch May 29, 2026 20:57
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