Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Sentry to the ingestion server #1105

Closed
krysal opened this issue Mar 31, 2023 · 0 comments · Fixed by #1106
Closed

Add Sentry to the ingestion server #1105

krysal opened this issue Mar 31, 2023 · 0 comments · Fixed by #1106
Assignees
@krysal
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: ingestion server Related to the ingestion/data refresh server 🐛 tooling: sentry Sentry issue

Comments

@krysal
Copy link
Member

krysal commented Mar 31, 2023 •
edited
Loading

Problem

In light of @obulat's recent discovery #1059, we might be missing more errors from the ingestion server that fails silently and nobody notice unless you're looking at the logs.

Description

Sentry should catch these issues and notify us by the usual channels.
@krysal krysal added 🟨 priority: medium Not blocking but should be addressed soon 🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🐛 tooling: sentry Sentry issue 🧱 stack: ingestion server Related to the ingestion/data refresh server labels Mar 31, 2023
@krysal krysal transferred this issue from WordPress/openverse-catalog Mar 31, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Openverse Backlog Mar 31, 2023
@krysal krysal self-assigned this Mar 31, 2023
@krysal krysal mentioned this issue Mar 31, 2023
Merged
5 tasks
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Openverse Backlog Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krysal krysal

Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: ingestion server Related to the ingestion/data refresh server 🐛 tooling: sentry Sentry issue
Projects
Openverse Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Sentry to the ingestion server WordPress/openverse
1 participant
@krysal