Skip to content

Commit 1dc5fc3

Browse files
committed
chore: cleanup snapshot test code
Do not unnecessarily wrap return value of snapshot latency producer into dictionary. Signed-off-by: Patrick Roy <[email protected]>
1 parent 3a351b1 commit 1dc5fc3

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

tests/integration_tests/performance/test_snapshot_restore_performance.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@ def default_lambda_consumer(env_id, workload):
7979
SnapRestoreBaselinesProvider(env_id, workload, raw_baselines),
8080
),
8181
func=consume_output,
82-
func_kwargs={},
8382
)
8483

8584

@@ -174,13 +173,12 @@ def get_snap_restore_latency(
174173

175174
full_snapshot.cleanup()
176175
vm.jailer.cleanup()
177-
return {RESTORE_LATENCY: values}
176+
return values
178177

179178

180-
def consume_output(cons, result):
179+
def consume_output(cons, latencies):
181180
"""Consumer function."""
182-
restore_latency = result[RESTORE_LATENCY]
183-
for value in restore_latency:
181+
for value in latencies:
184182
yield RESTORE_LATENCY, value, "Milliseconds"
185183
cons.consume_data(RESTORE_LATENCY, value)
186184

0 commit comments

Comments
 (0)