From 526127fc2f226992136f8aa852e19e227114eaf6 Mon Sep 17 00:00:00 2001 From: Luke Repko Date: Fri, 23 Feb 2024 16:08:02 -0600 Subject: [PATCH] Fix exec fail using shell in lieu of command The linter told me to use `command` in lieu of shell which breaks functionality due to additional commands being treated as extra arguments to the first command. Revert to using shell where needed so the right thing is done. --- ansible/playbooks/infra-deploy.yaml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ansible/playbooks/infra-deploy.yaml b/ansible/playbooks/infra-deploy.yaml index 2a15a3188..e9e959357 100644 --- a/ansible/playbooks/infra-deploy.yaml +++ b/ansible/playbooks/infra-deploy.yaml @@ -584,14 +584,16 @@ ignore_errors: true failed_when: hosts_updated.rc not in [0, 1] - name: Add inventory to hosts file iff no 172.31. IPs exist - command: cat /etc/genestack/inventory/{{ os_network_name }}-inventory.ini | awk '/ansible/ { print $2, $1 }' | cut -f2 -d"=" >> /etc/hosts + shell: cat /etc/genestack/inventory/{{ os_network_name }}-inventory.ini | awk '/ansible/ { print $2, $1 }' | cut -f2 -d"=" >> /etc/hosts when: hosts_updated.rc == 1 + tags: + - skip_ansible_lint - name: Check if kubectl exists command: test -f /usr/local/bin/kubectl register: kubectl_exists failed_when: kubectl_exists.rc not in [0, 1] - name: Download kubectl - command: curl -L "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl" -o /usr/local/bin/kubectl + shell: curl -L "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl" -o /usr/local/bin/kubectl when: kubectl_exists.rc == 1 tags: - skip_ansible_lint