Skip to content

Commit 9cde0a6

Browse files
committed
server: deflake TestServerShutdownReleasesSession
The tenant was not being fully stopped, so the test could encounter flakes. Release note: None
1 parent 3bbf620 commit 9cde0a6

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

pkg/server/drain_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -346,6 +346,7 @@ func TestServerShutdownReleasesSession(t *testing.T) {
346346
require.True(t, sessionExists(*session))
347347

348348
require.NoError(t, tmpTenant.DrainClients(context.Background()))
349+
tmpTenant.Stopper().Stop(ctx)
349350

350351
require.False(t, sessionExists(*session), "expected session %s to be deleted from the sqlliveness table, but it still exists", *session)
351352
require.Nil(t, queryOwner(tmpSQLInstance), "expected sql_instance %d to have no owning session_id", tmpSQLInstance)

0 commit comments

Comments
 (0)