Skip to content

Commit 635a56f

Browse files
committed
kvserver: more logging for TestReplicaCircuitBreaker_Partial_Retry
This test verifies the value of the `InLeaseTransferBackoffs` metric, but without more verbose `DistSender` logging, it's hard to investigate failures where the metric has unexpected values. Fixes: #154179 Release note: None
1 parent d358a24 commit 635a56f

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

pkg/kv/kvserver/client_replica_circuit_breaker_test.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -793,6 +793,11 @@ func TestReplicaCircuitBreaker_Partial_Retry(t *testing.T) {
793793
skip.UnderRace(t)
794794
skip.UnderDeadlock(t)
795795

796+
// To help debug issues like #154179.
797+
prevVModule := log.GetVModule()
798+
defer func() { _ = log.SetVModule(prevVModule) }()
799+
require.NoError(t, log.SetVModule("dist_sender=3"))
800+
796801
testutils.RunValues(t, "lease-type", roachpb.ExpirationAndLeaderLeaseType(),
797802
func(t *testing.T, leaseType roachpb.LeaseType) {
798803
// Use a context timeout, to prevent test hangs on failures.

0 commit comments

Comments
 (0)