issue-594-expose-services-deferred-status, deferred status for cluste…

…rs was handled

controllers/clusterresources/postgresqluser_controller.go

@@ -290,7 +290,6 @@ func (r *PostgreSQLUserReconciler) createUser(
 		return fmt.Errorf("cannot list nodes, err: %w", err)
 	}
 
-	// TODO: Handle scenario if there are no nodes with external IP, check private/public cluster
 	for _, node := range nodeList.Items {
 		for _, nodeAddress := range node.Status.Addresses {
 			if nodeAddress.Type == k8sCore.NodeExternalIP {

controllers/clusters/postgresql_controller_test.go

@@ -107,7 +107,7 @@ var _ = Describe("PostgreSQL Controller", func() {
 		})
 	})
 
-	When("Deleting the Kafka Connect resource by avoiding operator", func() {
+	When("Deleting the PostgreSQL resource by avoiding operator", func() {
 		It("should try to get the cluster details and receive StatusNotFound", func() {
 			postgresqlManifest2 := postgresqlManifest.DeepCopy()
 			postgresqlManifest2.Name += "-test-external-delete"

pkg/exposeservice/expose_service.go

@@ -37,6 +37,21 @@ func Create(
 	nodes []*v1beta1.Node,
 	targetPort int32,
 ) error {
+	addresses := []v1.EndpointAddress{}
+	for _, node := range nodes {
+		if node.PublicAddress == "" {
+			continue
+		}
+
+		addresses = append(addresses, v1.EndpointAddress{
+			IP: node.PublicAddress,
+		})
+	}
+
+	if len(addresses) == 0 {
+		return nil
+	}
+
 	svcName := fmt.Sprintf(models.ExposeServiceNameTemplate, clusterName)
 	service := &v1.Service{
 		ObjectMeta: metav1.ObjectMeta{
@@ -58,17 +73,6 @@ func Create(
 		},
 	}
 
-	addresses := []v1.EndpointAddress{}
-	for _, node := range nodes {
-		if node.PublicAddress == "" {
-			continue
-		}
-
-		addresses = append(addresses, v1.EndpointAddress{
-			IP: node.PublicAddress,
-		})
-	}
-
 	endpoints := &v1.Endpoints{
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      svcName,