Move to Helm v3

gmerlin-claranet · Oct 5, 2021 · 6b664e5 · 6b664e5
1 parent c9bb324
commit 6b664e5
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 25 deletions.
diff --git a/helm/aws-load-balancer-controller/Chart.yaml b/helm/aws-load-balancer-controller/Chart.yaml
@@ -1,4 +1,4 @@
-apiVersion: v1
+apiVersion: v2
 name: aws-load-balancer-controller
 description: AWS Load Balancer Controller Helm chart for Kubernetes
 version: 1.2.9

diff --git a/helm/aws-load-balancer-controller/README.md b/helm/aws-load-balancer-controller/README.md
@@ -26,6 +26,7 @@ AWS Load Balancer controller manages the following AWS resources
    - 1.18.18+ for 1.18
    - 1.19.10+ for 1.19
 - IAM permissions
+- Helm v3 is needed
 
 The controller runs on the worker nodes, so it needs access to the AWS ALB/NLB resources via IAM permissions. The
 IAM permissions can either be setup via IAM roles for ServiceAccount or can be attached directly to the worker node IAM roles.
@@ -187,7 +188,7 @@ The default values set by the application itself can be confirmed [here](https:/
 | `webhookTLS.caCert`                         | TLS CA certificate for webhook (auto-generated if not provided)                                          | ""                                                                                 |
 | `webhookTLS.cert`                           | TLS certificate for webhook (auto-generated if not provided)                                             | ""                                                                                 |
 | `webhookTLS.key`                            | TLS private key for webhook (auto-generated if not provided)                                             | ""                                                                                 |
-| `keepTLSSecret`                             | Keeps the usage of existing TLS Secret                                                                   | `false`                                                                            |
+| `keepTLSSecret`                             | Reuse existing TLS Secret during chart upgrade                                                           | `false`                                                                            |
 | `serviceAnnotations`                        | Annotations to be added to the provisioned webhook service resource                                      | `{}`                                                                               |
 | `serviceMaxConcurrentReconciles`            | Maximum number of concurrently running reconcile loops for service                                       | None                                                                               |
 | `targetgroupbindingMaxConcurrentReconciles` | Maximum number of concurrently running reconcile loops for targetGroupBinding                            | None                                                                               |

diff --git a/helm/aws-load-balancer-controller/values.yaml b/helm/aws-load-balancer-controller/values.yaml
@@ -133,7 +133,7 @@ webhookTLS:
   cert:
   key:
 
-# keepTLSSecret keeps using the existing TLS secrets, false by default
+# keepTLSSecret specifies whether to reuse existing TLS secret for chart upgrade
 keepTLSSecret: false
 
 # Maximum number of concurrently running reconcile loops for service (default 3)

diff --git a/test/helm/helm-lint.sh b/test/helm/helm-lint.sh
@@ -7,7 +7,6 @@ SCRIPTPATH="$( cd "$(dirname "$0")" ; pwd -P )"
 TMP_DIR="$SCRIPTPATH/../../build"
 PLATFORM=$(uname | tr '[:upper:]' '[:lower:]')
 HELM3_VERSION="3.3.1"
-HELM2_VERSION="2.16.10"
 HELM_DIR="${SCRIPTPATH}/../../helm"
 LB_HELM_CHART=${HELM_DIR}/aws-load-balancer-controller
 
@@ -21,37 +20,16 @@ if [ ! -x "$TMP_DIR/helm" ]; then
     echo " Downloaded the \"helm\" binary"
 fi
 
-if [ ! -x "$TMP_DIR/helm2" ]; then
-    echo " Downloading the \"helm2\" binary"
-    curl -L https://get.helm.sh/helm-v$HELM2_VERSION-$PLATFORM-amd64.tar.gz | tar zxf - -C $TMP_DIR
-    mv $TMP_DIR/$PLATFORM-amd64/helm $TMP_DIR/helm2
-    chmod +x $TMP_DIR/helm2
-    echo " Downloaded the \"helm2\" binary"
-fi
 export PATH=$TMP_DIR:$PATH
 
 echo "=============================================================================="
 echo "                     Linting Helm Chart w/ Helm v3"
 echo "=============================================================================="
 helm lint $LB_HELM_CHART
 
-echo "=============================================================================="
-echo "                     Linting Helm Chart w/ Helm v2"
-echo "=============================================================================="
-helm2 lint $LB_HELM_CHART
-
-echo " Helm Linting for v2 and v3 have successfully completed!"
-
 echo "=============================================================================="
 echo "                   Generate Template w/ Helm v3"
 echo "=============================================================================="
 
 helm template aws-load-balancer-controller "${LB_HELM_CHART}" --debug --namespace=kube-system -f "${LB_HELM_CHART}/test.yaml" > /dev/null
 
-echo "=============================================================================="
-echo "                   Generate Template w/ Helm v2"
-echo "=============================================================================="
-
-helm2 template --name aws-load-balancer-controller "${LB_HELM_CHART}" --debug --namespace=kube-system -f "${LB_HELM_CHART}/test.yaml" > /dev/null
-
-echo " Helm template generation for v2 and v3 have successfully completed!"