From f6ca94f01361019232d434c28500a875358a1d4f Mon Sep 17 00:00:00 2001 From: Florent Poinsard <35779988+frouioui@users.noreply.github.com> Date: Thu, 7 Nov 2024 14:44:48 -0600 Subject: [PATCH] Skip `TestRunFailsToStartTabletManager` for now (#17167) Signed-off-by: Florent Poinsard --- go/cmd/vttablet/cli/cli_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/go/cmd/vttablet/cli/cli_test.go b/go/cmd/vttablet/cli/cli_test.go index c88ebd1b8ae..603234b3c64 100644 --- a/go/cmd/vttablet/cli/cli_test.go +++ b/go/cmd/vttablet/cli/cli_test.go @@ -32,6 +32,9 @@ import ( // When starting, the TabletManager checks if it needs to restore, in tm.handleRestore but this step will // fail if we do not provide a cnf file and if the flag --restore_from_backup is provided. func TestRunFailsToStartTabletManager(t *testing.T) { + // Skipping the test for now, the test is unveiling some race conditions in the code. + // While working on a fix, this test will be skipped: https://github.com/vitessio/vitess/pull/17165 + t.Skip() ts, factory := memorytopo.NewServerAndFactory(context.Background(), "cell") topo.RegisterFactory("test", factory)