kvnemesis: add TestKVNemesisMultiNode_BufferedWritesNoPipelining

This new kvnemesis test is aimed at stressing the lock reliability
features more while working around a known bug in those features when
used in conjunction with pipelining.

Epic: none
Release note: None

Author: stevendanna

pkg/kv/kvnemesis/kvnemesis_test.go

@@ -361,6 +361,33 @@ func TestKVNemesisMultiNode_BufferedWrites(t *testing.T) {
 	})
 }
 
+// TestKVNemesisMultiNode_BufferedWritesNoPipelining turns on buffered
+// writes and turns off write pipelining. Turning off write pipelining
+// allows us to test the lock reliability features even without a fix
+// for #145458.
+func TestKVNemesisMultiNode_BufferedWritesNoPipelining(t *testing.T) {
+	defer leaktest.AfterTest(t)()
+	defer log.Scope(t).Close(t)
+
+	testKVNemesisImpl(t, kvnemesisTestCfg{
+		numNodes:                     3,
+		numSteps:                     defaultNumSteps,
+		concurrency:                  5,
+		seedOverride:                 0,
+		invalidLeaseAppliedIndexProb: 0.2,
+		injectReproposalErrorProb:    0.2,
+		assertRaftApply:              true,
+		bufferedWriteProb:            0.70,
+		testSettings: func(ctx context.Context, st *cluster.Settings) {
+			kvcoord.BufferedWritesEnabled.Override(ctx, &st.SV, true)
+			kvcoord.PipelinedWritesEnabled.Override(ctx, &st.SV, false)
+			concurrency.UnreplicatedLockReliabilityLeaseTransfer.Override(ctx, &st.SV, true)
+			concurrency.UnreplicatedLockReliabilityMerge.Override(ctx, &st.SV, true)
+			concurrency.UnreplicatedLockReliabilitySplit.Override(ctx, &st.SV, true)
+		},
+	})
+}
+
 func TestKVNemesisMultiNode(t *testing.T) {
 	defer leaktest.AfterTest(t)()
 	defer log.Scope(t).Close(t)