From 70f2bc42bd5ccf9bd82ccecff888d3451e1c3c69 Mon Sep 17 00:00:00 2001 From: Mohammed Naser Date: Tue, 9 Aug 2022 16:30:21 -0400 Subject: [PATCH] Start using ensure-helm role for publishing At the moment, our publishing scripts have their own little way of deploying Helm so it's using the old version of Helm which is failing linting. This updates it so that it matches how we're running it inside our lint playbook, and it will also fix the builds since it's also failing because of the old version of Helm. Change-Id: I719c2dadc3ca87912234ac13d87d63e8c7b779a7 --- playbooks/build-chart.yaml | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/playbooks/build-chart.yaml b/playbooks/build-chart.yaml index 9cd77d294..cd283ac6a 100644 --- a/playbooks/build-chart.yaml +++ b/playbooks/build-chart.yaml @@ -12,19 +12,11 @@ # limitations under the License. - hosts: all - tasks: - - name: install helm3 - become_user: root - shell: | - TMP_DIR=$(mktemp -d) - curl -sSL https://get.helm.sh/helm-${HELM_VERSION}-linux-amd64.tar.gz | tar -zxv --strip-components=1 -C ${TMP_DIR} - sudo mv ${TMP_DIR}/helm /usr/bin/helm - rm -rf ${TMP_DIR} - environment: - HELM_VERSION: "v3.3.4" - args: - executable: /bin/bash + roles: + - name: ensure-helm + helm_version: "3.6.3" + tasks: - name: make all make: chdir: "{{ zuul.project.src_dir }}"