Commit 38f8f0c7 by Cory Lee

OPS-2492 Rename ecommerce variables to be consistent with other ecommerce variables

parent e3b41451
- Role: ecommerce
- Added `ECOMMERCE_CELERY_BROKER_TRANSPORT` and renamed `ECOMMERCE_BROKER_HOST`
to `ECOMMERCE_CELERY_BROKER_HOSTNAME`. This is to support non-amqp brokers,
specifically redis. If `ECOMMERCE_CELERY_BROKER_HOSTNAME` is unset it will use
- Added `ECOMMERCE_BROKER_TRANSPORT` and renamed `ECOMMERCE_BROKER_HOST`
to `ECOMMERCE_BROKER_HOSTNAME`. This is to support non-amqp brokers,
specifically redis. If `ECOMMERCE_BROKER_HOSTNAME` is unset it will use
the value of `ECOMMERCE_BROKER_HOST`, however it is recommended to update
your configuration to set `ECOMMERCE_CELERY_BROKER_HOSTNAME` explicitly.
your configuration to set `ECOMMERCE_BROKER_HOSTNAME` explicitly.
- Role: credentials
- This role is now dependent on the edx_django_service role. Settings are all the same, but nearly all of the tasks are performed by the edx_django_service role.
......
......@@ -147,9 +147,9 @@ ECOMMERCE_BROKER_USERNAME: 'celery'
ECOMMERCE_BROKER_PASSWORD: 'celery'
ECOMMERCE_BROKER_HOST: '{{ ansible_default_ipv4.address }}'
ECOMMERCE_BROKER_PORT: 5672
ECOMMERCE_CELERY_BROKER_TRANSPORT: 'amqp'
ECOMMERCE_CELERY_BROKER_HOSTNAME: '{{ ECOMMERCE_BROKER_HOST }}'
ECOMMERCE_BROKER_URL: '{{ ECOMMERCE_CELERY_BROKER_TRANSPORT }}://{{ ECOMMERCE_BROKER_USERNAME }}:{{ ECOMMERCE_BROKER_PASSWORD }}@{{ ECOMMERCE_CELERY_BROKER_HOSTNAME }}:{{ ECOMMERCE_BROKER_PORT }}'
ECOMMERCE_BROKER_TRANSPORT: 'amqp'
ECOMMERCE_BROKER_HOSTNAME: '{{ ECOMMERCE_BROKER_HOST }}'
ECOMMERCE_BROKER_URL: '{{ ECOMMERCE_BROKER_TRANSPORT }}://{{ ECOMMERCE_BROKER_USERNAME }}:{{ ECOMMERCE_BROKER_PASSWORD }}@{{ ECOMMERCE_BROKER_HOSTNAME }}:{{ ECOMMERCE_BROKER_PORT }}'
ECOMMERCE_DISCOVERY_SERVICE_URL: 'http://localhost:8008'
ECOMMERCE_ENTERPRISE_URL: '{{ ECOMMERCE_LMS_URL_ROOT }}'
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment