From 5980e7e63a326289590a68a3e62d1f376082f0df Mon Sep 17 00:00:00 2001 From: Volodymyr Zotov Date: Tue, 8 Jul 2025 16:42:38 -0500 Subject: [PATCH] Wrap the error to not lose child errors --- internal/controller/deployment_controller.go | 4 ++-- internal/controller/onepassworditem_controller.go | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/controller/deployment_controller.go b/internal/controller/deployment_controller.go index f031620..b299cec 100644 --- a/internal/controller/deployment_controller.go +++ b/internal/controller/deployment_controller.go @@ -203,13 +203,13 @@ func (r *DeploymentReconciler) handleApplyingDeployment(ctx context.Context, dep item, err := op.GetOnePasswordItemByPath(ctx, r.OpClient, annotations[op.ItemPathAnnotation]) if err != nil { - return fmt.Errorf("Failed to retrieve item: %v", err) + return fmt.Errorf("failed to retrieve item: %w", err) } // Create owner reference. gvk, err := apiutil.GVKForObject(deployment, r.Scheme) if err != nil { - return fmt.Errorf("could not to retrieve group version kind: %v", err) + return fmt.Errorf("could not to retrieve group version kind: %w", err) } ownerRef := &metav1.OwnerReference{ APIVersion: gvk.GroupVersion().String(), diff --git a/internal/controller/onepassworditem_controller.go b/internal/controller/onepassworditem_controller.go index 3dfd046..078af65 100644 --- a/internal/controller/onepassworditem_controller.go +++ b/internal/controller/onepassworditem_controller.go @@ -173,13 +173,13 @@ func (r *OnePasswordItemReconciler) handleOnePasswordItem(ctx context.Context, r item, err := op.GetOnePasswordItemByPath(ctx, r.OpClient, resource.Spec.ItemPath) if err != nil { - return fmt.Errorf("Failed to retrieve item: %v", err) + return fmt.Errorf("failed to retrieve item: %w", err) } // Create owner reference. gvk, err := apiutil.GVKForObject(resource, r.Scheme) if err != nil { - return fmt.Errorf("could not to retrieve group version kind: %v", err) + return fmt.Errorf("could not to retrieve group version kind: %w", err) } ownerRef := &metav1.OwnerReference{ APIVersion: gvk.GroupVersion().String(),