feat: decouple ecomworker NR installation from COMMON_ENABLE_NEWRELIC… (

#53) * feat: decouple ecomworker NR installation from COMMON_ENABLE_NEWRELIC_APP * chore: Update playbooks/roles/ecomworker/defaults/main.yml Co-authored-by: Tim McCormack <[email protected]> --------- Co-authored-by: Tim McCormack <[email protected]>
edx · Jun 26, 2024 · acfb8b8 · acfb8b8
1 parent f32e4df
commit acfb8b8
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: '{{ COMMON_ENABLE_NEWRELIC_APP }}'
 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 }}'