Page MenuHomeSoftware Heritage

Add sentry integration to swh-deposit
ClosedPublic

Authored by vlorentz on Dec 11 2019, 3:47 PM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers

Diff Detail

Repository
rDDEP Push deposit
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9699
Build 14297: tox-on-jenkinsJenkins
Build 14296: arc lint + arc unit

Event Timeline

ardumont added inline comments.
swh/deposit/tests/test_gunicorn_config.py
8

the more the merrier ;)

(You've got 2 copyright headers)

remove extra copyright header

I've no idea what this allows to do.

Can you please enlighten me?

swh/deposit/tests/test_gunicorn_config.py
13

jsyk, pytest-mock (extra dep though, fixture is named mocker) has started to be used as well [1].
Maybe this could be set within a fixture using pytest-mock.

[1] https://forge.softwareheritage.org/source/swh-loader-core/browse/master/swh/loader/package/npm/tests/test_tasks.py$7-8

24

jsyk, there is the pytest's monkeypatch fixture we started using consistently now [1]

[1] https://forge.softwareheritage.org/source/swh-loader-core/browse/master/conftest.py$44

I've no idea what this allows to do.

Can you please enlighten me?

see https://forge.softwareheritage.org/D2418#change-CNJs5ITmJ36y

I stil don't see much what that is, even following the other diff.
i had a glimpse of the sentry ui and i quite like it

so ok, i guess.

This revision is now accepted and ready to land.Dec 17 2019, 7:25 PM

It defines a function that gets called by gunicorn workers when they start. This function calls sentry_sdk.init() so it hooks itself in exception handlers and other places

Landed as 0c1a59c9192d9e06f7d38f96b06dfb8e422d3004.