Add sentry integration #3600
Add sentry integration #3600
Conversation
willemarcel
added
scope: infrastructure
Difficulty: 3
status: in progress
type: enhancement
| import sentry_sdk | ||
| from sentry_sdk.integrations.flask import FlaskIntegration | ||
|
|
||
| sentry_sdk.init( |
There was a problem hiding this comment.
@dakotabenjamin is NetworkEnvironment available as a ENV VAR to the backend?
To set the environment on sentry, we just need to add environment=foo here
There was a problem hiding this comment.
can we use the stack naming convention for that, similar to how we do New Relic? https://github.com/hotosm/tasking-manager/blob/feature/sentry-backend-integration/scripts/aws/cloudformation/tasking-manager.template.js#L394
There was a problem hiding this comment.
Yes, we can. Could we change the env var from NEW_RELIC_ENVIRONMENT to TM_ENVIRONMENT? So if we stop using NewRelic, we don't need to change the env var and make the variable name more expressive.
There was a problem hiding this comment.
NEW_RELIC_ENVIRONMENT is something used internally by New Relic, so we can't change it.
|
@dakotabenjamin I also added sentry to the frontend and adjusted the backend sample rate |
willemarcel
force-pushed
the
feature/sentry-backend-integration
branch
from
f3cdad0
to
538c1cd
Compare
|
@willemarcel I added the environment variable assignment, and made the generic "TM_ENVIRONMENT" envvar. For production it will show up as "tm4-production". |
willemarcel
force-pushed
the
feature/sentry-backend-integration
branch
from
0aefb31
to
c4fe889
Compare
* add sentry integration * add sentry integration to frontend + adjust backend sample_rate * Add environment envvar for sentry * Add sentry dsn envvars for deployment Co-authored-by: Dakota Benjamin <[email protected]>
* add sentry integration * add sentry integration to frontend + adjust backend sample_rate * Add environment envvar for sentry * Add sentry dsn envvars for deployment Co-authored-by: Dakota Benjamin <[email protected]>
No description provided.