darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-17 04:58:50 -04:00
Code Issues Packages Projects Releases Wiki Activity

Turn statsd back on, but not for api main app, only for delivery

Browse Source 
workers.
This commit is contained in:
Pea Tyczynska
2020-07-08 16:04:31 +01:00
parent 9c4205c7c6
commit fff004da2b
3 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/config.py
View File

@@ -495,9 +495,6 @@ class Live(Config):
CRONITOR_ENABLED = True CRONITOR_ENABLED = True
# TODO: revert when we're confident statsd won't break things
STATSD_ENABLED = False
class CloudFoundryConfig(Config): class CloudFoundryConfig(Config):
pass pass

3
manifest.yml.j2
View File

@@ -3,6 +3,9 @@
'NOTIFY_APP_NAME': 'api', 'NOTIFY_APP_NAME': 'api',
'disk_quota': '2G', 'disk_quota': '2G',
'sqlalchemy_pool_size': 30, 'sqlalchemy_pool_size': 30,
'additional_env_vars': {
'STATSD_HOST': None
},
'routes': { 'routes': {
'preview': ['api.notify.works'], 'preview': ['api.notify.works'],
'staging': ['api.staging-notify.works'], 'staging': ['api.staging-notify.works'],

1
tests/app/dao/notification_dao/test_notification_dao.py
View File

@@ -17,7 +17,6 @@ from app.dao.notifications_dao import (
dao_timeout_notifications, dao_timeout_notifications,
dao_update_notification, dao_update_notification,
dao_update_notifications_by_reference, dao_update_notifications_by_reference,
delete_notifications_older_than_retention_by_type,
get_notification_by_id, get_notification_by_id,
get_notification_for_job, get_notification_for_job,
get_notification_with_personalisation, get_notification_with_personalisation,