[fixit] Scale down large tests (#30681)

* [fixit] Scale down large tests

We have many tests that create 100 threads or more, and mounting evidence that
this is harmful to our CI environment.

When the original code for many of these tests was written we ran our tests
under run_tests, which had explicit handling for tracking the number of threads
each test needed and making sure that we weren't over subscribing the test
runner. Bazel has no such facility (and the facility in run_tests has since
been removed) and so we need to adjust.

This PR adjusts down a single test and is part of a series so that we can
review and roll back easily if required.

* mark up cpu usage
pull/30696/head
Craig Tiller 2 years ago committed by GitHub
parent 686345acb5
commit 18deacbdd1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 1
      test/core/iomgr/BUILD
  2. 2
      test/core/iomgr/stranded_event_test.cc

@ -350,6 +350,7 @@ grpc_cc_test(
# TODO(apolcyn): This test is failing on Windows at entry, enable once passing.
# See e.g. https://source.cloud.google.com/results/invocations/6716596a-c9e1-4780-85ed-890d8758d582/targets
"no_windows",
"cpu:10",
],
deps = [
"//:gpr",

@ -316,7 +316,7 @@ TEST(Pollers, TestReadabilityNotificationsDontGetStrandedOnOneCq) {
/* 64 is a somewhat arbitary number, the important thing is that it
* exceeds the value of MAX_EPOLL_EVENTS_HANDLED_EACH_POLL_CALL (16), which
* is enough to repro a bug at time of writing. */
const int kNumCalls = 64;
const int kNumCalls = 32;
size_t ping_pong_round = 0;
size_t ping_pongs_done = 0;
grpc_core::Mutex ping_pong_round_mu;

Loading…
Cancel
Save