Skip to content

Commit

Permalink
temp: Try removing NR license key for edge LMS to see if DD is affect…
Browse files Browse the repository at this point in the history
…ed (#45)

temp: Try removing NR license key for stage/edge LMS to see if DD is affected
  • Loading branch information
timmc-edx authored Jun 20, 2024
1 parent 8004527 commit 54fd8e7
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions playbooks/roles/edxapp/templates/edx/app/edxapp/lms.sh.j2
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,18 @@ if command -v ec2metadata >/dev/null 2>&1; then
HOSTNAME=$(hostname)
export NEW_RELIC_PROCESS_HOST_DISPLAY_NAME="$HOSTNAME-$INSTANCEID"
fi


{% if COMMON_ENVIRONMENT == "stage" or COMMON_DEPLOYMENT == "edge" %}
# Another attempt at a temporary workaround for trace concatenation in Datadog.
# (Maybe this will keep NR trace instrumentation in place, but not cost us anything,
# because no data will be shipped to NR.)
# See https://github.com/edx/edx-arch-experiments/issues/692
export NEW_RELIC_LICENSE_KEY="xxx"
{% else %}
export NEW_RELIC_LICENSE_KEY="{{ NEWRELIC_LICENSE_KEY }}"
{% endif %}

{% endif -%}

{% if EDXAPP_DATADOG_ENABLE %}
Expand Down

0 comments on commit 54fd8e7

Please sign in to comment.