Skip to content

Commit b789019

Browse files
committed
Merge branch 'main' into HEA-677/Key_error_'Fishing'_in_consolidated_fixture-asset
2 parents 53a4d5b + 24886d8 commit b789019

File tree

9 files changed

+10
-27
lines changed

9 files changed

+10
-27
lines changed

apps/baseline/admin.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -767,7 +767,8 @@ class WealthGroupAdmin(admin.ModelAdmin):
767767
)
768768
search_fields = (
769769
"community__name",
770-
"wealth_group_category",
770+
"wealth_group_category__code__iexact",
771+
*translation_fields("wealth_group_category__name"),
771772
)
772773
list_filter = (
773774
"livelihood_zone_baseline__source_organization",

docker-compose.yml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,6 @@ services:
108108
AWS_ENDPOINT_URL_S3: ${MINIO_ENDPOINT_URL}
109109
SUPPORT_EMAIL_ADDRESS: ${SUPPORT_EMAIL_ADDRESS}
110110
DJANGO_MIGRATE: 1
111-
DD_TRACE_ENABLED: "false"
112-
DD_TRACE_AGENT_URL: http://localhost:8126
113-
DD_DOGSTATSD_URL: udp://localhost:8125
114-
DD_SERVICE: app
115-
DD_ENV: ${ENV}
116-
DD_TAGS: app:${APP},env:{ENV}
117-
DD_DJANGO_CACHE_SERVICE_NAME: django.core.cache
118111
KILUIGI_INTERMEDIATETARGET_BACKEND_CLASS: ${KILUIGI_INTERMEDIATETARGET_BACKEND_CLASS}
119112
KILUIGI_INTERMEDIATETARGET_ROOT_PATH: ${KILUIGI_INTERMEDIATETARGET_ROOT_PATH}
120113
KILUIGI_FINALTARGET_BACKEND_CLASS: ${KILUIGI_FINALTARGET_BACKEND_CLASS}
@@ -134,7 +127,6 @@ services:
134127
- db
135128
environment:
136129
<<: *base_env
137-
DD_SERVICE: dagster_webserver
138130
entrypoint: docker/app/run_dagster_webserver.sh
139131
x-aws-role:
140132
Statement:
@@ -160,7 +152,6 @@ services:
160152
- db
161153
environment:
162154
<<: *base_env
163-
DD_SERVICE: dagster_daemon
164155
entrypoint: docker/app/run_dagster_daemon.sh
165156
x-aws-role:
166157
Statement:

docker/app/gunicorn_config.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
env = environ.Env(
88
# set casting, default value
9-
DD_TRACE_ENABLED=(bool, False),
109
GUNICORN_PORT=(int, 8000),
1110
LOG_FORMATTER=(str, "standard"),
1211
)
@@ -24,9 +23,6 @@
2423
limit_request_line = 0
2524
# Set the access_log_format and handlers according to whether we are logging to DataDog
2625
access_handlers = ["logfile", "access_log"]
27-
if env("DD_TRACE_ENABLED"):
28-
access_handlers += ["console"]
29-
statsd_host = env("DD_DOGSTATSD_URL").split("//")[1] # Gunicorn doesn't recognize the udp:// scheme
3026
# Set the access_log_format according to whether we are using a log processor like FluentD or DataDog
3127
if env("LOG_FORMATTER") == "json":
3228
access_log_format = '{ "time_local":"%(t)s", "remote_addr":"%(h)s", "request":"%(U)s", "query":"%(q)s", "request_method":"%(m)s", "status":%(s)s, "body_bytes_sent":%(B)d, "request_time":%(D)d, "http_user_agent":"%(a)s", "http_referrer":"%(f)s", "x_forwarded_for": "%({X-Forwarded-For}i)s"}' # NOQA
@@ -87,7 +83,6 @@
8783
"level": "DEBUG",
8884
"propagate": False,
8985
},
90-
"ddtrace": {"level": "INFO", "propagate": False},
9186
"environ": {"level": "INFO", "propagate": False},
9287
"fiona": {"level": "INFO", "propagate": False},
9388
"rasterio": {"level": "INFO", "propagate": False},

docker/app/run_dagster_daemon.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/bin/bash
22

33
# Wait up to 30 minutes for outstanding migrations to finish running on another container, checking every 30 seconds
4-
for i in `seq 1 60`; do
4+
for i in $(seq 1 60); do
55
./manage.py migrate --plan | grep "No planned migration operations." && break
66
echo Waiting for outstanding database migrations
77
sleep 30
@@ -13,5 +13,5 @@ echo Setting up logs
1313
touch log/django.log
1414
chown -R django:django log/*
1515

16-
echo Starting Dagster with ddtrace to ${DD_TRACE_AGENT_URL}
17-
gosu django ddtrace-run dagster-daemon run $*
16+
echo Starting Dagster
17+
gosu django dagster-daemon run $*

docker/app/run_dagster_webserver.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/bin/bash
22

33
# Wait up to 30 minutes for outstanding migrations to finish running on another container, checking every 30 seconds
4-
for i in `seq 1 60`; do
4+
for i in $(seq 1 60); do
55
./manage.py migrate --plan | grep "No planned migration operations." && break
66
echo Waiting for outstanding database migrations
77
sleep 30
@@ -13,5 +13,5 @@ echo Setting up logs
1313
touch log/django.log
1414
chown -R django:django log/*
1515

16-
echo Starting Dagster with ddtrace to ${DD_TRACE_AGENT_URL}
17-
gosu django ddtrace-run dagster-webserver -h 0.0.0.0 -p 3000 -m pipelines --path-prefix /${DAGSTER_WEBSERVER_PREFIX} $*
16+
echo Starting Dagster
17+
gosu django dagster-webserver -h 0.0.0.0 -p 3000 -m pipelines --path-prefix /${DAGSTER_WEBSERVER_PREFIX} $*

docker/app/run_django.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ touch log/django.log
3939
touch log/django_sql.log
4040
chown -R django:django log/*
4141

42-
echo Starting Gunicorn with DJANGO_SETTINGS_MODULE=${DJANGO_SETTINGS_MODULE} and ddtrace to ${DD_TRACE_AGENT_URL}
43-
gosu django ddtrace-run gunicorn ${APP}.wsgi:application \
42+
echo Starting Gunicorn with DJANGO_SETTINGS_MODULE=${DJANGO_SETTINGS_MODULE}
43+
gosu django gunicorn ${APP}.wsgi:application \
4444
--name ${APP}${ENV} \
4545
--config $(dirname $(readlink -f "$0"))/gunicorn_config.py \
4646
$* 2>&1

env.example

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ MINIO_SECRET_KEY=wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY
2828
MINIO_ENDPOINT_URL=http://minio:9000 # change to http://localhost:9000 if using a local environment
2929
3030
PIP_INDEX_URL=https://pypi.python.org/simple/
31-
DD_TRACE_ENABLED=False
3231

3332
# Google API related settings
3433
# GOOGLE_APPLICATION_CREDENTIALS = '{

hea/settings/base.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,6 @@
106106
"django.contrib.admindocs",
107107
"binary_database_files",
108108
"django_extensions",
109-
"ddtrace.contrib.django",
110109
"rest_framework_gis",
111110
]
112111
PROJECT_APPS = ["common", "metadata", "baseline"]
@@ -259,7 +258,6 @@
259258
},
260259
},
261260
"loggers": {
262-
"ddtrace": {"handlers": ["logfile"], "level": "INFO"},
263261
"django.request": {"handlers": ["console", "logfile"], "level": "INFO", "propagate": False},
264262
"django.db.backends": {"handlers": ["console", "logfile"], "level": "INFO", "propagate": False},
265263
"django.security": {"handlers": ["console", "logfile"], "level": "ERROR", "propagate": False},

requirements/base.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ dagster_aws==0.22.8
44
dagster-pipes==1.6.8
55
dagster-postgres==0.22.8
66
dagster-webserver==1.6.8
7-
ddtrace==2.6.2
87
Django==5.1.4
98
django-binary-database-files==1.0.18
109
django-environ==0.11.2

0 commit comments

Comments
 (0)