Remove duplicate log values and remove the check for processes under …

…maintenance

fdbclient/admin_client.go

@@ -95,13 +95,12 @@ type cliAdminClient struct {
 }
 
 // NewCliAdminClient generates an Admin client for a cluster
-func NewCliAdminClient(cluster *fdbv1beta2.FoundationDBCluster, _ client.Client, log logr.Logger) (fdbadminclient.AdminClient, error) {
+func NewCliAdminClient(cluster *fdbv1beta2.FoundationDBCluster, _ client.Client, logger logr.Logger) (fdbadminclient.AdminClient, error) {
 	clusterFile, err := createClusterFile(cluster)
 	if err != nil {
 		return nil, err
 	}
 
-	logger := log.WithValues("namespace", cluster.Namespace, "cluster", cluster.Name)
 	return &cliAdminClient{
 		Cluster:         cluster,
 		clusterFilePath: clusterFile,

internal/locality/locality.go

@@ -377,8 +377,17 @@ func CheckCoordinatorValidity(logger logr.Logger, cluster *fdbv1beta2.Foundation
 			coordinatorAddress = dnsAddress.String()
 		}
 
-		if coordinatorAddress != "" && !process.Excluded && !process.UnderMaintenance {
-			coordinatorStatus[coordinatorAddress] = true
+		if coordinatorAddress != "" {
+			if process.Excluded {
+				pLogger.Info("Coordinator process is excluded, marking it as unhealthy", "class", process.ProcessClass, "address", coordinatorAddress)
+			} else {
+				// The process is not excluded and has an address, so we assume that the coordinator is healthy.
+				coordinatorStatus[coordinatorAddress] = true
+			}
+
+			if process.UnderMaintenance {
+				pLogger.Info("Coordinator process is under maintenance", "class", process.ProcessClass, "address", coordinatorAddress)
+			}
 		}
 
 		if coordinatorAddress != "" {