bcgov / entity

ServiceBC Registry Team working on Legal Entities
Apache License 2.0
23 stars 58 forks source link

BE - revert sentry config back to previous settings for dev/test #21699

Closed argush3 closed 1 month ago

argush3 commented 3 months ago

A recent update/PR was made to sentry config in the various BE services. This was an attempt to fix an issue where containers wouldn’t start up properly for some BE services due to sentry config having an incorrect null value.

The problem with the fix is that with sentry working properly in dev & test, it consumes all of the budget due to all the errors in dev/test.

We will need to revert the sentry changes made in the referenced PR and ensure that sentry_dsn populates with an empty string(null value seems to cause issues with containers not being able to start up properly).

TODOs

image.png image 2.png
argush3 commented 3 months ago

cc: @davemck513 @pwei1018

argush3 commented 1 month ago

Verified sentry no longer picking up errors for LEAR BE services after disabling.

image.png