Don't log-and-return error

It's enough for the caller to process the error, no need to log it.

Signed-off-by: Tom Wieczorek <twieczorek@mirantis.com>

inttest/ap-removedapis/removedapis_test.go

@@ -88,6 +88,7 @@ func (s *plansRemovedAPIsSuite) TestApply() {
 	plan, err := aptest.WaitForPlanState(s.Context(), client, apconst.AutopilotName, appc.PlanWarning)
 	if s.NoError(err) && s.Len(plan.Status.Commands, 1) {
 		s.Equal(appc.PlanWarning, plan.Status.Commands[0].State)
+		s.Equal("removedcrds.autopilot.k0sproject.io v1beta1 has been removed in Kubernetes v99.99.99, but there are 1 such resources in the cluster", plan.Status.Commands[0].Description)
 	}
 }
 

pkg/autopilot/checks/checks.go

@@ -91,9 +91,7 @@ func CanUpdate(log logrus.FieldLogger, clientFactory kubernetes.ClientFactoryInt
 				}
 			}
 			if found > 0 {
-				err = fmt.Errorf("%s is removed in Kubernetes %s. There are %d resources of the type in the cluster", gvk.String(), semver.MajorMinor(newVersion), found)
-				logrus.Error(err)
-				return err
+				return fmt.Errorf("%s.%s %s has been removed in Kubernetes %s, but there are %d such resources in the cluster", ar.Name, gv.Group, gv.Version, removedInVersion, found)
 			}
 		}
 	}