review fix

securesign · Nov 24, 2024 · dbad4da · dbad4da
1 parent 3a9beb3
commit dbad4da
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/molecule/default/prepare.yml b/molecule/default/prepare.yml
@@ -5,7 +5,7 @@
   vars:
     users:
       - user: testingUser
-        password: $y$j9T$WNDCnTZIfLYSpTiZ1yFdJ1$fvdsvfLEdpi39mS/SbfsgPQdAQ//D8xySIXF5hLFdU2
+        password: testingPassword0x
   vars_files:
     - vars/vars.yml
 

diff --git a/molecule/testing_user_setup.yaml b/molecule/testing_user_setup.yaml
@@ -2,12 +2,12 @@
 - name: Create a non-root sudoer user
   ansible.builtin.user:
     name: "{{ item.user }}"
-    password: "{{ item.password }}"
     shell: /bin/bash
+    group: wheel
     create_home: yes
 
-- name: Add user to wheel group
-  shell: usermod -aG wheel {{ item.user }}
+- name: Set password for testingUser
+  ansible.builtin.shell: echo {{ item.password }} | passwd --stdin {{ item.user }}
 
 - name: Configure Sudoers for the user
   ansible.builtin.lineinfile:
@@ -44,7 +44,7 @@
   ansible.builtin.lineinfile:
     path: "{{ molecule_ephemeral_directory }}/inventory/molecule_inventory.yml"
     regexp: 'ansible_user:'
-    line: "ansible_user: {{ item.user }}, ansible_become: true, ansible_become_password: {{ item.password }}}"
+    line: "ansible_user: {{ item.user }}, ansible_become_password: {{ item.password }}, ansible_become: true}"
   delegate_to: localhost
 
 - name: Force refresh inventory

diff --git a/molecule/user_provided/prepare.yml b/molecule/user_provided/prepare.yml
@@ -5,7 +5,7 @@
   vars:
     users:
       - user: testingUser
-        password: $y$j9T$WNDCnTZIfLYSpTiZ1yFdJ1$fvdsvfLEdpi39mS/SbfsgPQdAQ//D8xySIXF5hLFdU2
+        password: testingUser0x
   vars_files:
     - vars/vars.yml
   tasks: