ocp4_workload_showroom: move and update usernum creation to safer spot (

#8909) * move and update usernum creation to safer spot * Fix wrong indentation --------- Co-authored-by: Wolfgang Kulhanek <[email protected]>
redhat-cop · Dec 5, 2024 · f99be0e · f99be0e
1 parent ba05dce
commit f99be0e
Showing 1 changed file with 12 additions and 13 deletions.
diff --git a/ansible/roles_ocp_workloads/ocp4_workload_showroom/tasks/prepare-variables.yaml b/ansible/roles_ocp_workloads/ocp4_workload_showroom/tasks/prepare-variables.yaml
@@ -9,19 +9,6 @@
     quiet: true
     fail_msg: Required variables not defined
 
-- name: Set a default num_users
-  ansible.builtin.set_fact:
-    num_users: "{{ num_users | default(1) }}"
-
-- name: Set a user.usernum
-  agnosticd_user_info:
-    data:
-      usernum: "{{ n }}"
-    user: "user{{ n }}"
-  loop: "{{ range(1, num_users | int + 1) | list }}"
-  loop_control:
-    loop_var: n
-
 - name: Set passthrough variables to per-user data, if desired
   when:
   - agnosticd_passthrough_user_data is defined
@@ -42,6 +29,18 @@
   ansible.builtin.set_fact:
     _user_count: "{{ _showroom_user_data['users'] | length }}"
 
+- name: Set a user.usernum
+  when:
+  - _user_count is defined
+  - _user_count | int > 0
+  agnosticd_user_info:
+    data:
+      usernum: "{{ n }}"
+    user: "user{{ n }}"
+  loop: "{{ range(1, _user_count | int + 1) | list }}"
+  loop_control:
+    loop_var: n
+
 - name: Read user-info.yaml
   delegate_to: localhost
   ansible.builtin.slurp: