chore: simplify some variables

edx · Jun 25, 2024 · a1a0665 · a1a0665
1 parent e1aa586
commit a1a0665
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/playbooks/roles/edx_django_service/defaults/main.yml b/playbooks/roles/edx_django_service/defaults/main.yml
@@ -102,7 +102,7 @@ edx_django_service_basic_auth_exempted_paths_extra: []
 edx_django_service_basic_auth_exempted_paths: '{{ edx_django_service_basic_auth_exempted_paths_default + edx_django_service_basic_auth_exempted_paths_extra }}'
 
 edx_django_service_newrelic_appname: '{{ COMMON_ENVIRONMENT }}-{{ COMMON_DEPLOYMENT }}-{{ edx_django_service_name }}'
-edx_django_service_enable_newrelic_app: true
+edx_django_service_enable_newrelic_app: '{{ COMMON_ENABLE_NEWRELIC_APP  }}'
 edx_django_service_enable_newrelic_distributed_tracing: false
 edx_django_datadog_service: 'edx-{{ edx_django_service_name }}'
 edx_django_service_datadog_enable: '{{ COMMON_ENABLE_DATADOG and (COMMON_ENABLE_DATADOG_APP or edx_django_service_name in COMMON_ENABLE_DATADOG_APP_SERVICES) }}'

diff --git a/playbooks/roles/edx_django_service/templates/edx/app/app/app.sh.j2 b/playbooks/roles/edx_django_service/templates/edx/app/app/app.sh.j2
@@ -6,7 +6,7 @@
 
 {% set executable = edx_django_service_venv_bin + '/gunicorn' %}
 
-{% if COMMON_ENABLE_NEWRELIC_APP and edx_django_service_enable_newrelic_app  %}
+{% if edx_django_service_enable_newrelic_app %}
 {% set executable = edx_django_service_venv_bin + '/newrelic-admin run-program ' + executable %}
 
 export NEW_RELIC_DISTRIBUTED_TRACING_ENABLED="{{ edx_django_service_enable_newrelic_distributed_tracing }}"
@@ -53,7 +53,7 @@ exec docker run \
   --name {{ edx_django_service_name }} \
   --mount type=bind,src={{ edx_django_service_app_config_file }},dst={{ edx_django_service_app_config_file }} \
   --publish 127.0.0.1:{{ edx_django_service_gunicorn_port }}:{{ edx_django_service_gunicorn_port }}/tcp \
-{% if COMMON_ENABLE_NEWRELIC_APP and edx_django_service_enable_newrelic_app  %}
+{% if edx_django_service_enable_newrelic_app %}
   --env NEW_RELIC_DISTRIBUTED_TRACING_ENABLED \
   --env NEW_RELIC_APP_NAME \
   --env NEW_RELIC_PROCESS_HOST_DISPLAY_NAME \