fix(controller): re-enable some metrics disabled by mistake

Author: plaffitt

internal/controller/collector.go

@@ -17,15 +17,15 @@ const subsystem = "controller"
 var ProbeAddr = ""
 
 var (
-	imagePutInCache = prometheus.NewCounter(
+	ImagePutInCache = prometheus.NewCounter(
 		prometheus.CounterOpts{
 			Namespace: kuikMetrics.Namespace,
 			Subsystem: subsystem,
 			Name:      "image_put_in_cache_total",
 			Help:      "Number of images put in cache successfully",
 		},
 	)
-	imageRemovedFromCache = prometheus.NewCounter(
+	ImageRemovedFromCache = prometheus.NewCounter(
 		prometheus.CounterOpts{
 			Namespace: kuikMetrics.Namespace,
 			Subsystem: subsystem,
@@ -59,8 +59,8 @@ var (
 func RegisterMetrics(client client.Client) {
 	// Register custom metrics with the global prometheus registry
 	metrics.Registry.MustRegister(
-		imagePutInCache,
-		imageRemovedFromCache,
+		ImagePutInCache,
+		ImageRemovedFromCache,
 		kuikMetrics.NewInfo(subsystem),
 		isLeader,
 		up,

internal/controller/kuik/cachedimage_controller.go

@@ -139,7 +139,7 @@ func (r *CachedImageReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 				return ctrl.Result{}, err
 			}
 			r.Recorder.Eventf(&cachedImage, "Normal", "CleanedUp", "Image %s successfully removed from cache", cachedImage.Spec.SourceImage)
-			// imageRemovedFromCache.Inc()
+			kuikController.ImageRemovedFromCache.Inc()
 
 			log.Info("removing finalizer")
 			controllerutil.RemoveFinalizer(&cachedImage, cachedImageFinalizerName)
@@ -251,7 +251,7 @@ func (r *CachedImageReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 		} else {
 			log.Info("image cached")
 			r.Recorder.Eventf(&cachedImage, "Normal", "Cached", "Successfully cached image %s", cachedImage.Spec.SourceImage)
-			// imagePutInCache.Inc()
+			kuikController.ImagePutInCache.Inc()
 		}
 	} else {
 		log.Info("image already present in cache, ignoring")