To add reconciliation time stamps

cyclops-ctrl/api/v1alpha1/module_types.go

@@ -66,6 +66,7 @@ type ReconciliationStatus struct {
 	Reason string `json:"reason,omitempty"`
 	// +kubebuilder:validation:Optional
 	Errors []string `json:"errors"`
+	FinishedAt string `json:"finishedAt"`
 }
 
 type GroupVersionResource struct {
@@ -94,6 +95,7 @@ type HistoryEntry struct {
 	Generation  int64                `json:"generation"`
 	TemplateRef HistoryTemplateRef   `json:"template"`
 	Values      apiextensionsv1.JSON `json:"values"`
+	FinishedAt string `json:"finishedAt"`
 }
 
 //+kubebuilder:object:root=true

cyclops-ctrl/config/crd/bases/cyclops-ui.com_modules.yaml

@@ -29,6 +29,8 @@ spec:
           history:
             items:
               properties:
+                finishedAt:
+                  type: string
                 generation:
                   format: int64
                   type: integer
@@ -48,6 +50,7 @@ spec:
                 values:
                   x-kubernetes-preserve-unknown-fields: true
               required:
+              - finishedAt
               - generation
               - template
               - values
@@ -113,6 +116,8 @@ spec:
                     items:
                       type: string
                     type: array
+                  finishedAt:
+                    type: string
                   reason:
                     type: string
                   status:
@@ -122,6 +127,8 @@ spec:
                     - succeeded
                     - failed
                     type: string
+                required:
+                - finishedAt
                 type: object
               templateResolvedVersion:
                 type: string

cyclops-ctrl/internal/modulecontroller/module_controller.go

@@ -406,36 +406,38 @@ func (r *ModuleReconciler) mergeChildrenGVRs(existing, current []cyclopsv1alpha1
 }
 
 func (r *ModuleReconciler) setStatus(
-	ctx context.Context,
-	module cyclopsv1alpha1.Module,
-	namespacedName types.NamespacedName,
-	status cyclopsv1alpha1.ReconciliationStatusState,
-	templateResolvedVersion string,
-	reason string,
-	installErrors []string,
-	childrenResources []cyclopsv1alpha1.GroupVersionResource,
-	iconURL string,
+    ctx context.Context,
+    module cyclopsv1alpha1.Module,
+    namespacedName types.NamespacedName,
+    status cyclopsv1alpha1.ReconciliationStatusState,
+    templateResolvedVersion string,
+    reason string,
+    installErrors []string,
+    childrenResources []cyclopsv1alpha1.GroupVersionResource,
+    iconURL string,
 ) error {
-	trv := module.Status.TemplateResolvedVersion
-	if len(trv) == 0 {
-		trv = templateResolvedVersion
-	}
-
-	module.Status = cyclopsv1alpha1.ModuleStatus{
-		ReconciliationStatus: cyclopsv1alpha1.ReconciliationStatus{
-			Status: status,
-			Reason: reason,
-			Errors: installErrors,
-		},
-		ManagedGVRs:             r.mergeChildrenGVRs(module.Status.ManagedGVRs, childrenResources),
-		TemplateResolvedVersion: templateResolvedVersion,
-		IconURL:                 iconURL,
-	}
-
-	if err := r.Status().Update(ctx, &module); err != nil {
-		r.logger.Error(err, "error updating module status", "namespaced name", namespacedName)
-		return err
-	}
-
-	return nil
+    trv := module.Status.TemplateResolvedVersion
+    if len(trv) == 0 {
+        trv = templateResolvedVersion
+    }
+
+    module.Status = cyclopsv1alpha1.ModuleStatus{
+        ReconciliationStatus: cyclopsv1alpha1.ReconciliationStatus{
+            Status:       status,
+            Reason:       reason,
+            Errors:       installErrors,
+            FinishedAt:   time.Now().Format(time.RFC3339), // Convert time to string format
+        },
+        ManagedGVRs:             r.mergeChildrenGVRs(module.Status.ManagedGVRs, childrenResources),
+        TemplateResolvedVersion: templateResolvedVersion,
+        IconURL:                 iconURL,
+    }
+
+    if err := r.Status().Update(ctx, &module); err != nil {
+        r.logger.Error(err, "error updating module status", "namespaced name", namespacedName)
+        return err
+    }
+
+    return nil
 }
+