BOP-85/87 Add events & status to Addons & Manifests

api/v1alpha1/addon_types.go

@@ -33,14 +33,44 @@ type ManifestInfo struct {
 	URL string `json:"url"`
 }
 
+// StatusConditionType is a type of condition that may apply to a particular component.
+type StatusConditionType string
+
+const (
+	// ComponentAvailable indicates that the component is healthy.
+	ComponentAvailable StatusConditionType = "Available"
+
+	// ComponentProgressing means that the component is in the process of being installed or upgraded.
+	ComponentProgressing StatusConditionType = "Progressing"
+
+	// ComponentDegraded means the component is not operating as desired and user action is required.
+	ComponentDegraded StatusConditionType = "Degraded"
+
+	// ComponentReady indicates that the component is healthy and ready.it is identical to Available and used in Status conditions for CRs.
+	ComponentReady StatusConditionType = "Ready"
+)
+
 // AddonStatus defines the observed state of Addon
 type AddonStatus struct {
 	// INSERT ADDITIONAL STATUS FIELD - define observed state of cluster
 	// Important: Run "make" to regenerate code after modifying this file
+
+	// The type of condition. May be Available, Progressing, or Degraded.
+	Type StatusConditionType `json:"type"`
+
+	// The timestamp representing the start time for the current status.
+	LastTransitionTime metav1.Time `json:"lastTransitionTime"`
+
+	// A brief reason explaining the condition.
+	Reason string `json:"reason,omitempty"`
+
+	// Optionally, a detailed message providing additional context.
+	Message string `json:"message,omitempty"`
 }
 
 //+kubebuilder:object:root=true
 //+kubebuilder:subresource:status
+//+kubebuilder:printcolumn:name="Status",type="string",JSONPath=".status.type",description="Whether the component is running and stable."
 
 // Addon is the Schema for the addons API
 type Addon struct {

api/v1alpha1/manifest_types.go

@@ -21,6 +21,18 @@ type ManifestSpec struct {
 type ManifestStatus struct {
 	// INSERT ADDITIONAL STATUS FIELD - define observed state of cluster
 	// Important: Run "make" to regenerate code after modifying this file
+
+	// The type of condition. May be Available, Progressing, or Degraded.
+	Type StatusConditionType `json:"type"`
+
+	// The timestamp representing the start time for the current status.
+	LastTransitionTime metav1.Time `json:"lastTransitionTime"`
+
+	// A brief reason explaining the condition.
+	Reason string `json:"reason,omitempty"`
+
+	// Optionally, a detailed message providing additional context.
+	Message string `json:"message,omitempty"`
 }
 
 // ManifestObject consists of the fields required to update/delete an object
@@ -32,6 +44,7 @@ type ManifestObject struct {
 
 //+kubebuilder:object:root=true
 //+kubebuilder:subresource:status
+//+kubebuilder:printcolumn:name="Status",type="string",JSONPath=".status.type",description="Whether the component is running and stable."
 
 // Manifest is the Schema for the manifests API
 type Manifest struct {

config/crd/bases/boundless.mirantis.com_addons.yaml

@@ -15,7 +15,12 @@ spec:
     singular: addon
   scope: Namespaced
   versions:
-  - name: v1alpha1
+  - additionalPrinterColumns:
+    - description: Whether the component is running and stable.
+      jsonPath: .status.type
+      name: Status
+      type: string
+    name: v1alpha1
     schema:
       openAPIV3Schema:
         description: Addon is the Schema for the addons API
@@ -79,6 +84,25 @@ spec:
             type: object
           status:
             description: AddonStatus defines the observed state of Addon
+            properties:
+              lastTransitionTime:
+                description: The timestamp representing the start time for the current
+                  status.
+                format: date-time
+                type: string
+              message:
+                description: Optionally, a detailed message providing additional context.
+                type: string
+              reason:
+                description: A brief reason explaining the condition.
+                type: string
+              type:
+                description: The type of condition. May be Available, Progressing,
+                  or Degraded.
+                type: string
+            required:
+            - lastTransitionTime
+            - type
             type: object
         type: object
     served: true

config/crd/bases/boundless.mirantis.com_manifests.yaml

@@ -15,7 +15,12 @@ spec:
     singular: manifest
   scope: Namespaced
   versions:
-  - name: v1alpha1
+  - additionalPrinterColumns:
+    - description: Whether the component is running and stable.
+      jsonPath: .status.type
+      name: Status
+      type: string
+    name: v1alpha1
     schema:
       openAPIV3Schema:
         description: Manifest is the Schema for the manifests API
@@ -62,6 +67,25 @@ spec:
             type: object
           status:
             description: ManifestStatus defines the observed state of Manifest
+            properties:
+              lastTransitionTime:
+                description: The timestamp representing the start time for the current
+                  status.
+                format: date-time
+                type: string
+              message:
+                description: Optionally, a detailed message providing additional context.
+                type: string
+              reason:
+                description: A brief reason explaining the condition.
+                type: string
+              type:
+                description: The type of condition. May be Available, Progressing,
+                  or Degraded.
+                type: string
+            required:
+            - lastTransitionTime
+            - type
             type: object
         type: object
     served: true

config/rbac/role.yaml

@@ -5,6 +5,13 @@ metadata:
   creationTimestamp: null
   name: manager-role
 rules:
+- apiGroups:
+  - ""
+  resources:
+  - events
+  verbs:
+  - create
+  - patch
 - apiGroups:
   - boundless.mirantis.com
   resources:

controllers/addon_controller.go

@@ -3,9 +3,13 @@ package controllers
 import (
 	"context"
 	"fmt"
-
+	"github.com/go-logr/logr"
+	"github.com/mirantis/boundless-operator/pkg/event"
 	"k8s.io/apimachinery/pkg/api/errors"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
+	"k8s.io/apimachinery/pkg/types"
+	"k8s.io/client-go/tools/record"
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 	"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
@@ -27,12 +31,14 @@ const (
 // AddonReconciler reconciles a Addon object
 type AddonReconciler struct {
 	client.Client
-	Scheme *runtime.Scheme
+	Scheme   *runtime.Scheme
+	Recorder record.EventRecorder
 }
 
 //+kubebuilder:rbac:groups=boundless.mirantis.com,resources=addons,verbs=get;list;watch;create;update;patch;delete
 //+kubebuilder:rbac:groups=boundless.mirantis.com,resources=addons/status,verbs=get;update;patch
 //+kubebuilder:rbac:groups=boundless.mirantis.com,resources=addons/finalizers,verbs=update
+//+kubebuilder:rbac:groups="",resources=events,verbs=create;patch
 
 // Reconcile is part of the main kubernetes reconciliation loop which aims to
 // move the current state of the cluster closer to the desired state.
@@ -63,6 +69,11 @@ func (r *AddonReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
 		return ctrl.Result{}, err
 	}
 
+	// only update the status to progressing if its not already set to not trigger infinite reconciliations
+	if instance.Status.Type == "" {
+		r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentProgressing, "Creating Addon")
+	}
+
 	switch instance.Spec.Kind {
 	case kindChart:
 		chart := helm.Chart{
@@ -94,6 +105,7 @@ func (r *AddonReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
 				if err := hc.DeleteHelmChart(chart, instance.Spec.Namespace); err != nil {
 					// if fail to delete the helm chart here, return with error
 					// so that it can be retried
+					r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeWarning, event.ReasonFailedDelete, "Failed to Delete Chart Addon %s/%s: %s", instance.Spec.Namespace, instance.Name, err)
 					return ctrl.Result{}, err
 				}
 
@@ -108,11 +120,15 @@ func (r *AddonReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
 			return ctrl.Result{}, nil
 		}
 
-		logger.Info("Creating Addon HelmChart resource", "Name", chart.Name, "Version", chart.Version)
 		if err := hc.CreateHelmChart(chart, instance.Spec.Namespace); err != nil {
 			logger.Error(err, "failed to install addon", "Name", chart.Name, "Version", chart.Version)
+			r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeWarning, event.ReasonFailedCreate, "Failed to Create Chart Addon %s/%s : %s", instance.Spec.Namespace, instance.Name, err)
+			r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentDegraded, "Failed to Create HelmChart")
 			return ctrl.Result{Requeue: true}, err
 		}
+		r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeNormal, event.ReasonSuccessfulCreate, "Created Chart Addon %s/%s", instance.Spec.Namespace, instance.Name)
+		r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentAvailable, "Chart Addon Created")
+
 	case kindManifest:
 		mc := manifest.NewManifestController(r.Client, logger)
 
@@ -133,6 +149,8 @@ func (r *AddonReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
 				if err := mc.DeleteManifest(BoundlessNamespace, instance.Spec.Name, instance.Spec.Manifest.URL); err != nil {
 					// if fail to delete the manifest here, return with error
 					// so that it can be retried
+					r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeWarning, event.ReasonFailedDelete, "Failed to Delete Manifest Addon %s/%s : %s", instance.Spec.Namespace, instance.Name, err)
+					r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentDegraded, "Failed to Cleanup Manifest")
 					return ctrl.Result{}, err
 				}
 
@@ -150,9 +168,14 @@ func (r *AddonReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl
 		err = mc.CreateManifest(BoundlessNamespace, instance.Spec.Name, instance.Spec.Manifest.URL)
 		if err != nil {
 			logger.Error(err, "failed to install addon via manifest", "URL", instance.Spec.Manifest.URL)
+			r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeWarning, event.ReasonFailedCreate, "Failed to Create Manifest Addon %s/%s : %s", instance.Spec.Namespace, instance.Name, err)
+			r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentDegraded, "Failed to Create Manifest")
 			return ctrl.Result{Requeue: true}, err
 		}
 
+		r.Recorder.AnnotatedEventf(instance, map[string]string{event.AddonAnnotationKey: instance.Name}, event.TypeNormal, event.ReasonSuccessfulCreate, "Created Manifest Addon %s/%s", instance.Spec.Namespace, instance.Name)
+		r.updateStatus(ctx, logger, req.NamespacedName, boundlessv1alpha1.ComponentAvailable, "Manifest Addon Created")
+
 	default:
 		logger.Info("Unknown AddOn kind", "Kind", instance.Spec.Kind)
 		return ctrl.Result{Requeue: false}, fmt.Errorf("Unknown AddOn Kind: %w", err)
@@ -168,3 +191,30 @@ func (r *AddonReconciler) SetupWithManager(mgr ctrl.Manager) error {
 		For(&boundlessv1alpha1.Addon{}).
 		Complete(r)
 }
+
+func (r *AddonReconciler) updateStatus(ctx context.Context, logger logr.Logger, namespacedName types.NamespacedName, conditionTypeToApply boundlessv1alpha1.StatusConditionType, reasonToApply string, messageToApply ...string) error {
+	addon := &boundlessv1alpha1.Addon{}
+	err := r.Get(ctx, namespacedName, addon)
+	if err != nil {
+		logger.Error(err, "Failed to get addon to update status")
+		return err
+	}
+
+	if addon.Status.Type == conditionTypeToApply && addon.Status.Reason == reasonToApply {
+		// avoid infinite reconciliation loops
+		logger.Info("No updates to status needed")
+		return nil
+	}
+
+	logger.Info("Update status for addon", "Name", addon.Name)
+
+	patch := client.MergeFrom(addon.DeepCopy())
+	addon.Status.Type = conditionTypeToApply
+	addon.Status.Reason = reasonToApply
+	if len(messageToApply) > 0 {
+		addon.Status.Message = messageToApply[0]
+	}
+	addon.Status.LastTransitionTime = metav1.Now()
+
+	return r.Status().Patch(ctx, addon, patch)
+}