Skip to content

Commit

Permalink
Merge pull request #6 from f500/ansible-lint
Browse files Browse the repository at this point in the history
Get rid of non existing pipefail option
  • Loading branch information
mjmeijerman authored Dec 30, 2024
2 parents a423141 + 0b16eb5 commit 487cca4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .ansible-lint
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,8 @@ use_default_rules: true
# the use of skip_list below because that will hide violations from the output.
# When putting ignores inside the ignore file, they are marked as ignored, but
# still visible, making it easier to address later.
#skip_list:
skip_list:
- risky-shell-pipe
# - skip_this_tag

# Ansible-lint does not automatically load rules that have the 'opt-in' tag.
Expand Down
3 changes: 2 additions & 1 deletion tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,11 @@
- /etc/skel

- name: Determine ssh user homedir
ansible.builtin.shell: "set -o pipefail && getent passwd {{ item }} | cut -d: -f6"
ansible.builtin.shell: "getent passwd {{ item }} | cut -d: -f6"
register: bashrc_homedirs
with_items: "{{ bashrc_users }}"
changed_when: false
failed_when: bashrc_homedirs.rc != 0

- name: Write ssh user homedir .bashrc
ansible.builtin.copy:
Expand Down

0 comments on commit 487cca4

Please sign in to comment.