Skip to content

Commit 81dd0bf

Browse files
[Nexthop] Bump cmd_config_test to size=large
The suite is ~60 files of ConfigSession-backed tests, each doing several fsync-backed writeFileAtomic calls per case; with this PR's added peer-group tests the full run takes ~530s on a dev box and timed out at Bazel's default moderate limit (300s) in the bazel-build CI job — killed mid-fsync in ConfigSessionTestFixture.commandTrackingMultipleCommands, 50/51 remaining tests green. size=large raises the test timeout to 900s (same precedent as thrift_packet_stream/tests). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 4f236b9 commit 81dd0bf

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

  • fboss/cli/fboss2/test/config

fboss/cli/fboss2/test/config/BUCK

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ oncall("fboss_oss")
55

66
cpp_unittest(
77
name = "cmd_config_test",
8+
# The suite is ~60 files of ConfigSession tests, each doing several
9+
# fsync-backed writeFileAtomic calls; on CI disks the run can exceed the
10+
# default "moderate" (300s) Bazel test timeout.
11+
size = "large",
812
srcs = [
913
"BgpConfigSessionTest.cpp",
1014
"CmdConfigAclRuleTest.cpp",

0 commit comments

Comments
 (0)