Page MenuHomeSoftware Heritage

Make the CLI initialize sentry-sdk based on CLI options/envvars.
ClosedPublic

Authored by vlorentz on Dec 9 2019, 4:22 PM.

Diff Detail

Repository
rDCORE Foundations and core functionalities
Branch
sentry-cli
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9639
Build 14197: tox-on-jenkinsJenkins
Build 14196: arc lint + arc unit

Event Timeline

olasd requested changes to this revision.Dec 10 2019, 11:58 AM
olasd added a subscriber: olasd.

Could you set the sentry debug flags as hidden from the help text? That's _very_ verbose for a debugging option which most users will never turn on (or care about).

Can you make sure that the sentry logging integration works well when initialized /before/ we configure the logging framework itself?

This revision now requires changes to proceed.Dec 10 2019, 11:58 AM
In D2411#57190, @olasd wrote:

Could you set the sentry debug flags as hidden from the help text? That's _very_ verbose for a debugging option which most users will never turn on (or care about).

Done.

Can you make sure that the sentry logging integration works well when initialized /before/ we configure the logging framework itself?

I just checked and it does.

This revision is now accepted and ready to land.Dec 10 2019, 1:49 PM