Record the last pushed SHA1 and the time it was pushed

This adds fields to the ImageUpdateAutomation status for recording the
commit last pushed; handy to see when you are expecting a change.

It also adapts the "steady state" message of the ready condition to
mention the last commit, in case that's where people are looking.

Signed-off-by: Michael Bridgen <[email protected]>

Author: squaremo

api/v1alpha1/imageupdateautomation_types.go

@@ -94,6 +94,13 @@ type ImageUpdateAutomationStatus struct {
 	// made).
 	// +optional
 	LastAutomationRunTime *metav1.Time `json:"lastAutomationRunTime,omitempty"`
+	// LastPushCommit records the SHA1 of the last commit made by the
+	// controller, for this automation object
+	// +optional
+	LastPushCommit string `json:"lastPushCommit,omitempty"`
+	// LastPushTime records the time of the last pushed change.
+	// +optional
+	LastPushTime *metav1.Time `json:"lastPushTime,omitempty"`
 	// +optional
 	ObservedGeneration int64 `json:"observedGeneration,omitempty"`
 	// +optional

api/v1alpha1/zz_generated.deepcopy.go

@@ -187,6 +187,14 @@ spec:
                 description: LastHandledReconcileAt holds the value of the most recent
                   reconcile request value, so a change can be detected.
                 type: string
+              lastPushCommit:
+                description: LastPushCommit records the SHA1 of the last commit made
+                  by the controller, for this automation object
+                type: string
+              lastPushTime:
+                description: LastPushTime records the time of the last pushed change.
+                format: date-time
+                type: string
               observedGeneration:
                 format: int64
                 type: integer

config/crd/bases/image.toolkit.fluxcd.io_imageupdateautomations.yaml

@@ -199,12 +199,17 @@ func (r *ImageUpdateAutomationReconciler) Reconcile(req ctrl.Request) (ctrl.Resu
 			r.event(auto, events.EventSeverityInfo, "no updates made")
 			log.V(debug).Info("no changes made in working directory; no commit")
 			statusMessage = "no updates made"
+			if lastCommit, lastTime := auto.Status.LastPushCommit, auto.Status.LastPushTime; lastCommit != "" {
+				statusMessage = fmt.Sprintf("%s; last commit %s at %s", statusMessage, lastCommit, lastTime.Format(time.RFC3339))
+			}
 		} else {
 			return failWithError(err)
 		}
 	} else {
 		r.event(auto, events.EventSeverityInfo, "committed and pushed change "+rev)
 		log.Info("pushed commit to origin", "revision", rev)
+		auto.Status.LastPushCommit = rev
+		auto.Status.LastPushTime = &metav1.Time{Time: now}
 		statusMessage = "committed and pushed " + rev
 	}
 

controllers/imageupdateautomation_controller.go

@@ -241,6 +241,11 @@ var _ = Describe("ImageUpdateAutomation", func() {
 				Expect(err).ToNot(HaveOccurred())
 				Expect(commit.Message).To(Equal(commitMessage))
 
+				var newObj imagev1.ImageUpdateAutomation
+				Expect(k8sClient.Get(context.Background(), updateKey, &newObj)).To(Succeed())
+				Expect(newObj.Status.LastPushCommit).To(Equal(head.Hash().String()))
+				Expect(newObj.Status.LastPushTime).ToNot(BeNil())
+
 				compareRepoWithExpected(repoURL, branch, "testdata/appconfig-setters-expected", func(tmp string) {
 					replaceMarker(tmp, policyKey)
 				})