Add env var SWH_MAIN_PACKAGE to initialize sentry_sdk with a release.

Also deduplicates code used to call sentry_sdk.init by making all our init code go through the same function swh.core.sentry.init_sentry."

---

Migrated from D2514 (view on Phabricator)

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/383/ See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/383/console

I'm a bit puzzled by the 2 revisions in this diff. Are they related? I don't really see how/why.

Also please consider better commit message with the 'why' (why do we need "Allow subclasses of RPCClient to override generated methods", and even if i might appear king of obvious, why do we need this SWH_MAIN_PACKAGE).

BTW there seem to be a typo in this commit message (there is a closing " without an opening one).

Merge request was returned for changes

Oh maybe it's a mixed up 'arc diff --update' (with swh-scheduler!127 (closed))

remove unwanted commit

code from !116 (closed) got mixed up with this diff. fixed

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/384/ See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/384/console

fix test

Build is green See https://jenkins.softwareheritage.org/job/DCORE/job/tox/385/ for more details.

mentioned in merge request swh-scheduler!127 (closed)

I'm fine with the code, but as I already said, I'd really like the commit message to have a paragraph on why this is needed and what problem it solves.

It might be obvious for the author of the diff but not for someone with no particular knowledge in sentry reading the git log...

Merge request was accepted

approved this merge request

update commit message

Merge request was merged

closed

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/386/ See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/386/console