majic-ansible-roles Changeset

Changeset - 0be45939fc2a

Parent rev.

Child rev.

[Not reviewed]

0 4 0

Branko Majic (branko) - 15 months ago 2024-09-03 15:42:49
branko@majic.rs

MAR-218: Fix erroneous replacement of shell attribute (from FQCN fixes).

4 files changed with 6 insertions and 6 deletions:

roles/common/tasks/main.yml

roles/php_website/tasks/main.yml

roles/wsgi_website/tasks/main.yml

roles/xmpp_server/molecule/default/prepare.yml

0 comments (0 inline, 0 general)

roles/common/tasks/main.yml

➞

Show inline comments

@@ @@ -160,13 +160,13 @@ @@
   ansible.builtin.user:
     name: "{{ item.name }}"
     uid: "{{ item.uid | default(omit) }}"
     group: "{{ item.name }}"
     groups: "{{ ','.join(item.additional_groups | default([])) }}"
     append: true
-    ansible.builtin.shell: /bin/bash
     shell: /bin/bash
     state: present
     password: "{{ item.password | default('!') }}"
     update_password: on_create
   with_items: "{{ os_users }}"
 - name: Set-up authorised keys

roles/php_website/tasks/main.yml

➞

Show inline comments

@@ @@ -8,13 +8,13 @@ @@
 - name: Create PHP website admin user
   ansible.builtin.user:
     name: "{{ admin }}"
     uid: "{{ admin_uid | default(omit) }}"
     group: "{{ user }}"
-    ansible.builtin.shell: /bin/bash
     shell: /bin/bash
     createhome: true
     home: "{{ home }}"
     state: present
 - name: Set-up directory for storing user profile configuration files
   ansible.builtin.file:
@@ @@ -34,13 +34,13 @@ @@
     createhome: false
     state: present
     home: "{{ home }}"
     # This is a workaround for a rather stupid bug that Debian seems
     # uninterested to backport -
     # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=865762
-    ansible.builtin.shell: /bin/sh
     shell: /bin/sh
 - name: Add nginx user to website group
   ansible.builtin.user:
     name: "www-data"
     groups: "{{ user }}"
     append: "yes"

roles/wsgi_website/tasks/main.yml

➞

Show inline comments

@@ @@ -8,13 +8,13 @@ @@
 - name: Create WSGI website admin user
   ansible.builtin.user:
     name: "{{ admin }}"
     uid: "{{ admin_uid | default(omit) }}"
     group: "{{ user }}"
-    ansible.builtin.shell: /bin/bash
     shell: /bin/bash
     createhome: true
     home: "{{ home }}"
     state: present
 - name: Set-up directory for storing user profile configuration files
   ansible.builtin.file:
@@ @@ -50,13 +50,13 @@ @@
     createhome: false
     state: present
     home: "{{ home }}"
     # This is a workaround for a rather stupid bug that Debian seems
     # uninterested to backport -
     # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=865762
-    ansible.builtin.shell: /bin/sh
     shell: /bin/sh
 - name: Add nginx user to website group
   ansible.builtin.user:
     name: www-data
     groups: "{{ user }}"
     append: true

roles/xmpp_server/molecule/default/prepare.yml

➞

Show inline comments

@@ @@ -148,13 +148,13 @@ @@
         state: present
     - name: Create dedicated user for testing
       ansible.builtin.user:
         name: user
         group: user
-        ansible.builtin.shell: /bin/bash
         shell: /bin/bash
     - name: Deploy mcabber configuration files
       ansible.builtin.template:
         src: tests/data/mcabber.cfg.j2
         dest: "~user/{{ item.jid }}.cfg"
         owner: user

0 comments (0 inline, 0 general)