update status update logic patch from Andre

Signed-off-by: Mohamed Mahmoud <mmahmoud@redhat.com>
bpfman · May 28, 2024 · 7f71813 · 7f71813
1 parent 6e8d572
commit 7f71813
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/controllers/bpfman-agent/common.go b/controllers/bpfman-agent/common.go
@@ -478,7 +478,7 @@ func (r *ReconcilerCommon) updateStatus(ctx context.Context, bpfProgram *bpfmani
 	meta.SetStatusCondition(&bpfProgram.Status.Conditions, cond.Condition())
 
 	r.Logger.V(1).Info("Updating bpfProgram condition", "bpfProgram", bpfProgram.Name, "condition", cond.Condition().Type)
-	if err := r.Status().Update(ctx, bpfProgram); err != nil {
+	if err := r.Update(ctx, bpfProgram); err != nil {
 		r.Logger.Error(err, "failed to set bpfProgram object status")
 	}
 

diff --git a/controllers/bpfman-operator/common.go b/controllers/bpfman-operator/common.go
@@ -220,7 +220,7 @@ func (r *ReconcilerCommon) updateCondition(ctx context.Context, obj client.Objec
 
 	meta.SetStatusCondition(conditions, cond.Condition(message))
 
-	if err := r.Status().Update(ctx, obj); err != nil {
+	if err := r.Update(ctx, obj); err != nil {
 		r.Logger.V(1).Info("failed to set *Program object status...requeuing")
 		return ctrl.Result{Requeue: true, RequeueAfter: retryDurationOperator}, nil
 	}