Merge pull request #54 from annablender/aks-oms-agent-fix

Add checks for misconfigured oms agent
rancher · Feb 25, 2022 · 7eec500 · 7eec500
2 parents 20612fc + 55fa4bb
commit 7eec500
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/controller/aks-cluster-config-handler.go b/controller/aks-cluster-config-handler.go
@@ -665,6 +665,10 @@ func BuildUpstreamClusterState(ctx context.Context, secretsCache wranglerv1.Secr
 	// set addon monitoring profile
 	if addonProfile["omsAgent"] != nil {
 		upstreamSpec.Monitoring = addonProfile["omsAgent"].Enabled
+
+		if len(addonProfile["omsAgent"].Config) == 0 {
+			return nil, fmt.Errorf("cannot set OMS Agent configuration retrieved from Azure")
+		}
 		logAnalyticsWorkspaceResourceID := addonProfile["omsAgent"].Config["logAnalyticsWorkspaceResourceID"]
 
 		logAnalyticsWorkspaceGroup := matchWorkspaceGroup.FindStringSubmatch(to.String(logAnalyticsWorkspaceResourceID))[1]