Skip to content

Commit

Permalink
test: "useManagedIdentity": false for VMSS masters (Azure#3981)
Browse files Browse the repository at this point in the history
  • Loading branch information
jackfrancis authored Oct 28, 2020
1 parent abd9bec commit 7288c05
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
4 changes: 3 additions & 1 deletion test/e2e/engine/template.go
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,9 @@ func Build(cfg *config.Config, masterSubnetID string, agentSubnetIDs []string, i
prop.OrchestratorProfile.KubernetesConfig.UserAssignedID = config.MSIUserAssignedID
}

prop.OrchestratorProfile.KubernetesConfig.UseManagedIdentity = to.BoolPtr(config.UseManagedIdentity)
if prop.OrchestratorProfile.KubernetesConfig.UseManagedIdentity == nil {
prop.OrchestratorProfile.KubernetesConfig.UseManagedIdentity = to.BoolPtr(config.UseManagedIdentity)
}

if config.ClientID != "" && config.ClientSecret != "" && !(prop.OrchestratorProfile.KubernetesConfig != nil && to.Bool(prop.OrchestratorProfile.KubernetesConfig.UseManagedIdentity)) {
if !prop.IsAzureStackCloud() {
Expand Down
5 changes: 4 additions & 1 deletion test/e2e/test_cluster_configs/vmss_master.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,10 @@
"apiVersion": "vlabs",
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes"
"orchestratorType": "Kubernetes",
"kubernetesConfig": {
"useManagedIdentity": false
}
},
"masterProfile": {
"count": 3,
Expand Down

0 comments on commit 7288c05

Please sign in to comment.