Merge pull request #3522 from alphagov/disable-redis-enabled

remove `REDIS_ENABLED` flag from creds

app/config.py

@@ -114,7 +114,7 @@ class Config(object):
 
     # URL of redis instance
     REDIS_URL = os.getenv('REDIS_URL')
-    REDIS_ENABLED = os.getenv('REDIS_ENABLED') == '1'
+    REDIS_ENABLED = True
     EXPIRE_CACHE_TEN_MINUTES = 600
     EXPIRE_CACHE_EIGHT_DAYS = 8 * 24 * 60 * 60
 
@@ -404,6 +404,8 @@ class Development(Config):
     DEBUG = True
     SQLALCHEMY_ECHO = False
 
+    REDIS_ENABLED = os.getenv('REDIS_ENABLED') == '1'
+
     CSV_UPLOAD_BUCKET_NAME = 'development-notifications-csv-upload'
     CONTACT_LIST_BUCKET_NAME = 'development-contact-list'
     TEST_LETTERS_BUCKET_NAME = 'development-test-letters'

manifest.yml.j2

@@ -145,8 +145,6 @@ applications:
       FIRETEXT_INTERNATIONAL_API_KEY: '{{ FIRETEXT_INTERNATIONAL_API_KEY }}'
       FIRETEXT_INBOUND_SMS_AUTH: '{{ FIRETEXT_INBOUND_SMS_AUTH | tojson }}'
 
-      REDIS_ENABLED: '{{ REDIS_ENABLED }}'
-
       TEMPLATE_PREVIEW_API_HOST: '{{ TEMPLATE_PREVIEW_API_HOST }}'
       TEMPLATE_PREVIEW_API_KEY: '{{ TEMPLATE_PREVIEW_API_KEY }}'
 

pytest.ini

@@ -5,5 +5,6 @@ env =
     MMG_API_KEY=mmg-secret-key
     FIRETEXT_API_KEY=Firetext
     NOTIFICATION_QUEUE_PREFIX=testing
+    REDIS_ENABLED=0
 addopts = -p no:warnings
 xfail_strict = true