airship
/
treasuremap
Code Issues Proposed changes

zuul: Remove sudo from install-docker playbook 

Ansible recommends the use `become: yes` over `sudo` in playbooks and
only allows the use of `sudo` for backwards compatibility. This commit
removes sudo from the install-docker playbook.

[0] https://docs.ansible.com/ansible/2.7/user_guide/become.html#id1

Change-Id: I3edf82ee118827b8d94f169778463152ebc044f8
This commit is contained in:
Drew Walters 2018-11-21 18:33:05 -06:00
parent 6d6a598945
commit 77cd63509b
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tools/gate/playbooks/install-docker.yaml
View File

@ -18,13 +18,14 @@
shell: |
set -xe;
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add -
sudo apt-add-repository \
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | apt-key add -
apt-add-repository \
"deb [arch=amd64] https://download.docker.com/linux/ubuntu \
$(lsb_release -cs) \
stable"
sudo apt-get update
sudo apt-get install --no-install-recommends -y docker-ce
apt-get update
apt-get install --no-install-recommends -y docker-ce
args:
chdir: "{{ zuul.project.src_dir }}"
become: yes