Add assertion for required deploy-time variable

companieshouse · Jun 28, 2021 · cf56531 · cf56531
1 parent f3cc2c1
commit cf56531
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/roles/deploy/defaults/main.yml b/roles/deploy/defaults/main.yml
@@ -178,12 +178,6 @@ tuxedo_log_files:
       cloudwatch_log_group_name: frontend-tuxedo-xml-ulog
 
 tuxedo_service_group: tuxedo
-tuxedo_service_users:
-  - ceu
-  - chd
-  - ewf
-  - xml
-
 tuxedo_service_ipc_key_start: 48000
 tuxedo_service_ipc_key_increment: 100
 tuxedo_service_domain_port_start: 38000

diff --git a/roles/deploy/tasks/main.yml b/roles/deploy/tasks/main.yml
@@ -10,6 +10,7 @@
 - name: Check required variables are set
   assert:
     that:
+      - tuxedo_service_users is defined and tuxedo_service_users | length > 0
       - environment_name is defined and environment_name | trim | length > 0
       - application_artifact_path is defined and application_artifact_path | trim | length > 0
       - application_configs_path is defined and application_configs_path | trim | length > 0