Skip to content

Commit 31b24e5

Browse files
committed
roachtest: pin liveness lease to live node in lease prefs test
The lease preferences roachtest could occasionally fail, if the liveness leaseholder were on a stopped node. We should address this issue, for now, pin the liveness lease to a live node to prevent flakes. Informs: cockroachdb#108512 Resolves: cockroachdb#108425 Release note: None
1 parent 1fd3eb8 commit 31b24e5

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

pkg/cmd/roachtest/tests/lease_preferences.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,14 @@ func runLeasePreferences(
252252
// enforcement.
253253
require.NoError(t, WaitForReplication(ctx, t, conn, spec.replFactor))
254254

255+
// Set a lease preference for the liveness range, to be on n5. This test
256+
// would occasionally fail due to the liveness heartbeat failures, when the
257+
// liveness lease is on a stopped node. This is not ideal behavior, #108512.
258+
configureZone(t, ctx, conn, "RANGE liveness", zoneConfig{
259+
replicas: spec.replFactor,
260+
leaseNode: 5,
261+
})
262+
255263
t.L().Printf("setting lease preferences: %s", spec.preferences)
256264
setLeasePreferences(ctx, spec.preferences)
257265
t.L().Printf("waiting for initial lease preference conformance")

0 commit comments

Comments
 (0)