Skip to content

Commit f96dda6

Browse files
committed
removed comments referencing akse
1 parent 3322ff9 commit f96dda6

File tree

1 file changed

+0
-3
lines changed
  • staging/src/k8s.io/legacy-cloud-providers/azure/auth

1 file changed

+0
-3
lines changed

staging/src/k8s.io/legacy-cloud-providers/azure/auth/azure_auth.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,14 +173,11 @@ func decodePkcs12(pkcs []byte, password string) (*x509.Certificate, *rsa.Private
173173
// azureStackOverrides ensures that the Environment matches what AKSe currently generates for Azure Stack
174174
func azureStackOverrides(env *azure.Environment, resourceManagerEndpoint, identitySystem string) {
175175
env.ManagementPortalURL = strings.Replace(resourceManagerEndpoint, "https://management.", "https://portal.", -1)
176-
// TODO: figure out why AKSe does this, why is autorest not setting ServiceManagementEndpoint?
177176
env.ServiceManagementEndpoint = env.TokenAudience
178-
// TODO: figure out why AKSe does this, may not be required, ResourceManagerVMDNSSuffix is not referenced anywhere
179177
env.ResourceManagerVMDNSSuffix = strings.Replace(resourceManagerEndpoint, "https://management.", "cloudapp.", -1)
180178
env.ResourceManagerVMDNSSuffix = strings.TrimSuffix(env.ResourceManagerVMDNSSuffix, "/")
181179
if strings.EqualFold(identitySystem, ADFSIdentitySystem) {
182180
env.ActiveDirectoryEndpoint = strings.TrimSuffix(env.ActiveDirectoryEndpoint, "/")
183181
env.ActiveDirectoryEndpoint = strings.TrimSuffix(env.ActiveDirectoryEndpoint, "adfs")
184182
}
185-
// NOTE: autorest sets KeyVaultEndpoint while AKSe does not
186183
}

0 commit comments

Comments
 (0)