[Fix] Operator: DNS handling fixed

Recent changes/refactoring caused an issue due to misplaced slice append
 causing only 1 subdomain entry to be handled by domain resources.
This change fixes the issue.

Author: Pavan-SAP

internal/controller/dns-manager.go

@@ -171,6 +171,11 @@ func getDnsInfo[T v1alpha1.DomainEntity](c *Controller, dom T) (resolvedDNSInfo
 	tpl := template.New("dnsTemplate").Funcs(sprig.FuncMap())
 
 	resolvedDNSInfo = []*dnsInfo{}
+	checkAndAppendDNSInfo := func(dnsInfo *dnsInfo) {
+		if dnsInfo != nil {
+			resolvedDNSInfo = append(resolvedDNSInfo, dnsInfo)
+		}
+	}
 
 	for _, dnsTemplate := range dnsTemplates {
 		var parsedDnsInfo *dnsInfo
@@ -180,6 +185,8 @@ func getDnsInfo[T v1alpha1.DomainEntity](c *Controller, dom T) (resolvedDNSInfo
 			if err != nil {
 				return nil, err
 			}
+			checkAndAppendDNSInfo(parsedDnsInfo)
+			continue
 		}
 		for subDomain, appId := range subdomainInfo {
 			domVars["subDomain"] = subDomain
@@ -188,9 +195,7 @@ func getDnsInfo[T v1alpha1.DomainEntity](c *Controller, dom T) (resolvedDNSInfo
 				return nil, err
 			}
 			parsedDnsInfo.appId = appId
-		}
-		if parsedDnsInfo != nil {
-			resolvedDNSInfo = append(resolvedDNSInfo, parsedDnsInfo)
+			checkAndAppendDNSInfo(parsedDnsInfo)
 		}
 	}
 

internal/controller/reconcile_test.go

@@ -45,7 +45,6 @@ const (
 	globalAccountId    = "global-id-test"
 	primaryDomain      = "app.sme.sap.com"
 	secondaryDomain    = "sec.sme.sap.com"
-	dnsTarget          = "public-ingress.some.cluster.sap"
 	defaultVersion     = "0.0.1"
 )