Merge cockroachdb#107690

craig[bot] · tbg · craig[bot] · commit b76a900fd29f · 2023-07-27T11:17:05.000Z
107690: kvserver: downgrade "unable to transfer lease" log r=kvoli a=tbg It doesn't rise up to "ERROR" level and also doesn't need to log a large stack trace. Seen while looking into cockroachdb#106140. Epic: none Release note: None Co-authored-by: Tobias Grieger <tobias.b.grieger@gmail.com>
diff --git a/pkg/kv/kvserver/store_rebalancer.go b/pkg/kv/kvserver/store_rebalancer.go
@@ -513,7 +513,7 @@ func (sr *StoreRebalancer) applyLeaseRebalance(
 			candidateReplica.RangeUsageInfo(),
 		)
 	}); err != nil {
-		log.KvDistribution.Errorf(ctx, "unable to transfer lease to s%d: %+v", target.StoreID, err)
+		log.KvDistribution.Infof(ctx, "unable to transfer lease to s%d: %v", target.StoreID, err)
 		return false
 	}
 	return true

Original file line number	Diff line number	Diff line change
`@@ -513,7 +513,7 @@ func (sr *StoreRebalancer) applyLeaseRebalance(`
`513`	`513`	`candidateReplica.RangeUsageInfo(),`
`514`	`514`	`)`
`515`	`515`	`}); err != nil {`
`516`		`- log.KvDistribution.Errorf(ctx, "unable to transfer lease to s%d: %+v", target.StoreID, err)`
	`516`	`+ log.KvDistribution.Infof(ctx, "unable to transfer lease to s%d: %v", target.StoreID, err)`
`517`	`517`	`return false`
`518`	`518`	`}`
`519`	`519`	`return true`