From 12c8e08d6f3492744726d7b42de24f2b793ca632 Mon Sep 17 00:00:00 2001 From: Erin Call Date: Fri, 27 Dec 2019 10:49:22 -0800 Subject: [PATCH] Mention that EKS isn't currently supported [#5] We'd like to support it eventually, but the current state of affairs doesn't justify the effort. Also removed some vestigial code that was copy-pasta from the kubeconfig in drone-helm. --- README.md | 1 + assets/kubeconfig.tpl | 12 ------------ 2 files changed, 1 insertion(+), 12 deletions(-) diff --git a/README.md b/README.md index 592baee..96eedaa 100644 --- a/README.md +++ b/README.md @@ -59,6 +59,7 @@ steps: drone-helm3 is largely backwards-compatible with drone-helm. There are some known differences: * `prefix` must be supplied via the `settings` block, not `environment`. +* EKS is not supported. See [#5](https://github.com/pelotech/drone-helm3/issues/5) for more information. * Several settings no longer have any effect: * `purge` -- this is the default behavior in Helm 3 * `recreate_pods` diff --git a/assets/kubeconfig.tpl b/assets/kubeconfig.tpl index 92ec2c7..3ed8630 100644 --- a/assets/kubeconfig.tpl +++ b/assets/kubeconfig.tpl @@ -24,16 +24,4 @@ users: user: {{- if .Token }} token: {{ .Token }} -{{- else if .EKSCluster }} - exec: - apiVersion: client.authentication.k8s.io/v1alpha1 - command: aws-iam-authenticator - args: - - "token" - - "-i" - - "{{ .EKSCluster }}" - {{- if .EKSRoleARN }} - - "-r" - - "{{ .EKSRoleARN }}" - {{- end }} {{- end }}