sql: fix lease duration in TestReacquireLeaseOnRestart

The test jumps the clock forward in order to expire KV leases. However,
the DefaultDescriptorLeaseDuration constant that it chooses seems to be
by mistake taken as the KV lease duration. Use the RangeLeaseDuration
from the server config instead.

This also fixes early SQL pod termination when multi-tenancy is enabled.
Currently, the lifetime of the SQL pod is the same as that of the
session, which has a TTL of 40s (server.sqlliveness.ttl setting).  The
10 minute clock jump in this test was causing early SQL pod termination
and flakes.

Epic: none
Release note: none

Author: pav-kv

pkg/sql/txn_restart_test.go

@@ -1265,6 +1265,9 @@ func TestReacquireLeaseOnRestart(t *testing.T) {
 	clientTestingKnobs := &kvcoord.ClientTestingKnobs{
 		MaxTxnRefreshAttempts: refreshAttempts,
 	}
+	var params base.TestServerArgs
+	params.RaftConfig.SetDefaults()
+	leaseDuration := params.RaftConfig.RangeLeaseDuration
 
 	testKey := []byte("test_key")
 	var s serverutils.ApplicationLayerInterface
@@ -1280,7 +1283,7 @@ func TestReacquireLeaseOnRestart(t *testing.T) {
 						atomic.AddInt32(&clockUpdate, 1)
 						// Hack to advance the transaction timestamp on a
 						// transaction restart.
-						const advancement = 2 * base.DefaultDescriptorLeaseDuration
+						advancement := 2 * leaseDuration
 						now := s.Clock().NowAsClockTimestamp()
 						now.WallTime += advancement.Nanoseconds()
 						s.Clock().Update(now)
@@ -1310,7 +1313,6 @@ func TestReacquireLeaseOnRestart(t *testing.T) {
 		DisableMaxOffsetCheck: true,
 	}
 
-	var params base.TestServerArgs
 	params.Knobs.Store = storeTestingKnobs
 	params.Knobs.KVClient = clientTestingKnobs
 	params.DefaultTestTenant = base.TestDoesNotWorkWithExternalProcessMode(156333)