Skip to content

Commit

Permalink
nodereload controller fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Bohdan Siryk authored and Bohdan Siryk committed Nov 8, 2023
1 parent 1a20736 commit 763415d
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions controllers/clusterresources/nodereload_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -165,8 +165,7 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
err = r.Status().Patch(ctx, nrs, patch)
if err != nil {
l.Error(err, "Failed to patch current operation status",
"node ID", nrs.Status.NodeInProgress.ID,
"currentOperationStatus", nodeReloadStatus,
"status", nodeReloadStatus,
)
r.EventRecorder.Eventf(nrs, models.Warning, models.FetchFailed,
"Failed to patch current operation status. Reason: %w", err,
Expand All @@ -187,7 +186,7 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
"status", nrs.Status,
)
r.EventRecorder.Eventf(nrs, models.Normal, models.UpdatedEvent,
"Node %s has been successfully reloaded", nrs.Status,
"Node %s has been successfully reloaded", nodeReloadStatus.NodeID,
)

patch = nrs.NewPatch()
Expand Down

0 comments on commit 763415d

Please sign in to comment.