Skip to content

Commit a59d7ab

Browse files
authored
Fix and simplify perf usage with the test infra (#7160)
1 parent cba9a90 commit a59d7ab

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+54
-126
lines changed

tests/acme_endorsement.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def wait_for_certificates(
5151
args, network_name, root_cert, interface_name, challenge_interface, timeout=5
5252
):
5353
with infra.network.network(
54-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
54+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
5555
) as network:
5656
network.start_and_open(args)
5757

@@ -427,7 +427,7 @@ def run_unsecured(args):
427427
node.rpc_interfaces["unsecured_interface"] = challenge_server_interface
428428

429429
with infra.network.network(
430-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
430+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
431431
) as network:
432432
network.start_and_open(args)
433433
test_unsecured_interfaces(network, "secured_interface", "unsecured_interface")

tests/client_protocols.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ def test_http2(network, args):
135135

136136
def run(args):
137137
with infra.network.network(
138-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
138+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
139139
) as network:
140140
network.start_and_open(args)
141141
test_tls(network, args)
@@ -145,7 +145,7 @@ def run(args):
145145
args.http2 = True
146146
args.nodes = infra.e2e_args.nodes(args, 1)
147147
with infra.network.network(
148-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
148+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
149149
) as network:
150150
network.start_and_open(args)
151151
test_http2(network, args)

tests/code_update.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -874,7 +874,7 @@ def test_add_node_with_no_uvm_endorsements_in_kv(network, args):
874874

875875
def run(args):
876876
with infra.network.network(
877-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
877+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
878878
) as network:
879879
network.start_and_open(args)
880880

tests/commit_latency.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ def measure_commit_latency(args, sig_interval=100):
7979

8080
times = []
8181
with infra.network.network(
82-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
82+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
8383
) as network:
8484
network.start_and_open(args)
8585

tests/committable.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def run(args):
4040
hosts = ["local://localhost"] * 5
4141

4242
with infra.network.network(
43-
hosts, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
43+
hosts, args.binary_dir, args.debug_nodes, pdb=args.pdb
4444
) as network:
4545
network.start_and_open(args)
4646
primary, backups = network.find_nodes()

tests/connections.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ def run_connection_caps_tests(args):
7272
args.ubsan_options = "suppressions=" + str(supp_file)
7373

7474
with infra.network.network(
75-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
75+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
7676
) as network:
7777
check = infra.checker.Checker()
7878
network.start_and_open(args)
@@ -288,7 +288,7 @@ def verbose_sleep(sleep_time):
288288
args.idle_connection_timeout_s = timeout
289289

290290
with infra.network.network(
291-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
291+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
292292
) as network:
293293
network.start_and_open(args)
294294

@@ -343,7 +343,7 @@ def node_tcp_socket(node):
343343
# NB: This does rudimentary smoke testing. See fuzzing.py for more thorough test
344344
def run_node_socket_robustness_tests(args):
345345
with infra.network.network(
346-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
346+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
347347
) as network:
348348
network.start_and_open(args)
349349

tests/consistency_trace_validation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
def run(args):
1515
with infra.network.network(
16-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
16+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
1717
) as network:
1818
network.start_and_open(args)
1919
targets = [

tests/e2e_batched.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def test(network, args, batch_size=100, write_key_divisor=1, write_size_multipli
5959

6060
def run(args):
6161
with infra.network.network(
62-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
62+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
6363
) as network:
6464
network.start_and_open(args)
6565

@@ -91,7 +91,7 @@ def run(args):
9191

9292
def run_to_destruction(args):
9393
with infra.network.network(
94-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
94+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
9595
) as network:
9696
network.start_and_open(args)
9797

tests/e2e_common_endpoints.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ def get_sizes(n, http2):
312312

313313
def run(args):
314314
with infra.network.network(
315-
args.nodes, args.binary_dir, args.debug_nodes, args.perf_nodes, pdb=args.pdb
315+
args.nodes, args.binary_dir, args.debug_nodes, pdb=args.pdb
316316
) as network:
317317
network.start_and_open(args)
318318

tests/e2e_logging.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2150,7 +2150,6 @@ def run_udp_tests(args):
21502150
args.nodes,
21512151
args.binary_dir,
21522152
args.debug_nodes,
2153-
args.perf_nodes,
21542153
pdb=args.pdb,
21552154
txs=txs,
21562155
) as network:
@@ -2179,7 +2178,6 @@ def additional_interfaces(local_node_id):
21792178
args.nodes,
21802179
args.binary_dir,
21812180
args.debug_nodes,
2182-
args.perf_nodes,
21832181
pdb=args.pdb,
21842182
txs=txs,
21852183
) as network:
@@ -2194,7 +2192,6 @@ def run_app_space_js(args):
21942192
args.nodes,
21952193
args.binary_dir,
21962194
args.debug_nodes,
2197-
args.perf_nodes,
21982195
pdb=args.pdb,
21992196
txs=txs,
22002197
) as network:
@@ -2315,7 +2312,6 @@ def run_parsing_errors(args):
23152312
args.nodes,
23162313
args.binary_dir,
23172314
args.debug_nodes,
2318-
args.perf_nodes,
23192315
pdb=args.pdb,
23202316
txs=txs,
23212317
) as network:

0 commit comments

Comments
 (0)