Skip to content

feat(test-suite): gw reliable benchmark #923

Merged
eudelins-zama merged 4 commits intomainfrom
simon/feat/gateway-test-tool-bench-mode
Oct 2, 2025
Merged

feat(test-suite): gw reliable benchmark #923
eudelins-zama merged 4 commits intomainfrom
simon/feat/gateway-test-tool-bench-mode

Conversation

@eudelins-zama
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements a reliable benchmarking feature for the gateway stress test tool. The benchmark functionality can run multiple decryption bursts with configurable parameters and collect performance metrics.

  • Adds a new benchmark CLI subcommand that takes CSV input files to define test parameters
  • Implements statistical analysis (mean, standard deviation) of latency and throughput metrics
  • Refactors existing burst functions to return structured performance results instead of just logging

Reviewed Changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test-suite/gateway-stress/templates/small_bench.csv Template CSV file with small benchmark configuration
test-suite/gateway-stress/templates/bench.csv Template CSV file with comprehensive benchmark configuration
test-suite/gateway-stress/src/main.rs Adds benchmark module import and subcommand handling
test-suite/gateway-stress/src/decryption/user.rs Refactors user decryption to return performance metrics
test-suite/gateway-stress/src/decryption/public.rs Refactors public decryption to return performance metrics
test-suite/gateway-stress/src/decryption/mod.rs Adds BurstResult struct for performance data
test-suite/gateway-stress/src/cli.rs Adds benchmark subcommand and arguments structure
test-suite/gateway-stress/src/bench.rs New module implementing benchmark data structures and statistics
test-suite/gateway-stress/src/app.rs Implements benchmark execution logic and refactors progress bar initialization
test-suite/gateway-stress/config/config.toml Updates configuration values for testing
test-suite/gateway-stress/README.md Documents the new benchmarking functionality
test-suite/gateway-stress/Cargo.toml Adds csv dependency for benchmark file processing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread test-suite/gateway-stress/src/bench.rs
Comment thread test-suite/gateway-stress/src/bench.rs
Comment thread test-suite/gateway-stress/src/decryption/public.rs
@eudelins-zama eudelins-zama self-assigned this Sep 29, 2025
@eudelins-zama eudelins-zama force-pushed the simon/feat/gateway-test-tool-bench-mode branch from b9651d7 to e3d0afe Compare September 29, 2025 15:04
@eudelins-zama eudelins-zama force-pushed the simon/feat/gateway-test-tool-bench-mode branch from e3d0afe to c2d73f2 Compare September 29, 2025 15:36
@eudelins-zama eudelins-zama marked this pull request as ready for review September 30, 2025 07:09
Copy link
Copy Markdown
Collaborator

@leventdem leventdem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK!

@eudelins-zama eudelins-zama merged commit 3c45b4e into main Oct 2, 2025
101 checks passed
@eudelins-zama eudelins-zama deleted the simon/feat/gateway-test-tool-bench-mode branch October 2, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants