chore: enable errorlint (#673)

mmorel-35 · web-flow · commit b111e500826b · 2025-02-07T11:25:43.000-05:00
Signed-off-by: Matthieu MOREL &lt;matthieu.morel35@gmail.com&gt;
diff --git a/.golangci.yaml b/.golangci.yaml
@@ -8,7 +8,7 @@ issues:
 linters:
   enable:
     - errcheck
-#    - errorlint
+    - errorlint
     - gocritic
     - gofumpt
     - goimports
diff --git a/agent/main.go b/agent/main.go
@@ -84,7 +84,7 @@ func (s *settings) parseManifests() ([]*unstructured.Unstructured, string, error
 			}
 			items, err := kube.SplitYAML(data)
 			if err != nil {
-				return fmt.Errorf("failed to parse %s: %v", path, err)
+				return fmt.Errorf("failed to parse %s: %w", path, err)
 			}
 			res = append(res, items...)
 			return nil
diff --git a/pkg/cache/cluster.go b/pkg/cache/cluster.go
@@ -977,7 +977,7 @@ func (c *clusterCache) sync() error {
 	})
 	if err != nil {
 		c.log.Error(err, "Failed to sync cluster")
-		return fmt.Errorf("failed to sync cluster %s: %v", c.config.Host, err)
+		return fmt.Errorf("failed to sync cluster %s: %w", c.config.Host, err)
 	}
 
 	c.log.Info("Cluster successfully synced")
diff --git a/pkg/diff/diff.go b/pkg/diff/diff.go
@@ -234,7 +234,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa
 		mfs := &fieldpath.Set{}
 		err := mfs.FromJSON(bytes.NewReader(mfEntry.FieldsV1.Raw))
 		if err != nil {
-			return nil, fmt.Errorf("error building managedFields set: %s", err)
+			return nil, fmt.Errorf("error building managedFields set: %w", err)
 		}
 		if comparison.Added != nil && !comparison.Added.Empty() {
 			// exclude the added fields owned by this manager from the comparison
@@ -262,7 +262,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa
 		// revert modified fields not owned by any manager
 		typedPredictedLive, err = typedPredictedLive.Merge(liveModValues)
 		if err != nil {
-			return nil, fmt.Errorf("error reverting webhook modified fields in predicted live resource: %s", err)
+			return nil, fmt.Errorf("error reverting webhook modified fields in predicted live resource: %w", err)
 		}
 	}
 
@@ -271,7 +271,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa
 		// revert removed fields not owned by any manager
 		typedPredictedLive, err = typedPredictedLive.Merge(liveRmValues)
 		if err != nil {
-			return nil, fmt.Errorf("error reverting webhook removed fields in predicted live resource: %s", err)
+			return nil, fmt.Errorf("error reverting webhook removed fields in predicted live resource: %w", err)
 		}
 	}
 
@@ -287,7 +287,7 @@ func jsonStrToUnstructured(jsonString string) (*unstructured.Unstructured, error
 	res := make(map[string]any)
 	err := json.Unmarshal([]byte(jsonString), &res)
 	if err != nil {
-		return nil, fmt.Errorf("unmarshal error: %s", err)
+		return nil, fmt.Errorf("unmarshal error: %w", err)
 	}
 	return &unstructured.Unstructured{Object: res}, nil
 }
@@ -770,7 +770,7 @@ func GetLastAppliedConfigAnnotation(live *unstructured.Unstructured) (*unstructu
 	var obj unstructured.Unstructured
 	err := json.Unmarshal([]byte(lastAppliedStr), &obj)
 	if err != nil {
-		return nil, fmt.Errorf("failed to unmarshal %s in %s: %v", corev1.LastAppliedConfigAnnotation, live.GetName(), err)
+		return nil, fmt.Errorf("failed to unmarshal %s in %s: %w", corev1.LastAppliedConfigAnnotation, live.GetName(), err)
 	}
 	return &obj, nil
 }
@@ -1027,7 +1027,7 @@ func HideSecretData(target *unstructured.Unstructured, live *unstructured.Unstru
 		} else {
 			lastAppliedData, err := json.Marshal(liveLastAppliedAnnotation)
 			if err != nil {
-				return nil, nil, fmt.Errorf("error marshaling json: %s", err)
+				return nil, nil, fmt.Errorf("error marshaling json: %w", err)
 			}
 			annotations[corev1.LastAppliedConfigAnnotation] = string(lastAppliedData)
 		}
@@ -1055,7 +1055,7 @@ func hide(target, live, liveLastAppliedAnnotation *unstructured.Unstructured, ke
 				var err error
 				data, _, err = unstructured.NestedMap(obj.Object, fields...)
 				if err != nil {
-					return nil, nil, nil, fmt.Errorf("unstructured.NestedMap error: %s", err)
+					return nil, nil, nil, fmt.Errorf("unstructured.NestedMap error: %w", err)
 				}
 			}
 			if data == nil {
@@ -1075,7 +1075,7 @@ func hide(target, live, liveLastAppliedAnnotation *unstructured.Unstructured, ke
 			data[k] = replacement
 			err := unstructured.SetNestedField(obj.Object, data, fields...)
 			if err != nil {
-				return nil, nil, nil, fmt.Errorf("unstructured.SetNestedField error: %s", err)
+				return nil, nil, nil, fmt.Errorf("unstructured.SetNestedField error: %w", err)
 			}
 		}
 	}
diff --git a/pkg/health/health_apiservice.go b/pkg/health/health_apiservice.go
@@ -18,14 +18,14 @@ func getAPIServiceHealth(obj *unstructured.Unstructured) (*HealthStatus, error)
 		var apiService apiregistrationv1.APIService
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &apiService)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured APIService to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured APIService to typed: %w", err)
 		}
 		return getApiregistrationv1APIServiceHealth(&apiService)
 	case apiregistrationv1beta1.SchemeGroupVersion.WithKind(kube.APIServiceKind):
 		var apiService apiregistrationv1beta1.APIService
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &apiService)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured APIService to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured APIService to typed: %w", err)
 		}
 		return getApiregistrationv1beta1APIServiceHealth(&apiService)
 	default:
diff --git a/pkg/health/health_daemonset.go b/pkg/health/health_daemonset.go
@@ -17,7 +17,7 @@ func getDaemonSetHealth(obj *unstructured.Unstructured) (*HealthStatus, error) {
 		var daemon appsv1.DaemonSet
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &daemon)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured DaemonSet to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured DaemonSet to typed: %w", err)
 		}
 		return getAppsv1DaemonSetHealth(&daemon)
 	default:
diff --git a/pkg/health/health_deployment.go b/pkg/health/health_deployment.go
@@ -17,7 +17,7 @@ func getDeploymentHealth(obj *unstructured.Unstructured) (*HealthStatus, error)
 		var deployment appsv1.Deployment
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &deployment)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured Deployment to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured Deployment to typed: %w", err)
 		}
 		return getAppsv1DeploymentHealth(&deployment)
 	default:
diff --git a/pkg/health/health_job.go b/pkg/health/health_job.go
@@ -19,7 +19,7 @@ func getJobHealth(obj *unstructured.Unstructured) (*HealthStatus, error) {
 		var job batchv1.Job
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &job)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured Job to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured Job to typed: %w", err)
 		}
 		return getBatchv1JobHealth(&job)
 	default:
diff --git a/pkg/health/health_pod.go b/pkg/health/health_pod.go
@@ -19,7 +19,7 @@ func getPodHealth(obj *unstructured.Unstructured) (*HealthStatus, error) {
 		var pod corev1.Pod
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &pod)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured Pod to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured Pod to typed: %w", err)
 		}
 		return getCorev1PodHealth(&pod)
 	default:
diff --git a/pkg/health/health_pvc.go b/pkg/health/health_pvc.go
@@ -17,7 +17,7 @@ func getPVCHealth(obj *unstructured.Unstructured) (*HealthStatus, error) {
 		var pvc corev1.PersistentVolumeClaim
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &pvc)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured PersistentVolumeClaim to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured PersistentVolumeClaim to typed: %w", err)
 		}
 		return getCorev1PVCHealth(&pvc)
 	default:
diff --git a/pkg/health/health_replicaset.go b/pkg/health/health_replicaset.go
@@ -18,7 +18,7 @@ func getReplicaSetHealth(obj *unstructured.Unstructured) (*HealthStatus, error)
 		var replicaSet appsv1.ReplicaSet
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &replicaSet)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured ReplicaSet to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured ReplicaSet to typed: %w", err)
 		}
 		return getAppsv1ReplicaSetHealth(&replicaSet)
 	default:
diff --git a/pkg/health/health_service.go b/pkg/health/health_service.go
@@ -17,7 +17,7 @@ func getServiceHealth(obj *unstructured.Unstructured) (*HealthStatus, error) {
 		var service corev1.Service
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &service)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured Service to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured Service to typed: %w", err)
 		}
 		return getCorev1ServiceHealth(&service)
 	default:
diff --git a/pkg/health/health_statefulset.go b/pkg/health/health_statefulset.go
@@ -17,7 +17,7 @@ func getStatefulSetHealth(obj *unstructured.Unstructured) (*HealthStatus, error)
 		var sts appsv1.StatefulSet
 		err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &sts)
 		if err != nil {
-			return nil, fmt.Errorf("failed to convert unstructured StatefulSet to typed: %v", err)
+			return nil, fmt.Errorf("failed to convert unstructured StatefulSet to typed: %w", err)
 		}
 		return getAppsv1StatefulSetHealth(&sts)
 	default:
diff --git a/pkg/sync/sync_context.go b/pkg/sync/sync_context.go
@@ -864,7 +864,7 @@ func (sc *syncContext) autoCreateNamespace(tasks syncTasks) syncTasks {
 			} else if apierr.IsNotFound(err) {
 				tasks = sc.appendNsTask(tasks, &syncTask{phase: common.SyncPhasePreSync, targetObj: managedNs, liveObj: nil}, managedNs, nil)
 			} else {
-				tasks = sc.appendFailedNsTask(tasks, managedNs, fmt.Errorf("Namespace auto creation failed: %s", err))
+				tasks = sc.appendFailedNsTask(tasks, managedNs, fmt.Errorf("Namespace auto creation failed: %w", err))
 			}
 		} else {
 			sc.setOperationPhase(common.OperationFailed, fmt.Sprintf("Namespace auto creation failed: %s", err))
@@ -876,7 +876,7 @@ func (sc *syncContext) autoCreateNamespace(tasks syncTasks) syncTasks {
 func (sc *syncContext) appendNsTask(tasks syncTasks, preTask *syncTask, managedNs, liveNs *unstructured.Unstructured) syncTasks {
 	modified, err := sc.syncNamespace(managedNs, liveNs)
 	if err != nil {
-		tasks = sc.appendFailedNsTask(tasks, managedNs, fmt.Errorf("namespaceModifier error: %s", err))
+		tasks = sc.appendFailedNsTask(tasks, managedNs, fmt.Errorf("namespaceModifier error: %w", err))
 	} else if modified {
 		tasks = append(tasks, preTask)
 	}
diff --git a/pkg/utils/kube/ctl.go b/pkg/utils/kube/ctl.go
@@ -132,23 +132,23 @@ func (k *KubectlCmd) LoadOpenAPISchema(config *rest.Config) (openapi.Resources,
 	oapiGetter := openapi.NewOpenAPIGetter(disco)
 	oapiResources, err := openapi.NewOpenAPIParser(oapiGetter).Parse()
 	if err != nil {
-		return nil, nil, fmt.Errorf("error getting openapi resources: %s", err)
+		return nil, nil, fmt.Errorf("error getting openapi resources: %w", err)
 	}
 	gvkParser, err := k.newGVKParser(oapiGetter)
 	if err != nil {
-		return oapiResources, nil, fmt.Errorf("error getting gvk parser: %s", err)
+		return oapiResources, nil, fmt.Errorf("error getting gvk parser: %w", err)
 	}
 	return oapiResources, gvkParser, nil
 }
 
 func (k *KubectlCmd) newGVKParser(oapiGetter discovery.OpenAPISchemaInterface) (*managedfields.GvkParser, error) {
 	doc, err := oapiGetter.OpenAPISchema()
 	if err != nil {
-		return nil, fmt.Errorf("error getting openapi schema: %s", err)
+		return nil, fmt.Errorf("error getting openapi schema: %w", err)
 	}
 	models, err := proto.NewOpenAPIData(doc)
 	if err != nil {
-		return nil, fmt.Errorf("error getting openapi data: %s", err)
+		return nil, fmt.Errorf("error getting openapi data: %w", err)
 	}
 	var taintedGVKs []schema.GroupVersionKind
 	models, taintedGVKs = newUniqueModels(models)
@@ -274,13 +274,13 @@ func (k *KubectlCmd) DeleteResource(ctx context.Context, config *rest.Config, gv
 func (k *KubectlCmd) ManageResources(config *rest.Config, openAPISchema openapi.Resources) (ResourceOperations, func(), error) {
 	f, err := os.CreateTemp(utils.TempDir, "")
 	if err != nil {
-		return nil, nil, fmt.Errorf("failed to generate temp file for kubeconfig: %v", err)
+		return nil, nil, fmt.Errorf("failed to generate temp file for kubeconfig: %w", err)
 	}
 	_ = f.Close()
 	err = WriteKubeConfig(config, "", f.Name())
 	if err != nil {
 		utils.DeleteFile(f.Name())
-		return nil, nil, fmt.Errorf("failed to write kubeconfig: %v", err)
+		return nil, nil, fmt.Errorf("failed to write kubeconfig: %w", err)
 	}
 	fact := kubeCmdFactory(f.Name(), "", config)
 	cleanup := func() {
@@ -299,13 +299,13 @@ func (k *KubectlCmd) ManageResources(config *rest.Config, openAPISchema openapi.
 func ManageServerSideDiffDryRuns(config *rest.Config, openAPISchema openapi.Resources, tracer tracing.Tracer, log logr.Logger, onKubectlRun OnKubectlRunFunc) (diff.KubeApplier, func(), error) {
 	f, err := os.CreateTemp(utils.TempDir, "")
 	if err != nil {
-		return nil, nil, fmt.Errorf("failed to generate temp file for kubeconfig: %v", err)
+		return nil, nil, fmt.Errorf("failed to generate temp file for kubeconfig: %w", err)
 	}
 	_ = f.Close()
 	err = WriteKubeConfig(config, "", f.Name())
 	if err != nil {
 		utils.DeleteFile(f.Name())
-		return nil, nil, fmt.Errorf("failed to write kubeconfig: %v", err)
+		return nil, nil, fmt.Errorf("failed to write kubeconfig: %w", err)
 	}
 	fact := kubeCmdFactory(f.Name(), "", config)
 	cleanup := func() {
diff --git a/pkg/utils/kube/kube.go b/pkg/utils/kube/kube.go
@@ -4,6 +4,7 @@ package kube
 import (
 	"bytes"
 	"context"
+	"errors"
 	"fmt"
 	"io"
 	"regexp"
@@ -102,11 +103,11 @@ func GetObjectRef(obj *unstructured.Unstructured) v1.ObjectReference {
 func TestConfig(config *rest.Config) error {
 	kubeclientset, err := kubernetes.NewForConfig(config)
 	if err != nil {
-		return fmt.Errorf("REST config invalid: %s", err)
+		return fmt.Errorf("REST config invalid: %w", err)
 	}
 	_, err = kubeclientset.ServerVersion()
 	if err != nil {
-		return fmt.Errorf("REST config invalid: %s", err)
+		return fmt.Errorf("REST config invalid: %w", err)
 	}
 	return nil
 }
@@ -315,7 +316,7 @@ func SplitYAML(yamlData []byte) ([]*unstructured.Unstructured, error) {
 	for _, yml := range ymls {
 		u := &unstructured.Unstructured{}
 		if err := yaml.Unmarshal([]byte(yml), u); err != nil {
-			return objs, fmt.Errorf("failed to unmarshal manifest: %v", err)
+			return objs, fmt.Errorf("failed to unmarshal manifest: %w", err)
 		}
 		objs = append(objs, u)
 	}
@@ -334,10 +335,10 @@ func SplitYAMLToString(yamlData []byte) ([]string, error) {
 	for {
 		ext := runtime.RawExtension{}
 		if err := d.Decode(&ext); err != nil {
-			if err == io.EOF {
+			if errors.Is(err, io.EOF) {
 				break
 			}
-			return objs, fmt.Errorf("failed to unmarshal manifest: %v", err)
+			return objs, fmt.Errorf("failed to unmarshal manifest: %w", err)
 		}
 		ext.Raw = bytes.TrimSpace(ext.Raw)
 		if len(ext.Raw) == 0 || bytes.Equal(ext.Raw, []byte("null")) {
diff --git a/pkg/utils/kube/resource_ops.go b/pkg/utils/kube/resource_ops.go
@@ -95,13 +95,13 @@ func createManifestFile(obj *unstructured.Unstructured, log logr.Logger) (*os.Fi
 	}
 	manifestFile, err := os.CreateTemp(io.TempDir, "")
 	if err != nil {
-		return nil, fmt.Errorf("Failed to generate temp file for manifest: %v", err)
+		return nil, fmt.Errorf("Failed to generate temp file for manifest: %w", err)
 	}
 	if _, err = manifestFile.Write(manifestBytes); err != nil {
-		return nil, fmt.Errorf("Failed to write manifest: %v", err)
+		return nil, fmt.Errorf("Failed to write manifest: %w", err)
 	}
 	if err = manifestFile.Close(); err != nil {
-		return nil, fmt.Errorf("Failed to close manifest: %v", err)
+		return nil, fmt.Errorf("Failed to close manifest: %w", err)
 	}
 
 	err = maybeLogManifest(manifestBytes, log)
@@ -123,7 +123,7 @@ func (k *kubectlResourceOperations) runResourceCommand(ctx context.Context, obj
 	if obj.GetAPIVersion() == "rbac.authorization.k8s.io/v1" {
 		outReconcile, err := k.rbacReconcile(ctx, obj, manifestFile.Name(), dryRunStrategy)
 		if err != nil {
-			return "", fmt.Errorf("error running rbacReconcile: %s", err)
+			return "", fmt.Errorf("error running rbacReconcile: %w", err)
 		}
 		out = append(out, outReconcile)
 		// We still want to fallthrough and run `kubectl apply` in order set the

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ func (s settings) parseManifests() ([]unstructured.Unstructured, string, error`
`84`	`84`	`}`
`85`	`85`	`items, err := kube.SplitYAML(data)`
`86`	`86`	`if err != nil {`
`87`		`- return fmt.Errorf("failed to parse %s: %v", path, err)`
	`87`	`+ return fmt.Errorf("failed to parse %s: %w", path, err)`
`88`	`88`	`}`
`89`	`89`	`res = append(res, items...)`
`90`	`90`	`return nil`
Original file line number	Diff line number	Diff line change
`@@ -977,7 +977,7 @@ func (c *clusterCache) sync() error {`
`977`	`977`	`})`
`978`	`978`	`if err != nil {`
`979`	`979`	`c.log.Error(err, "Failed to sync cluster")`
`980`		`- return fmt.Errorf("failed to sync cluster %s: %v", c.config.Host, err)`
	`980`	`+ return fmt.Errorf("failed to sync cluster %s: %w", c.config.Host, err)`
`981`	`981`	`}`
`982`	`982`
`983`	`983`	`c.log.Info("Cluster successfully synced")`
Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa`
`234`	`234`	`mfs := &fieldpath.Set{}`
`235`	`235`	`err := mfs.FromJSON(bytes.NewReader(mfEntry.FieldsV1.Raw))`
`236`	`236`	`if err != nil {`
`237`		`- return nil, fmt.Errorf("error building managedFields set: %s", err)`
	`237`	`+ return nil, fmt.Errorf("error building managedFields set: %w", err)`
`238`	`238`	`}`
`239`	`239`	`if comparison.Added != nil && !comparison.Added.Empty() {`
`240`	`240`	`// exclude the added fields owned by this manager from the comparison`
`@@ -262,7 +262,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa`
`262`	`262`	`// revert modified fields not owned by any manager`
`263`	`263`	`typedPredictedLive, err = typedPredictedLive.Merge(liveModValues)`
`264`	`264`	`if err != nil {`
`265`		`- return nil, fmt.Errorf("error reverting webhook modified fields in predicted live resource: %s", err)`
	`265`	`+ return nil, fmt.Errorf("error reverting webhook modified fields in predicted live resource: %w", err)`
`266`	`266`	`}`
`267`	`267`	`}`
`268`	`268`
`@@ -271,7 +271,7 @@ func removeWebhookMutation(predictedLive, live *unstructured.Unstructured, gvkPa`
`271`	`271`	`// revert removed fields not owned by any manager`
`272`	`272`	`typedPredictedLive, err = typedPredictedLive.Merge(liveRmValues)`
`273`	`273`	`if err != nil {`
`274`		`- return nil, fmt.Errorf("error reverting webhook removed fields in predicted live resource: %s", err)`
	`274`	`+ return nil, fmt.Errorf("error reverting webhook removed fields in predicted live resource: %w", err)`
`275`	`275`	`}`
`276`	`276`	`}`
`277`	`277`
`@@ -287,7 +287,7 @@ func jsonStrToUnstructured(jsonString string) (*unstructured.Unstructured, error`
`287`	`287`	`res := make(map[string]any)`
`288`	`288`	`err := json.Unmarshal([]byte(jsonString), &res)`
`289`	`289`	`if err != nil {`
`290`		`- return nil, fmt.Errorf("unmarshal error: %s", err)`
	`290`	`+ return nil, fmt.Errorf("unmarshal error: %w", err)`
`291`	`291`	`}`
`292`	`292`	`return &unstructured.Unstructured{Object: res}, nil`
`293`	`293`	`}`
`@@ -770,7 +770,7 @@ func GetLastAppliedConfigAnnotation(live unstructured.Unstructured) (unstructu`
`770`	`770`	`var obj unstructured.Unstructured`
`771`	`771`	`err := json.Unmarshal([]byte(lastAppliedStr), &obj)`
`772`	`772`	`if err != nil {`
`773`		`- return nil, fmt.Errorf("failed to unmarshal %s in %s: %v", corev1.LastAppliedConfigAnnotation, live.GetName(), err)`
	`773`	`+ return nil, fmt.Errorf("failed to unmarshal %s in %s: %w", corev1.LastAppliedConfigAnnotation, live.GetName(), err)`
`774`	`774`	`}`
`775`	`775`	`return &obj, nil`
`776`	`776`	`}`
`@@ -1027,7 +1027,7 @@ func HideSecretData(target unstructured.Unstructured, live unstructured.Unstru`
`1027`	`1027`	`} else {`
`1028`	`1028`	`lastAppliedData, err := json.Marshal(liveLastAppliedAnnotation)`
`1029`	`1029`	`if err != nil {`
`1030`		`- return nil, nil, fmt.Errorf("error marshaling json: %s", err)`
	`1030`	`+ return nil, nil, fmt.Errorf("error marshaling json: %w", err)`
`1031`	`1031`	`}`
`1032`	`1032`	`annotations[corev1.LastAppliedConfigAnnotation] = string(lastAppliedData)`
`1033`	`1033`	`}`
`@@ -1055,7 +1055,7 @@ func hide(target, live, liveLastAppliedAnnotation *unstructured.Unstructured, ke`
`1055`	`1055`	`var err error`
`1056`	`1056`	`data, _, err = unstructured.NestedMap(obj.Object, fields...)`
`1057`	`1057`	`if err != nil {`
`1058`		`- return nil, nil, nil, fmt.Errorf("unstructured.NestedMap error: %s", err)`
	`1058`	`+ return nil, nil, nil, fmt.Errorf("unstructured.NestedMap error: %w", err)`
`1059`	`1059`	`}`
`1060`	`1060`	`}`
`1061`	`1061`	`if data == nil {`
`@@ -1075,7 +1075,7 @@ func hide(target, live, liveLastAppliedAnnotation *unstructured.Unstructured, ke`
`1075`	`1075`	`data[k] = replacement`
`1076`	`1076`	`err := unstructured.SetNestedField(obj.Object, data, fields...)`
`1077`	`1077`	`if err != nil {`
`1078`		`- return nil, nil, nil, fmt.Errorf("unstructured.SetNestedField error: %s", err)`
	`1078`	`+ return nil, nil, nil, fmt.Errorf("unstructured.SetNestedField error: %w", err)`
`1079`	`1079`	`}`
`1080`	`1080`	`}`
`1081`	`1081`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ func getDaemonSetHealth(obj unstructured.Unstructured) (HealthStatus, error) {`
`17`	`17`	`var daemon appsv1.DaemonSet`
`18`	`18`	`err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &daemon)`
`19`	`19`	`if err != nil {`
`20`		`- return nil, fmt.Errorf("failed to convert unstructured DaemonSet to typed: %v", err)`
	`20`	`+ return nil, fmt.Errorf("failed to convert unstructured DaemonSet to typed: %w", err)`
`21`	`21`	`}`
`22`	`22`	`return getAppsv1DaemonSetHealth(&daemon)`
`23`	`23`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ func getDeploymentHealth(obj unstructured.Unstructured) (HealthStatus, error)`
`17`	`17`	`var deployment appsv1.Deployment`
`18`	`18`	`err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &deployment)`
`19`	`19`	`if err != nil {`
`20`		`- return nil, fmt.Errorf("failed to convert unstructured Deployment to typed: %v", err)`
	`20`	`+ return nil, fmt.Errorf("failed to convert unstructured Deployment to typed: %w", err)`
`21`	`21`	`}`
`22`	`22`	`return getAppsv1DeploymentHealth(&deployment)`
`23`	`23`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ func getJobHealth(obj unstructured.Unstructured) (HealthStatus, error) {`
`19`	`19`	`var job batchv1.Job`
`20`	`20`	`err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &job)`
`21`	`21`	`if err != nil {`
`22`		`- return nil, fmt.Errorf("failed to convert unstructured Job to typed: %v", err)`
	`22`	`+ return nil, fmt.Errorf("failed to convert unstructured Job to typed: %w", err)`
`23`	`23`	`}`
`24`	`24`	`return getBatchv1JobHealth(&job)`
`25`	`25`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ func getPodHealth(obj unstructured.Unstructured) (HealthStatus, error) {`
`19`	`19`	`var pod corev1.Pod`
`20`	`20`	`err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &pod)`
`21`	`21`	`if err != nil {`
`22`		`- return nil, fmt.Errorf("failed to convert unstructured Pod to typed: %v", err)`
	`22`	`+ return nil, fmt.Errorf("failed to convert unstructured Pod to typed: %w", err)`
`23`	`23`	`}`
`24`	`24`	`return getCorev1PodHealth(&pod)`
`25`	`25`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ func getPVCHealth(obj unstructured.Unstructured) (HealthStatus, error) {`
`17`	`17`	`var pvc corev1.PersistentVolumeClaim`
`18`	`18`	`err := runtime.DefaultUnstructuredConverter.FromUnstructured(obj.Object, &pvc)`
`19`	`19`	`if err != nil {`
`20`		`- return nil, fmt.Errorf("failed to convert unstructured PersistentVolumeClaim to typed: %v", err)`
	`20`	`+ return nil, fmt.Errorf("failed to convert unstructured PersistentVolumeClaim to typed: %w", err)`
`21`	`21`	`}`
`22`	`22`	`return getCorev1PVCHealth(&pvc)`
`23`	`23`	`default:`