diff --git a/federation/pkg/federation-controller/service/dns.go b/federation/pkg/federation-controller/service/dns.go
index 496acfb8e48..5dd4a0d2964 100644
--- a/federation/pkg/federation-controller/service/dns.go
+++ b/federation/pkg/federation-controller/service/dns.go
@@ -283,12 +283,12 @@ func (s *ServiceController) ensureDnsRecords(clusterName string, cachedService *
 	serviceName := cachedService.lastState.Name
 	namespaceName := cachedService.lastState.Namespace
 	zoneNames, regionName, err := s.getClusterZoneNames(clusterName)
-	if zoneNames == nil {
-		return fmt.Errorf("fail to get cluster zone names")
-	}
 	if err != nil {
 		return err
 	}
+	if zoneNames == nil {
+		return fmt.Errorf("failed to get cluster zone names")
+	}
 	dnsZoneName, err := s.getFederationDNSZoneName()
 	if err != nil {
 		return err