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

Make sentry integration optional #14

Closed
jmaupetit opened this issue Oct 21, 2020 · 0 comments · Fixed by #34
Closed

Make sentry integration optional #14

jmaupetit opened this issue Oct 21, 2020 · 0 comments · Fixed by #34
Labels
feature
Projects
Ralph 1.0.0
Milestone
Ralph 1.0.0

Comments

@jmaupetit
Copy link
Contributor

Purpose

As Ralph will manipulate sensible data, we should carefully watch jobs execution. Sentry will be a great help to detect failures.

Proposal

We should think of a plugin architecture for Ralph for this kind of integration.
@jmaupetit jmaupetit mentioned this issue Oct 21, 2020
Closed
@jmaupetit jmaupetit added this to To do in Ralph 1.0.0 Oct 21, 2020
@jmaupetit jmaupetit added this to the Ralph 1.0.0 milestone Oct 21, 2020
@jmaupetit jmaupetit linked a pull request Dec 8, 2020 that will close this issue
📈(project) Add optional sentry integration #34
Merged
@jmaupetit jmaupetit moved this from To do to Done in Ralph 1.0.0 Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
No open projects
Ralph 1.0.0
Done
Milestone
Ralph 1.0.0
Development

Successfully merging a pull request may close this issue.

📈(project) Add optional sentry integration
1 participant
@jmaupetit