remove export log function

Author: Sarthak Agrawal

internal/controller/state/graph/backend_tls_policy.go

@@ -316,7 +316,7 @@ func addGatewaysForBackendTLSPolicies(
 						"Gateway not found in the graph", "policy", policyName, "ancestor", gatewayName)
 				}
 
-				LogAncestorLimitReached(logger, policyName, "BackendTLSPolicy", gatewayName)
+				logAncestorLimitReached(logger, policyName, "BackendTLSPolicy", gatewayName)
 				continue
 			}
 

internal/controller/state/graph/backend_tls_policy_test.go

@@ -800,7 +800,7 @@ func TestAddGatewaysForBackendTLSPoliciesAncestorLimit(t *testing.T) {
 	g.Expect(gateway2.Conditions).To(BeEmpty(), "Normal gateway should not have conditions")
 
 	// Verify logging function works - test the logging function directly
-	LogAncestorLimitReached(testLogger, "test/btp-full-ancestors", "BackendTLSPolicy", "test/gateway1")
+	logAncestorLimitReached(testLogger, "test/btp-full-ancestors", "BackendTLSPolicy", "test/gateway1")
 	logOutput := logBuf.String()
 
 	g.Expect(logOutput).To(ContainSubstring("Policy ancestor limit reached"))

internal/controller/state/graph/policies.go

@@ -200,7 +200,7 @@ func attachPolicyToService(
 			policyKind := getPolicyKind(policy.Source)
 
 			gw.Conditions = addPolicyAncestorLimitCondition(gw.Conditions, policyName, policyKind)
-			LogAncestorLimitReached(logger, policyName, policyKind, gwNsName.String())
+			logAncestorLimitReached(logger, policyName, policyKind, gwNsName.String())
 
 			// Mark this gateway as invalid for the policy due to ancestor limits
 			policy.InvalidForGateways[gwNsName] = struct{}{}
@@ -248,7 +248,7 @@ func attachPolicyToRoute(
 		routeName := getAncestorName(ancestorRef)
 
 		route.Conditions = addPolicyAncestorLimitCondition(route.Conditions, policyName, policyKind)
-		LogAncestorLimitReached(logger, policyName, policyKind, routeName)
+		logAncestorLimitReached(logger, policyName, policyKind, routeName)
 
 		return
 	}
@@ -326,7 +326,7 @@ func attachPolicyToGateway(
 			// Log in the controller log.
 			logger.Info("Gateway target not found and ancestors slice is full.", "policy", policyName, "ancestor", ancestorName)
 		}
-		LogAncestorLimitReached(logger, policyName, policyKind, ancestorName)
+		logAncestorLimitReached(logger, policyName, policyKind, ancestorName)
 
 		policy.InvalidForGateways[ref.Nsname] = struct{}{}
 		return

internal/controller/state/graph/policy_ancestor.go

@@ -13,8 +13,8 @@ import (
 
 const maxAncestors = 16
 
-// LogAncestorLimitReached logs when a policy ancestor limit is reached.
-func LogAncestorLimitReached(logger logr.Logger, policyName, policyKind, ancestorName string) {
+// logAncestorLimitReached logs when a policy ancestor limit is reached.
+func logAncestorLimitReached(logger logr.Logger, policyName, policyKind, ancestorName string) {
 	logger.Info("Policy ancestor limit reached", "policy", policyName, "policyKind", policyKind, "ancestor", ancestorName)
 }