From 69ff9fcdd32aa5cf1fc75dac0b0932f4a79f3dbe Mon Sep 17 00:00:00 2001 From: Robert Raposa Date: Wed, 18 Dec 2024 16:07:48 -0500 Subject: [PATCH] temp: enable DD_DJANGO_INSTRUMENT_MIDDLEWARE Temporarily enabling middleware for end-of-year to avoid deployments if an issue arises, and to not have middleware as a blindspot. See https://github.com/edx/edx-arch-experiments/issues/879 --- playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2 | 3 ++- playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2 | 3 ++- playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2 | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2 b/playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2 index b85021300ff..c2f20ce3a16 100644 --- a/playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2 +++ b/playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2 @@ -20,7 +20,8 @@ export DD_TRACE_LOG_STREAM_HANDLER=false # Suppress middleware spans because there are about 100 for each request. # Remove (or set to true) for debugging. -export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false +# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879 +export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true {% endif -%} # We want to be able to toggle this on separately from DD in general. diff --git a/playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2 b/playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2 index fc8e295cb59..730f313a060 100644 --- a/playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2 +++ b/playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2 @@ -20,7 +20,8 @@ export DD_TRACE_LOG_STREAM_HANDLER=false # Suppress middleware spans because there are about 100 for each request. # Remove (or set to true) for debugging. -export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false +# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879 +export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true {% endif -%} diff --git a/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2 b/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2 index 457c0120445..95dd4023ef2 100644 --- a/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2 +++ b/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2 @@ -20,7 +20,8 @@ export DD_TRACE_LOG_STREAM_HANDLER=false # Suppress middleware spans because there are about 100 for each request. # Remove (or set to true) for debugging. -export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false +# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879 +export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true {% endif -%} # We want to be able to toggle this on separately from DD in general.