*: update bazel config (#41296)

pingcap · Feb 13, 2023 · 79ae1e3 · 79ae1e3
1 parent 8dd78d5
commit 79ae1e3
Show file tree

Hide file tree

Showing 46 changed files with 64 additions and 31 deletions.
diff --git a/.bazelrc b/.bazelrc
@@ -20,7 +20,7 @@ build:race --@io_bazel_rules_go//go/config:race --test_env=GORACE=halt_on_error=
 
 test --test_env=TZ=Asia/Shanghai
 test --test_output=errors --test_summary=testcase
-test:ci --color=yes --spawn_strategy=local
+test:ci --color=yes
 test:ci --verbose_failures --test_verbose_timeout_warnings
 test:ci --test_env=GO_TEST_WRAP_TESTV=1
 test:ci --experimental_ui_max_stdouterr_bytes=104857600

diff --git a/autoid_service/BUILD.bazel b/autoid_service/BUILD.bazel
@@ -27,8 +27,10 @@ go_library(
 
 go_test(
     name = "autoid_service_test",
+    timeout = "short",
     srcs = ["autoid_test.go"],
     embed = [":autoid_service"],
+    flaky = True,
     deps = [
         "//parser/model",
         "//testkit",

diff --git a/br/pkg/aws/BUILD.bazel b/br/pkg/aws/BUILD.bazel
@@ -23,8 +23,10 @@ go_library(
 
 go_test(
     name = "aws_test",
+    timeout = "short",
     srcs = ["ebs_test.go"],
     embed = [":aws"],
+    flaky = True,
     deps = [
         "@com_github_aws_aws_sdk_go//aws",
         "@com_github_aws_aws_sdk_go//service/ec2",

diff --git a/br/pkg/checkpoint/BUILD.bazel b/br/pkg/checkpoint/BUILD.bazel
@@ -22,7 +22,9 @@ go_library(
 
 go_test(
     name = "checkpoint_test",
+    timeout = "short",
     srcs = ["checkpoint_test.go"],
+    flaky = True,
     deps = [
         ":checkpoint",
         "//br/pkg/rtree",

diff --git a/br/pkg/config/BUILD.bazel b/br/pkg/config/BUILD.bazel
@@ -15,8 +15,10 @@ go_library(
 
 go_test(
     name = "config_test",
+    timeout = "short",
     srcs = ["ebs_test.go"],
     data = ["ebs_backup.json"],
     embed = [":config"],
+    flaky = True,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/br/pkg/restore/prealloc_table_id/BUILD.bazel b/br/pkg/restore/prealloc_table_id/BUILD.bazel
@@ -13,6 +13,7 @@ go_library(
 
 go_test(
     name = "prealloc_table_id_test",
+    timeout = "short",
     srcs = ["alloc_test.go"],
     flaky = True,
     race = "on",

diff --git a/br/pkg/restore/split/BUILD.bazel b/br/pkg/restore/split/BUILD.bazel
@@ -42,7 +42,9 @@ go_library(
 
 go_test(
     name = "split_test",
+    timeout = "short",
     srcs = ["sum_sorted_test.go"],
+    flaky = True,
     deps = [
         ":split",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/restore/tiflashrec/BUILD.bazel b/br/pkg/restore/tiflashrec/BUILD.bazel
@@ -19,6 +19,7 @@ go_library(
 
 go_test(
     name = "tiflashrec_test",
+    timeout = "short",
     srcs = ["tiflash_recorder_test.go"],
     flaky = True,
     deps = [

diff --git a/br/pkg/streamhelper/daemon/BUILD.bazel b/br/pkg/streamhelper/daemon/BUILD.bazel
@@ -18,6 +18,7 @@ go_library(
 
 go_test(
     name = "daemon_test",
+    timeout = "short",
     srcs = ["owner_daemon_test.go"],
     flaky = True,
     deps = [

diff --git a/br/pkg/streamhelper/spans/BUILD.bazel b/br/pkg/streamhelper/spans/BUILD.bazel
@@ -19,11 +19,13 @@ go_library(
 
 go_test(
     name = "spans_test",
+    timeout = "short",
     srcs = [
         "sorted_test.go",
         "utils_test.go",
         "value_sorted_test.go",
     ],
+    flaky = True,
     deps = [
         ":spans",
         "@com_github_stretchr_testify//require",

diff --git a/br/pkg/utils/BUILD.bazel b/br/pkg/utils/BUILD.bazel
@@ -81,7 +81,7 @@ go_test(
     ],
     embed = [":utils"],
     flaky = True,
-    shard_count = 20,
+    shard_count = 30,
     deps = [
         "//br/pkg/errors",
         "//br/pkg/metautil",

diff --git a/br/pkg/utils/iter/BUILD.bazel b/br/pkg/utils/iter/BUILD.bazel
@@ -20,6 +20,7 @@ go_library(
 
 go_test(
     name = "iter_test",
+    timeout = "short",
     srcs = ["combinator_test.go"],
     flaky = True,
     race = "on",

diff --git a/config/BUILD.bazel b/config/BUILD.bazel
@@ -37,6 +37,7 @@ go_test(
     data = glob(["**"]),
     embed = [":config"],
     flaky = True,
+    shard_count = 2,
     deps = [
         "//testkit/testsetup",
         "//util/logutil",

diff --git a/ddl/fktest/BUILD.bazel b/ddl/fktest/BUILD.bazel
@@ -2,6 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "fktest_test",
+    timeout = "short",
     srcs = [
         "foreign_key_test.go",
         "main_test.go",

diff --git a/ddl/ingest/BUILD.bazel b/ddl/ingest/BUILD.bazel
@@ -46,6 +46,7 @@ go_library(
 
 go_test(
     name = "ingest_test",
+    timeout = "short",
     srcs = [
         "env_test.go",
         "mem_root_test.go",

diff --git a/ddl/internal/callback/BUILD.bazel b/ddl/internal/callback/BUILD.bazel
@@ -17,33 +17,10 @@ go_library(
 
 go_test(
     name = "callback_test",
+    timeout = "short",
     srcs = ["callback_test.go"],
     embed = [":callback"],
-    deps = [
-        "//ddl",
-        "@com_github_stretchr_testify//require",
-    ],
-)
-
-go_library(
-    name = "ddlcallback",
-    srcs = ["callback.go"],
-    importpath = "github.com/pingcap/tidb/ddl/internal/ddlcallback",
-    visibility = ["//ddl:__subpackages__"],
-    deps = [
-        "//ddl",
-        "//infoschema",
-        "//parser/model",
-        "//sessionctx",
-        "//util/logutil",
-        "@org_uber_go_zap//:zap",
-    ],
-)
-
-go_test(
-    name = "ddlcallback_test",
-    srcs = ["callback_test.go"],
-    embed = [":ddlcallback"],
+    flaky = True,
     deps = [
         "//ddl",
         "@com_github_stretchr_testify//require",

diff --git a/ddl/metadatalocktest/BUILD.bazel b/ddl/metadatalocktest/BUILD.bazel
@@ -2,6 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "metadatalocktest_test",
+    timeout = "short",
     srcs = [
         "main_test.go",
         "mdl_test.go",

diff --git a/ddl/resourcegroup/BUILD.bazel b/ddl/resourcegroup/BUILD.bazel
@@ -17,8 +17,10 @@ go_library(
 
 go_test(
     name = "resourcegroup_test",
+    timeout = "short",
     srcs = ["group_test.go"],
     embed = [":resourcegroup"],
+    flaky = True,
     deps = [
         "//parser/model",
         "@com_github_pingcap_kvproto//pkg/resource_manager",

diff --git a/ddl/syncer/BUILD.bazel b/ddl/syncer/BUILD.bazel
@@ -22,6 +22,7 @@ go_library(
 
 go_test(
     name = "syncer_test",
+    timeout = "short",
     srcs = ["syncer_test.go"],
     flaky = True,
     deps = [

diff --git a/distsql/BUILD.bazel b/distsql/BUILD.bazel
@@ -64,6 +64,7 @@ go_test(
     embed = [":distsql"],
     flaky = True,
     race = "on",
+    shard_count = 2,
     deps = [
         "//kv",
         "//parser/charset",

diff --git a/executor/aggfuncs/BUILD.bazel b/executor/aggfuncs/BUILD.bazel
@@ -89,7 +89,7 @@ go_test(
     embed = [":aggfuncs"],
     flaky = True,
     race = "on",
-    shard_count = 25,
+    shard_count = 40,
     deps = [
         "//expression",
         "//expression/aggregation",

diff --git a/executor/autoidtest/BUILD.bazel b/executor/autoidtest/BUILD.bazel
@@ -2,6 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "autoidtest_test",
+    timeout = "short",
     srcs = [
         "autoid_test.go",
         "main_test.go",

diff --git a/executor/issuetest/BUILD.bazel b/executor/issuetest/BUILD.bazel
@@ -2,6 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "issuetest_test",
+    timeout = "short",
     srcs = [
         "executor_issue_test.go",
         "main_test.go",

diff --git a/executor/loadremotetest/BUILD.bazel b/executor/loadremotetest/BUILD.bazel
@@ -2,11 +2,13 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "loadremotetest_test",
+    timeout = "short",
     srcs = [
         "main_test.go",
         "one_csv_test.go",
         "util_test.go",
     ],
+    flaky = True,
     deps = [
         "//executor",
         "//kv",

diff --git a/executor/writetest/BUILD.bazel b/executor/writetest/BUILD.bazel
@@ -2,6 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "writetest_test",
+    timeout = "short",
     srcs = [
         "main_test.go",
         "write_test.go",

diff --git a/extension/BUILD.bazel b/extension/BUILD.bazel
@@ -29,6 +29,7 @@ go_library(
 
 go_test(
     name = "extension_test",
+    timeout = "short",
     srcs = [
         "bootstrap_test.go",
         "event_listener_test.go",
@@ -37,6 +38,7 @@ go_test(
         "registry_test.go",
     ],
     embed = [":extension"],
+    flaky = True,
     deps = [
         "//expression",
         "//parser/ast",

diff --git a/parser/duration/BUILD.bazel b/parser/duration/BUILD.bazel
@@ -10,7 +10,9 @@ go_library(
 
 go_test(
     name = "duration_test",
+    timeout = "short",
     srcs = ["duration_test.go"],
     embed = [":duration"],
+    flaky = True,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/resourcemanager/pooltask/BUILD.bazel b/resourcemanager/pooltask/BUILD.bazel
@@ -18,7 +18,9 @@ go_library(
 
 go_test(
     name = "pooltask_test",
+    timeout = "short",
     srcs = ["task_test.go"],
     embed = [":pooltask"],
+    flaky = True,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/resourcemanager/util/BUILD.bazel b/resourcemanager/util/BUILD.bazel
@@ -18,8 +18,10 @@ go_library(
 
 go_test(
     name = "util_test",
+    timeout = "short",
     srcs = ["shard_pool_map_test.go"],
     embed = [":util"],
+    flaky = True,
     deps = [
         "//util/intest",
         "@com_github_stretchr_testify//require",

diff --git a/sessiontxn/BUILD.bazel b/sessiontxn/BUILD.bazel
@@ -20,7 +20,7 @@ go_library(
 
 go_test(
     name = "sessiontxn_test",
-    timeout = "moderate",
+    timeout = "short",
     srcs = [
         "txn_context_test.go",
         "txn_manager_test.go",

diff --git a/store/copr/copr_test/BUILD.bazel b/store/copr/copr_test/BUILD.bazel
@@ -2,10 +2,12 @@ load("@io_bazel_rules_go//go:def.bzl", "go_test")
 
 go_test(
     name = "copr_test_test",
+    timeout = "short",
     srcs = [
         "coprocessor_test.go",
         "main_test.go",
     ],
+    flaky = True,
     deps = [
         "//config",
         "//kv",

diff --git a/tablecodec/BUILD.bazel b/tablecodec/BUILD.bazel
@@ -36,6 +36,7 @@ go_test(
     ],
     embed = [":tablecodec"],
     flaky = True,
+    shard_count = 30,
     deps = [
         "//kv",
         "//parser/mysql",

diff --git a/telemetry/BUILD.bazel b/telemetry/BUILD.bazel
@@ -51,7 +51,7 @@ go_library(
 
 go_test(
     name = "telemetry_test",
-    timeout = "moderate",
+    timeout = "short",
     srcs = [
         "data_cluster_hardware_test.go",
         "data_feature_usage_test.go",

diff --git a/ttl/client/BUILD.bazel b/ttl/client/BUILD.bazel
@@ -23,6 +23,7 @@ go_test(
     timeout = "short",
     srcs = ["command_test.go"],
     embed = [":client"],
+    flaky = True,
     shard_count = 5,
     deps = [
         "@com_github_pingcap_errors//:errors",

diff --git a/ttl/metrics/BUILD.bazel b/ttl/metrics/BUILD.bazel
@@ -16,6 +16,7 @@ go_test(
     timeout = "short",
     srcs = ["metrics_test.go"],
     embed = [":metrics"],
+    flaky = True,
     shard_count = 5,
     deps = ["@com_github_stretchr_testify//require"],
 )
diff --git a/ttl/ttlworker/BUILD.bazel b/ttl/ttlworker/BUILD.bazel
@@ -44,7 +44,7 @@ go_library(
 
 go_test(
     name = "ttlworker_test",
-    timeout = "moderate",
+    timeout = "short",
     srcs = [
         "del_test.go",
         "job_manager_integration_test.go",

diff --git a/util/cgroup/BUILD.bazel b/util/cgroup/BUILD.bazel
@@ -21,6 +21,7 @@ go_library(
 
 go_test(
     name = "cgroup_test",
+    timeout = "short",
     srcs = [
         "cgroup_cpu_test.go",
         "cgroup_mock_test.go",

diff --git a/util/cpu/BUILD.bazel b/util/cpu/BUILD.bazel
@@ -18,6 +18,7 @@ go_library(
 
 go_test(
     name = "cpu_test",
+    timeout = "short",
     srcs = ["cpu_test.go"],
     embed = [":cpu"],
     flaky = True,

diff --git a/util/gctuner/BUILD.bazel b/util/gctuner/BUILD.bazel
@@ -20,6 +20,7 @@ go_library(
 
 go_test(
     name = "gctuner_test",
+    timeout = "short",
     srcs = [
         "finalizer_test.go",
         "mem_test.go",