🚑 hotfix reconcile error

modified:   controllers/pod_controller.go; modified:   controllers/pod_controller_functions.go

Author: amitai-devops

controllers/pod_controller.go

@@ -44,9 +44,6 @@ func cacheKeyFunc(obj interface{}) (string, error) {
 var cacheStore = cache.NewTTLStore(cacheKeyFunc, 48*time.Hour)
 
 // Reconcile handles a reconciliation request for a Pod.
-// If the Pod has the podHasAnnotation annotation, then Reconcile
-// will make sure the podNameLabel label is present with the correct value.
-// If the annotation is absent, then Reconcile will make sure the label is too.
 func (r *PodReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
 	log := r.Log.WithValues("pod", req.NamespacedName)
 
@@ -78,7 +75,7 @@ func (r *PodReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.R
 
 		if err != nil {
 			log.Error(err, "failed to get stats from pod")
-			return ctrl.Result{}, err
+			return ctrl.Result{}, nil
 		}
 		PodUsageData := GeneratePodRequestsObjectFromRestData(data)
 

controllers/pod_controller_functions.go

@@ -41,10 +41,10 @@ func (r *PodReconciler) UpdateKubeObject(pod client.Object, ctx context.Context)
 		if apierrors.IsNotFound(err) {
 			// The Pod has been deleted since we read it.
 			// Requeue the Pod to try to reconciliate again.
-			return ctrl.Result{Requeue: false}, err
+			return ctrl.Result{}, err
 		}
 		log.Error(err, "unable to update pod")
-		return ctrl.Result{Requeue: false}, err
+		return ctrl.Result{}, err
 	}
 	return ctrl.Result{}, nil
 }