feat: decouple ecomworker NR installation from COMMON_ENABLE_NEWRELIC…

…_APP
edx · Jun 26, 2024 · 911d224 · 911d224
1 parent cb036a5
commit 911d224
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/playbooks/roles/ecomworker/defaults/main.yml b/playbooks/roles/ecomworker/defaults/main.yml
@@ -26,6 +26,7 @@ ECOMMERCE_WORKER_REPOS:
 # Requires that New Relic be enabled via COMMON_ENABLE_NEWRELIC, and that
 # a key be provided via NEWRELIC_LICENSE_KEY.
 ECOMMERCE_WORKER_NEWRELIC_APPNAME: '{{ COMMON_ENVIRONMENT }}-{{ COMMON_DEPLOYMENT }}-{{ ecommerce_worker_service_name }}'
+ECOMMERCE_WORKER_ENABLE_NEWRELIC_APP: true
 ECOMMERCE_WORKER_ENABLE_NEWRELIC_DISTRIBUTED_TRACING: false
 
 # Datadog

diff --git a/playbooks/roles/ecomworker/templates/edx/app/ecomworker/ecomworker.sh.j2 b/playbooks/roles/ecomworker/templates/edx/app/ecomworker/ecomworker.sh.j2
@@ -6,7 +6,7 @@
 
 {% set executable = ecommerce_worker_venv_bin + '/celery' %}
 
-{% if COMMON_ENABLE_NEWRELIC_APP %}
+{% if ECOMMERCE_WORKER_ENABLE_NEWRELIC_APP %}
 {% set executable = ecommerce_worker_venv_bin + '/newrelic-admin run-program ' + ecommerce_worker_venv_bin + '/celery' %}
 
 export NEW_RELIC_DISTRIBUTED_TRACING_ENABLED='{{ ECOMMERCE_WORKER_ENABLE_NEWRELIC_DISTRIBUTED_TRACING }}'