Skip to content

Commit

Permalink
Revert "temp: enable DD_DJANGO_INSTRUMENT_MIDDLEWARE (#142)" (#151)
Browse files Browse the repository at this point in the history
This reverts commit 5ace2fe.
  • Loading branch information
robrap authored Jan 6, 2025
1 parent 9a148c4 commit ad70187
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
3 changes: 1 addition & 2 deletions playbooks/roles/edxapp/templates/edx/app/edxapp/cms.sh.j2
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ 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.
# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false
{% endif -%}

# We want to be able to toggle this on separately from DD in general.
Expand Down
3 changes: 1 addition & 2 deletions playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ 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.
# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false

{% endif -%}

Expand Down
3 changes: 1 addition & 2 deletions playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ 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.
# Temporarily enabled as part of https://github.com/edx/edx-arch-experiments/issues/879
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=true
export DD_DJANGO_INSTRUMENT_MIDDLEWARE=false
{% endif -%}

# We want to be able to toggle this on separately from DD in general.
Expand Down

0 comments on commit ad70187

Please sign in to comment.