Add ready status field

Author: MathieuCesbron

api/v1/metabase_types.go

@@ -97,6 +97,15 @@ type VolumeSpec struct {
 type MetabaseStatus struct {
 	// INSERT ADDITIONAL STATUS FIELD - define observed state of cluster
 	// Important: Run "make" to regenerate code after modifying this file
+
+	// Instance ready to accept connections.
+	// +kubebuilder:validation:default=false
+	// +kubebuilder:validation:Required
+	Ready bool `json:"ready"`
+
+	// Host to connect to the metabase.
+	// +kubebuilder:validation:Optional
+	Host *string `json:"host"`
 }
 
 //+kubebuilder:object:root=true
@@ -108,7 +117,8 @@ type Metabase struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// +kubebuilder:validation:Required
-	Spec   MetabaseSpec   `json:"spec"`
+	Spec MetabaseSpec `json:"spec"`
+
 	Status MetabaseStatus `json:"status,omitempty"`
 }
 

api/v1/zz_generated.deepcopy.go

@@ -2,6 +2,6 @@ apiVersion: v2
 name: metabase-operator
 description: Helm chart to deploy [unagex-metabase-operator](https://github.com/unagex/metabase-operator)
 type: application
-version: 0.0.5
-appVersion: 0.0.5
+version: 0.0.6
+appVersion: 0.0.6
 home: https://github.com/unagex/metabase-operator

charts/operator/Chart.yaml

@@ -204,6 +204,15 @@ spec:
             type: object
           status:
             description: MetabaseStatus defines the observed state of Metabase
+            properties:
+              host:
+                description: Host to connect to the database.
+                type: string
+              ready:
+                description: Instance ready to accept connections.
+                type: boolean
+            required:
+            - ready
             type: object
         required:
         - spec

charts/operator/templates/crds/unagex.com_metabases.yaml

@@ -204,6 +204,15 @@ spec:
             type: object
           status:
             description: MetabaseStatus defines the observed state of Metabase
+            properties:
+              host:
+                description: Host to connect to the database.
+                type: string
+              ready:
+                description: Instance ready to accept connections.
+                type: boolean
+            required:
+            - ready
             type: object
         required:
         - spec

config/crd/bases/unagex.com_metabases.yaml

@@ -42,6 +42,25 @@ func (r *MetabaseReconciler) ManageMetabase(ctx context.Context, metabase *unage
 		return fmt.Errorf("error getting deployment: %w", err)
 	}
 
+	// update status if not sync with deployment
+	// metabase dep always have one pod
+	ready := dep.Status.ReadyReplicas == 1
+	if ready != metabase.Status.Ready {
+		metabase.Status.Ready = ready
+		metabase.Status.Host = nil
+		if ready {
+			host := fmt.Sprintf("%s-http.%s.svc.cluster.local:3000", metabase.Name, metabase.Namespace)
+			metabase.Status.Host = &host
+		}
+
+		err = r.Status().Update(ctx, metabase)
+		if err != nil {
+			return fmt.Errorf("error updating metabase field status.ready: %w", err)
+		}
+
+		r.Log.Info(fmt.Sprintf("updated metabase field status.ready to %t", ready))
+	}
+
 	return nil
 }