From 985c8a991a994ca1c558acc938713aa87ae72283 Mon Sep 17 00:00:00 2001 From: Marco Dinis Date: Tue, 22 Oct 2024 13:53:58 +0100 Subject: [PATCH] Discover Wizard: drop v13 edge case for Kube Access (#47790) --- .../src/Discover/Kubernetes/HelmChart/HelmChart.tsx | 8 -------- 1 file changed, 8 deletions(-) diff --git a/web/packages/teleport/src/Discover/Kubernetes/HelmChart/HelmChart.tsx b/web/packages/teleport/src/Discover/Kubernetes/HelmChart/HelmChart.tsx index 53a94e1b17cb4..6d383690f48c1 100644 --- a/web/packages/teleport/src/Discover/Kubernetes/HelmChart/HelmChart.tsx +++ b/web/packages/teleport/src/Discover/Kubernetes/HelmChart/HelmChart.tsx @@ -365,14 +365,6 @@ export function generateCmd(data: GenerateCmdProps) { // AutomaticUpgradesTargetVersion contains a v, eg, v13.4.2. // However, helm chart expects no 'v', eg, 13.4.2. deployVersion = data.automaticUpgradesTargetVersion.replace(/^v/, ''); - - // TODO(marco): remove when stable/cloud moves to v14 - // For v13 releases of the helm chart, we must remove the App role. - // We get the following error otherwise: - // Error: INSTALLATION FAILED: execution error at (teleport-kube-agent/templates/statefulset.yaml:26:28): at least one of 'apps' and 'appResources' is required in chart values when app role is enabled, see README - if (deployVersion.startsWith('13.')) { - roles = ['Kube']; - } } const yamlRoles = roles.join(',').toLowerCase();