Don't add sudo group by default.

This formula doesn't really require the sudo group (unless
there are actually users in that group). Moreover, on FreeBSD
the 'admin' group would be wheel and not sudo.
This commit is contained in:
Simon Lloyd 2015-08-13 23:57:09 +02:00
parent b0e964f942
commit 60e94564d1
2 changed files with 4 additions and 8 deletions

View File

@ -39,9 +39,11 @@ include:
{% for group in user.get('groups', []) %} {% for group in user.get('groups', []) %}
users_{{ name }}_{{ group }}_group: users_{{ name }}_{{ group }}_group:
group: group.present:
- name: {{ group }} - name: {{ group }}
- present {% if group == 'sudo' %}
- system: True
{% endif %}
{% endfor %} {% endfor %}
users_{{ name }}_user: users_{{ name }}_user:

View File

@ -6,16 +6,10 @@ users_bash-package:
pkg.installed: pkg.installed:
- name: {{ users.bash_package }} - name: {{ users.bash_package }}
users_sudo-group:
group.present:
- name: sudo
- system: True
users_sudo-package: users_sudo-package:
pkg.installed: pkg.installed:
- name: {{ users.sudo_package }} - name: {{ users.sudo_package }}
- require: - require:
- group: users_sudo-group
- file: {{ users.sudoers_dir }} - file: {{ users.sudoers_dir }}
users_{{ users.sudoers_dir }}: users_{{ users.sudoers_dir }}: