search

geerlingguy / ansible-role-docker

Ansible Role - Docker
https://galaxy.ansible.com/geerlingguy/docker/
MIT License
1.85k stars 859 forks source link

ansible-lint warn #191

Closed jamel-hrt closed 4 years ago

jamel-hrt commented 4 years ago

Hi

When we use ansible-lint to check the role, there is this warning. ansible-lint version : 4.2.0 geerlingguy.docker version : 2.7.0

$ ansible-lint roles/geerlingguy.docker/
[306] Shells that use pipes should set the pipefail option
roles/geerlingguy.docker/tasks/setup-Debian.yml:29
Task/Handler: Add Docker apt key (alternative for older systems without SNI).
geerlingguy commented 4 years ago

@jamel-hrt - I specifically ignore 306 because that only works on Red Hat systems (see https://github.com/geerlingguy/ansible-role-docker/blob/master/.ansible-lint). There's an open issue upstream which could use feedback if you also agree: https://github.com/ansible/ansible-lint/issues/497

Thanks!