Skip to content

Commit

Permalink
Fix elastic apm sentry issues (#529)
Browse files Browse the repository at this point in the history
  • Loading branch information
SamDudley authored Oct 3, 2024
1 parent 5f17995 commit 6810284
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 8 deletions.
3 changes: 0 additions & 3 deletions config/settings/dev.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
import sentry_sdk
from sentry_sdk.integrations.django import DjangoIntegration

from .paas import * # noqa


Expand Down
2 changes: 0 additions & 2 deletions config/settings/paas.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import dj_database_url
from dbt_copilot_python.database import database_url_from_env
from dbt_copilot_python.network import setup_allowed_hosts
from dbt_copilot_python.utility import is_copilot

Expand Down
8 changes: 5 additions & 3 deletions config/settings/prod.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import sentry_sdk
from sentry_sdk.integrations.django import DjangoIntegration
from dbt_copilot_python.utility import is_copilot

from .paas import * # noqa

Expand All @@ -14,10 +13,13 @@
]

INSTALLED_APPS += [
"elasticapm.contrib.django",
"django_audit_log_middleware",
]

# TODO: Remove once we have migrated to the DBT Platform.
if not is_copilot():
INSTALLED_APPS += ["elasticapm.contrib.django"]

# X_ROBOTS_TAG (https://man.uktrade.io/docs/procedures/1st-go-live.html)
X_ROBOTS_TAG = [
"noindex",
Expand Down

0 comments on commit 6810284

Please sign in to comment.