Skip to content

Commit 9e0d95b

Browse files
authored
install helm 3.7 for canary tests (#58)
Description of changes: Latest release of `applicationautoscaling` will use `helm` 3.7 and the current helm installation will now longer be compatible. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 5bd4b38 commit 9e0d95b

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

test/canary/Dockerfile.canary

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ RUN curl -LO https://storage.googleapis.com/kubernetes-release/release/v1.18.6/b
3333
RUN curl --silent --location "https://github.com/weaveworks/eksctl/releases/latest/download/eksctl_$(uname -s)_amd64.tar.gz" | tar xz -C /tmp && mv /tmp/eksctl /bin
3434

3535
# Install Helm
36-
RUN curl -q -L "https://get.helm.sh/helm-v3.2.4-linux-amd64.tar.gz" | tar zxf - -C /usr/local/bin/ \
36+
RUN curl -q -L "https://get.helm.sh/helm-v3.7.0-linux-amd64.tar.gz" | tar zxf - -C /usr/local/bin/ \
3737
&& mv /usr/local/bin/linux-amd64/helm /usr/local/bin/helm \
3838
&& rm -r /usr/local/bin/linux-amd64 \
3939
&& chmod +x /usr/local/bin/helm

test/canary/scripts/install_controller_helm.sh

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,10 @@ function install_helm_chart() {
77
local oidc_role_arn="$2"
88
local region="$3"
99
local namespace="$4"
10-
local account_id=$(aws sts get-caller-identity --output text --query "Account")
1110

1211
yq w -i helm/values.yaml "serviceAccount.annotations" ""
1312
yq w -i helm/values.yaml 'serviceAccount.annotations."eks.amazonaws.com/role-arn"' "$oidc_role_arn"
1413
yq w -i helm/values.yaml "aws.region" $region
15-
yq w -i helm/values.yaml "aws.account_id" $account_id
1614

1715
kubectl create namespace $namespace
1816
kubectl apply -f helm/crds

0 commit comments

Comments
 (0)