Merge pull request #53 from systemli/Remove-secrets-variable-from-role

Remove secrets variable from role
systemli · Oct 22, 2023 · 450b019 · 450b019
2 parents e9b1ebe + 4dc758b
commit 450b019
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/defaults/main.yml b/defaults/main.yml
@@ -1,7 +1,7 @@
 ---
 userli_version: "3.2.2"
 userli_mysql_user: "userli"
-userli_mysql_password: "{{ secrets_userli_mysql_password }}"
+userli_mysql_password: ""
 userli_mysql_db: "userli"
 userli_mysql_priv: "{{ userli_mysql_db }}.*:ALL"
 userli_db_type: mysql

diff --git a/tasks/main.yml b/tasks/main.yml
@@ -1,4 +1,7 @@
 ---
+- name: Preflight checks
+  ansible.builtin.include_tasks: preflight.yml
+
 - name: Ensure group is present
   ansible.builtin.group:
     name: "{{ userli_group }}"

diff --git a/tasks/preflight.yml b/tasks/preflight.yml
@@ -0,0 +1,7 @@
+---
+- name: Assert that required variables are set
+  ansible.builtin.assert:
+    that:
+      - userli_mysql_password is defined and userli_mysql_password != ""
+    fail_msg: "userli_mysql_password is not defined or empty"
+    success_msg: "userli_mysql_password is defined and not empty"