fix: Fixing complaint from golancilint for SQA (#79)

intel · Aug 4, 2023 · ef33d59 · ef33d59
1 parent 35aecb0
commit ef33d59
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/controllers/powerprofile_controller.go b/controllers/powerprofile_controller.go
@@ -106,7 +106,11 @@ func (r *PowerProfileReconciler) Reconcile(c context.Context, req ctrl.Request)
 			// everything has already been removed. Finally, we remove the Extended Resources from the Node
 			// first we make sure the profile isn't the one used by the shared pool
 			if r.PowerLibrary.GetSharedPool().GetPowerProfile() != nil && req.Name == r.PowerLibrary.GetSharedPool().GetPowerProfile().Name() {
-				r.PowerLibrary.GetSharedPool().SetPowerProfile(nil)
+				err := r.PowerLibrary.GetSharedPool().SetPowerProfile(nil)
+				if err != nil {
+					logger.Error(err, "error setting nil profile")
+					return ctrl.Result{}, err
+				}
 				return ctrl.Result{}, nil
 			}
 			pool := r.PowerLibrary.GetExclusivePool(req.Name)