From f96dda6fcfd17c38ef308b549c27335e42528041 Mon Sep 17 00:00:00 2001 From: jadarsie Date: Sat, 23 Nov 2019 11:17:12 -0800 Subject: [PATCH] removed comments referencing akse --- .../src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/staging/src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go b/staging/src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go index 1771b731d2f..7332a1240d5 100644 --- a/staging/src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go +++ b/staging/src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go @@ -173,14 +173,11 @@ func decodePkcs12(pkcs []byte, password string) (*x509.Certificate, *rsa.Private // azureStackOverrides ensures that the Environment matches what AKSe currently generates for Azure Stack func azureStackOverrides(env *azure.Environment, resourceManagerEndpoint, identitySystem string) { env.ManagementPortalURL = strings.Replace(resourceManagerEndpoint, "https://management.", "https://portal.", -1) - // TODO: figure out why AKSe does this, why is autorest not setting ServiceManagementEndpoint? env.ServiceManagementEndpoint = env.TokenAudience - // TODO: figure out why AKSe does this, may not be required, ResourceManagerVMDNSSuffix is not referenced anywhere env.ResourceManagerVMDNSSuffix = strings.Replace(resourceManagerEndpoint, "https://management.", "cloudapp.", -1) env.ResourceManagerVMDNSSuffix = strings.TrimSuffix(env.ResourceManagerVMDNSSuffix, "/") if strings.EqualFold(identitySystem, ADFSIdentitySystem) { env.ActiveDirectoryEndpoint = strings.TrimSuffix(env.ActiveDirectoryEndpoint, "/") env.ActiveDirectoryEndpoint = strings.TrimSuffix(env.ActiveDirectoryEndpoint, "adfs") } - // NOTE: autorest sets KeyVaultEndpoint while AKSe does not }