avoid error status accumulation when failing to fetch hub channel

Signed-off-by: Xiangjing Li <[email protected]>
open-cluster-management-io · Jul 29, 2024 · a694481 · a694481
1 parent 7ecb081
commit a694481
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/subscription/subscription_controller.go b/pkg/controller/subscription/subscription_controller.go
@@ -337,7 +337,7 @@ func (r *ReconcileSubscription) doReconcile(instance *appv1.Subscription) error
 
  err = r.hubclient.Get(context.TODO(), chnkey, subitem.Channel)
  if err != nil {
- return gerr.Wrapf(err, "failed to get channel of subscription %v", instance)
+ return gerr.Wrap(err, "failed to get channel")
  }
  }
 
@@ -351,7 +351,7 @@ func (r *ReconcileSubscription) doReconcile(instance *appv1.Subscription) error
 
  err = r.hubclient.Get(context.TODO(), scndChnkey, subitem.SecondaryChannel)
  if err != nil {
- return gerr.Wrapf(err, "failed to get the secondary channel of subscription %v", instance)
+ return gerr.Wrap(err, "failed to get the secondary channel")
  }
  }
  }