Skip to content

docs(paperclip-ops): reflect direct Sentry->Paperclip integration#213

Merged
ohld merged 1 commit intoproductionfrom
docs/sentry-paperclip-direct-integration
Apr 29, 2026
Merged

docs(paperclip-ops): reflect direct Sentry->Paperclip integration#213
ohld merged 1 commit intoproductionfrom
docs/sentry-paperclip-direct-integration

Conversation

@ohld
Copy link
Copy Markdown
Member

@ohld ohld commented Apr 29, 2026

Summary

Follow-up to PR #212 (which deleted the webhook proxy). Updates docs/paperclip-ops-runbook.md to match the now-live direct Sentry→Paperclip path:

  • Webhook table: Sentry row now shows signingMode=none; Coolify and Prefect rows removed (no longer wired).
  • How to test: Adds a runnable e2e probe (unique exception class so Sentry creates a fresh issue and the webhook actually fires — recurring instances of an existing issue do not trigger).
  • How to revert: One curl to flip the trigger back to bearer if direct delivery breaks.
  • Fix stale cron: QA Log Scan was listed as 7 * * * * (hourly); the live schedule is 7 */3 * * * (every 3h).

Pure docs — no code change.

Test plan

  • grep shows no remaining stale references to qa-alert / notify_qa_sync / removed env vars in repo docs

Update the runbook's webhook section now that PR #212 deleted the proxy:
- Webhook table: Sentry path is direct (signingMode=none); Coolify and
  Prefect rows removed (no longer wired)
- Add 'How to test' end-to-end recipe with unique exception class
- Add revert recipe (PATCH trigger back to bearer)
- Fix QA Log Scan cron from '7 * * * *' to '7 */3 * * *' to match the live
  schedule (every 3h, not hourly)
@ohld
Copy link
Copy Markdown
Member Author

ohld commented Apr 29, 2026

STAFF ENGINEER REVIEW: APPROVED — pure docs PR (43 lines in docs/paperclip-ops-runbook.md). Accurately reflects post-#212 state: Sentry direct integration with signingMode=none, QA cron 7 */3 * * *, Coolify/Prefect rows correctly removed. PR #212 verified merged 09:46 UTC. No code, no SQL, no secrets. Skipped /codex (no code surface) and /cso (no sensitive surfaces). Queueing auto-merge.

@ohld ohld merged commit c25521e into production Apr 29, 2026
3 checks passed
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