various: fix some more shell variables that were accidentally converted to builtin.shell

Signed-off-by: Kevin Fenzi <kevin@scrye.com>

playbooks/groups/pkgs.yml

@@ -49,7 +49,7 @@
     user:
       name: pagure
       uid: "{{ pagure_static_uid }}"
-      ansible.builtin.shell: /sbin/nologin
+      shell: /sbin/nologin
       comment: "Pagure User"
     when: env == "staging"
     ## The above should be deleted when we rebuild pkgs02 in future.

roles/bodhi2/base/tasks/main.yml

@@ -19,7 +19,7 @@
     name: bodhi
     group: bodhi
     comment: Bodhi Server
-    ansible.builtin.shell: /sbin/nologin
+    shell: /sbin/nologin
     home: /usr/share/bodhi
     system: yes
     state: present

roles/mirror_pagure_ansible/tasks/main.yml

@@ -14,7 +14,7 @@
 - name: Setup pagure user
   user:
     name: mirror_pagure_ansible
-    ansible.builtin.shell: /sbin/nologin
+    shell: /sbin/nologin
     comment: "mirror_pagure_ansible User"
   tags:
   - mirror_pagure_ansible

roles/openqa/server/tasks/main.yml

@@ -97,7 +97,7 @@
     home: "/var/lib/openqa"
     createhome: no
     system: yes
-    ansible.builtin.shell: /sbin/nologin
+    shell: /sbin/nologin
   when: "openqa_static_uid is defined"
 
 - name: Remove old scratch repo directory

roles/robosignatory/tasks/main.yml

@@ -29,7 +29,7 @@
     system: yes
     home: /etc/robosignatory
     comment: Robosignatory
-    ansible.builtin.shell: /sbin/nologin
+    shell: /sbin/nologin
     uid: 263
   tags:
   - config

roles/web-data-analysis/tasks/main.yml

@@ -108,7 +108,7 @@
   user:
     name: countme
     group: countme
-    ansible.builtin.shell: /sbin/nologin
+    shell: /sbin/nologin
     home: /srv/countme
     comment: "DNF countme counter"
   tags: