*: enable flaky test for all test (pingcap#36385)

close pingcap#36404
morgo · Jul 22, 2022 · bf743a6 · bf743a6
1 parent 61175e8
commit bf743a6
Show file tree

Hide file tree

Showing 196 changed files with 227 additions and 4 deletions.
diff --git a/Makefile b/Makefile
@@ -435,10 +435,10 @@ bazel_test: failpoint-enable bazel_ci_prepare
 
 
 bazel_coverage_test: failpoint-enable bazel_ci_prepare
-	bazel --output_user_root=/home/jenkins/.tidb/tmp coverage --config=ci --@io_bazel_rules_go//go/config:cover_format=go_cover \
+	bazel --output_user_root=/home/jenkins/.tidb/tmp coverage --config=ci --build_event_json_file=bazel_1.json --@io_bazel_rules_go//go/config:cover_format=go_cover \
 		-- //... -//cmd/... -//tests/graceshutdown/... \
 		-//tests/globalkilltest/... -//tests/readonlytest/... -//br/pkg/task:task_test
-	bazel --output_user_root=/home/jenkins/.tidb/tmp coverage --config=ci --@io_bazel_rules_go//go/config:cover_format=go_cover --define gotags=featuretag \
+	bazel --output_user_root=/home/jenkins/.tidb/tmp coverage --config=ci --build_event_json_file=bazel_2.json --@io_bazel_rules_go//go/config:cover_format=go_cover --define gotags=featuretag \
 		-- //... -//cmd/... -//tests/graceshutdown/... \
 		-//tests/globalkilltest/... -//tests/readonlytest/... -//br/pkg/task:task_test
 

diff --git a/bindinfo/BUILD.bazel b/bindinfo/BUILD.bazel
@@ -55,6 +55,7 @@ go_test(
         "temptable_test.go",
     ],
     embed = [":bindinfo"],
+    flaky = True,
     deps = [
         "//config",
         "//domain",

diff --git a/br/cmd/br/BUILD.bazel b/br/cmd/br/BUILD.bazel
@@ -56,4 +56,5 @@ go_test(
     name = "br_test",
     srcs = ["main_test.go"],
     embed = [":br_lib"],
+    flaky = True,
 )
diff --git a/br/cmd/tidb-lightning-ctl/BUILD.bazel b/br/cmd/tidb-lightning-ctl/BUILD.bazel
@@ -28,4 +28,5 @@ go_test(
     name = "tidb-lightning-ctl_test",
     srcs = ["main_test.go"],
     embed = [":tidb-lightning-ctl_lib"],
+    flaky = True,
 )
diff --git a/br/cmd/tidb-lightning/BUILD.bazel b/br/cmd/tidb-lightning/BUILD.bazel
@@ -25,4 +25,5 @@ go_test(
     name = "tidb-lightning_test",
     srcs = ["main_test.go"],
     embed = [":tidb-lightning_lib"],
+    flaky = True,
 )
diff --git a/br/pkg/backup/BUILD.bazel b/br/pkg/backup/BUILD.bazel
@@ -64,6 +64,7 @@ go_test(
         "schema_test.go",
     ],
     embed = [":backup"],
+    flaky = True,
     deps = [
         "//br/pkg/conn",
         "//br/pkg/metautil",

diff --git a/br/pkg/checksum/BUILD.bazel b/br/pkg/checksum/BUILD.bazel
@@ -35,6 +35,7 @@ go_test(
         "main_test.go",
     ],
     embed = [":checksum"],
+    flaky = True,
     deps = [
         "//br/pkg/backup",
         "//br/pkg/metautil",

diff --git a/br/pkg/conn/BUILD.bazel b/br/pkg/conn/BUILD.bazel
@@ -14,6 +14,7 @@ go_library(
         "//br/pkg/version",
         "//domain",
         "//kv",
+        "//util/engine",
         "@com_github_docker_go_units//:go-units",
         "@com_github_opentracing_opentracing_go//:opentracing-go",
         "@com_github_pingcap_errors//:errors",
@@ -41,6 +42,7 @@ go_test(
         "main_test.go",
     ],
     embed = [":conn"],
+    flaky = True,
     deps = [
         "//br/pkg/pdutil",
         "//br/pkg/utils",

diff --git a/br/pkg/errors/BUILD.bazel b/br/pkg/errors/BUILD.bazel
@@ -11,6 +11,7 @@ go_library(
 go_test(
     name = "errors_test",
     srcs = ["errors_test.go"],
+    flaky = True,
     deps = [
         ":errors",
         "@com_github_pingcap_errors//:errors",

diff --git a/br/pkg/glue/BUILD.bazel b/br/pkg/glue/BUILD.bazel
@@ -25,6 +25,7 @@ go_library(
 go_test(
     name = "glue_test",
     srcs = ["console_glue_test.go"],
+    flaky = True,
     deps = [
         ":glue",
         "@com_github_fatih_color//:color",

diff --git a/br/pkg/gluetikv/BUILD.bazel b/br/pkg/gluetikv/BUILD.bazel
@@ -22,5 +22,6 @@ go_test(
     name = "gluetikv_test",
     srcs = ["glue_test.go"],
     embed = [":gluetikv"],
+    flaky = True,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/br/pkg/lightning/BUILD.bazel b/br/pkg/lightning/BUILD.bazel
@@ -48,6 +48,7 @@ go_test(
         "main_test.go",
     ],
     embed = [":lightning"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/checkpoints",
         "//br/pkg/lightning/config",

diff --git a/br/pkg/lightning/backend/BUILD.bazel b/br/pkg/lightning/backend/BUILD.bazel
@@ -26,6 +26,7 @@ go_library(
 go_test(
     name = "backend_test",
     srcs = ["backend_test.go"],
+    flaky = True,
     deps = [
         ":backend",
         "//br/pkg/lightning/backend/kv",

diff --git a/br/pkg/lightning/backend/kv/BUILD.bazel b/br/pkg/lightning/backend/kv/BUILD.bazel
@@ -50,6 +50,7 @@ go_test(
         "sql2kv_test.go",
     ],
     embed = [":kv"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/common",
         "//br/pkg/lightning/log",

diff --git a/br/pkg/lightning/backend/local/BUILD.bazel b/br/pkg/lightning/backend/local/BUILD.bazel
@@ -43,6 +43,7 @@ go_library(
         "//table",
         "//tablecodec",
         "//util/codec",
+        "//util/engine",
         "//util/hack",
         "//util/mathutil",
         "//util/ranger",
@@ -88,6 +89,7 @@ go_test(
         "localhelper_test.go",
     ],
     embed = [":local"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/backend",
         "//br/pkg/lightning/backend/kv",
@@ -100,14 +102,14 @@ go_test(
         "//br/pkg/pdutil",
         "//br/pkg/restore",
         "//br/pkg/utils",
-        "//br/pkg/version",
         "//kv",
         "//parser/mysql",
         "//sessionctx/stmtctx",
         "//store/pdtypes",
         "//tablecodec",
         "//types",
         "//util/codec",
+        "//util/engine",
         "//util/hack",
         "@com_github_cockroachdb_pebble//:pebble",
         "@com_github_cockroachdb_pebble//sstable",

diff --git a/br/pkg/lightning/backend/tidb/BUILD.bazel b/br/pkg/lightning/backend/tidb/BUILD.bazel
@@ -32,6 +32,7 @@ go_library(
 go_test(
     name = "tidb_test",
     srcs = ["tidb_test.go"],
+    flaky = True,
     deps = [
         ":tidb",
         "//br/pkg/lightning/backend",

diff --git a/br/pkg/lightning/checkpoints/BUILD.bazel b/br/pkg/lightning/checkpoints/BUILD.bazel
@@ -40,6 +40,7 @@ go_test(
         "main_test.go",
     ],
     embed = [":checkpoints"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/checkpoints/checkpointspb",
         "//br/pkg/lightning/config",

diff --git a/br/pkg/lightning/common/BUILD.bazel b/br/pkg/lightning/common/BUILD.bazel
@@ -50,6 +50,7 @@ go_test(
         "util_test.go",
     ],
     embed = [":common"],
+    flaky = True,
     deps = [
         "//br/pkg/errors",
         "//br/pkg/lightning/log",

diff --git a/br/pkg/lightning/config/BUILD.bazel b/br/pkg/lightning/config/BUILD.bazel
@@ -36,6 +36,7 @@ go_test(
         "config_test.go",
         "configlist_test.go",
     ],
+    flaky = True,
     deps = [
         ":config",
         "//parser/mysql",

diff --git a/br/pkg/lightning/errormanager/BUILD.bazel b/br/pkg/lightning/errormanager/BUILD.bazel
@@ -24,6 +24,7 @@ go_test(
     name = "errormanager_test",
     srcs = ["errormanager_test.go"],
     embed = [":errormanager"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/config",
         "//br/pkg/lightning/log",

diff --git a/br/pkg/lightning/log/BUILD.bazel b/br/pkg/lightning/log/BUILD.bazel
@@ -27,6 +27,7 @@ go_test(
         "filter_test.go",
         "log_test.go",
     ],
+    flaky = True,
     importpath = "github.com/pingcap/tidb/br/pkg/lightning/log_test",
     deps = [
         ":log",

diff --git a/br/pkg/lightning/metric/BUILD.bazel b/br/pkg/lightning/metric/BUILD.bazel
@@ -15,6 +15,7 @@ go_library(
 go_test(
     name = "metric_test",
     srcs = ["metric_test.go"],
+    flaky = True,
     deps = [
         ":metric",
         "//util/promutil",

diff --git a/br/pkg/lightning/mydump/BUILD.bazel b/br/pkg/lightning/mydump/BUILD.bazel
@@ -59,6 +59,7 @@ go_test(
         "examples/*",
     ]),
     embed = [":mydump"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/config",
         "//br/pkg/lightning/log",

diff --git a/br/pkg/lightning/restore/BUILD.bazel b/br/pkg/lightning/restore/BUILD.bazel
@@ -54,6 +54,7 @@ go_library(
         "//table/tables",
         "//types",
         "//util/collate",
+        "//util/engine",
         "//util/mathutil",
         "//util/mock",
         "@com_github_coreos_go_semver//semver",
@@ -91,6 +92,7 @@ go_test(
         "tidb_test.go",
     ],
     embed = [":restore"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/backend",
         "//br/pkg/lightning/backend/kv",

diff --git a/br/pkg/lightning/restore/mock/BUILD.bazel b/br/pkg/lightning/restore/mock/BUILD.bazel
@@ -19,6 +19,7 @@ go_test(
     name = "mock_test",
     srcs = ["mock_test.go"],
     embed = [":mock"],
+    flaky = True,
     deps = [
         "//br/pkg/lightning/restore",
         "//parser/model",

diff --git a/br/pkg/lightning/tikv/BUILD.bazel b/br/pkg/lightning/tikv/BUILD.bazel
@@ -26,6 +26,7 @@ go_library(
 go_test(
     name = "tikv_test",
     srcs = ["tikv_test.go"],
+    flaky = True,
     deps = [
         ":tikv",
         "//br/pkg/lightning/common",

diff --git a/br/pkg/lightning/verification/BUILD.bazel b/br/pkg/lightning/verification/BUILD.bazel
@@ -14,6 +14,7 @@ go_library(
 go_test(
     name = "verification_test",
     srcs = ["checksum_test.go"],
+    flaky = True,
     deps = [
         ":verification",
         "//br/pkg/lightning/common",

diff --git a/br/pkg/lightning/worker/BUILD.bazel b/br/pkg/lightning/worker/BUILD.bazel
@@ -11,6 +11,7 @@ go_library(
 go_test(
     name = "worker_test",
     srcs = ["worker_test.go"],
+    flaky = True,
     deps = [
         ":worker",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/logutil/BUILD.bazel b/br/pkg/logutil/BUILD.bazel
@@ -28,6 +28,7 @@ go_library(
 go_test(
     name = "logutil_test",
     srcs = ["logging_test.go"],
+    flaky = True,
     deps = [
         ":logutil",
         "//br/pkg/errors",

diff --git a/br/pkg/membuf/BUILD.bazel b/br/pkg/membuf/BUILD.bazel
@@ -11,5 +11,6 @@ go_test(
     name = "membuf_test",
     srcs = ["buffer_test.go"],
     embed = [":membuf"],
+    flaky = True,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/br/pkg/metautil/BUILD.bazel b/br/pkg/metautil/BUILD.bazel
@@ -32,6 +32,7 @@ go_test(
         "metafile_test.go",
     ],
     embed = [":metautil"],
+    flaky = True,
     deps = [
         "//br/pkg/mock/storage",
         "//testkit/testsetup",

diff --git a/br/pkg/mock/BUILD.bazel b/br/pkg/mock/BUILD.bazel
@@ -53,6 +53,7 @@ go_library(
 go_test(
     name = "mock_test",
     srcs = ["mock_cluster_test.go"],
+    flaky = True,
     deps = [
         ":mock",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/pdutil/BUILD.bazel b/br/pkg/pdutil/BUILD.bazel
@@ -35,6 +35,7 @@ go_test(
         "pd_serial_test.go",
     ],
     embed = [":pdutil"],
+    flaky = True,
     deps = [
         "//store/pdtypes",
         "//testkit/testsetup",

diff --git a/br/pkg/redact/BUILD.bazel b/br/pkg/redact/BUILD.bazel
@@ -11,6 +11,7 @@ go_library(
 go_test(
     name = "redact_test",
     srcs = ["redact_test.go"],
+    flaky = True,
     deps = [
         ":redact",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/restore/BUILD.bazel b/br/pkg/restore/BUILD.bazel
@@ -103,6 +103,7 @@ go_test(
         "util_test.go",
     ],
     embed = [":restore"],
+    flaky = True,
     deps = [
         "//br/pkg/backup",
         "//br/pkg/conn",

diff --git a/br/pkg/rtree/BUILD.bazel b/br/pkg/rtree/BUILD.bazel
@@ -26,6 +26,7 @@ go_test(
         "main_test.go",
         "rtree_test.go",
     ],
+    flaky = True,
     deps = [
         ":rtree",
         "//testkit/testsetup",

diff --git a/br/pkg/storage/BUILD.bazel b/br/pkg/storage/BUILD.bazel
@@ -63,6 +63,7 @@ go_test(
         "writer_test.go",
     ],
     embed = [":storage"],
+    flaky = True,
     deps = [
         "//br/pkg/mock",
         "@com_github_aws_aws_sdk_go//aws",

diff --git a/br/pkg/stream/BUILD.bazel b/br/pkg/stream/BUILD.bazel
@@ -8,6 +8,7 @@ go_library(
         "rewrite_meta_rawkv.go",
         "stream_mgr.go",
         "stream_status.go",
+        "util.go",
     ],
     importpath = "github.com/pingcap/tidb/br/pkg/stream",
     visibility = ["//visibility:public"],
@@ -45,8 +46,10 @@ go_test(
         "meta_kv_test.go",
         "rewrite_meta_rawkv_test.go",
         "stream_misc_test.go",
+        "util_test.go",
     ],
     embed = [":stream"],
+    flaky = True,
     deps = [
         "//br/pkg/streamhelper",
         "//meta",
@@ -56,5 +59,6 @@ go_test(
         "//util/table-filter",
         "@com_github_pingcap_kvproto//pkg/brpb",
         "@com_github_stretchr_testify//require",
+        "@com_github_tikv_client_go_v2//oracle",
     ],
 )
diff --git a/br/pkg/streamhelper/BUILD.bazel b/br/pkg/streamhelper/BUILD.bazel
@@ -26,6 +26,7 @@ go_library(
         "//kv",
         "//metrics",
         "//owner",
+        "//util/mathutil",
         "@com_github_gogo_protobuf//proto",
         "@com_github_golang_protobuf//proto",
         "@com_github_google_btree//:btree",
@@ -56,6 +57,7 @@ go_test(
         "regioniter_test.go",
         "tsheap_test.go",
     ],
+    flaky = True,
     deps = [
         ":streamhelper",
         "//br/pkg/errors",

diff --git a/br/pkg/summary/BUILD.bazel b/br/pkg/summary/BUILD.bazel
@@ -23,6 +23,7 @@ go_test(
         "main_test.go",
     ],
     embed = [":summary"],
+    flaky = True,
     deps = [
         "//testkit/testsetup",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/task/BUILD.bazel b/br/pkg/task/BUILD.bazel
@@ -34,7 +34,6 @@ go_library(
         "//config",
         "//kv",
         "//parser/mysql",
-        "//sessionctx",
         "//sessionctx/stmtctx",
         "//sessionctx/variable",
         "//statistics/handle",
@@ -77,6 +76,7 @@ go_test(
         "stream_test.go",
     ],
     embed = [":task"],
+    flaky = True,
     deps = [
         "//br/pkg/conn",
         "//br/pkg/metautil",