Skip to content

Commit

Permalink
Revert "workaround cloud provider chart federated token bug"
Browse files Browse the repository at this point in the history
This reverts commit d6420d0.
  • Loading branch information
nojnhuh committed Nov 3, 2023
1 parent 18531b1 commit 1867a47
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
1 change: 0 additions & 1 deletion scripts/ci-entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,6 @@ install_cloud_provider_azure() {
--set cloudControllerManager.cloudConfig="${CLOUD_CONFIG}" \
--set cloudControllerManager.cloudConfigSecretName="${CONFIG_SECRET_NAME}" \
--set cloudControllerManager.logVerbosity="${CCM_LOG_VERBOSITY}" \
--set-string cloudControllerManager.federatedTokenPath= \
--set-string cloudControllerManager.clusterCIDR="${CCM_CLUSTER_CIDR}" "${CCM_IMG_ARGS[@]}" || return 1
}

Expand Down
5 changes: 1 addition & 4 deletions test/e2e/cloud-provider-azure.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,7 @@ func InstallCNIAndCloudProviderAzureHelmChart(ctx context.Context, input cluster
fmt.Sprintf("infra.clusterName=%s", input.ClusterName),
"cloudControllerManager.logVerbosity=4",
},
StringValues: []string{
fmt.Sprintf("cloudControllerManager.clusterCIDR=%s", strings.Join(cidrBlocks, `\,`)),
"cloudControllerManager.federatedTokenPath=",
},
StringValues: []string{fmt.Sprintf("cloudControllerManager.clusterCIDR=%s", strings.Join(cidrBlocks, `\,`))},
}
// If testing a CI version of Kubernetes, use CCM and CNM images built from source.
if useCIArtifacts || usePRArtifacts {
Expand Down

0 comments on commit 1867a47

Please sign in to comment.