-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat(test-suite): add stress testing via db #1016
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 expands the gateway stress testing tool to support stress testing via database connections in addition to the existing Gateway chain approach. It updates the tooling to work with fhevm v0.9.0 and restructures the CLI to offer four distinct commands for both stress testing and benchmarking through either Gateway chain or direct database operations.
Key changes include:
- Added database stress testing functionality by inserting decryption requests directly into Connectors' databases
- Updated from fhevm v0.8.0 to v0.9.0 with corresponding API changes
- Restructured CLI with four commands:
gw,bench-gw,db, andbench-db
Reviewed Changes
Copilot reviewed 24 out of 33 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
test-suite/gateway-stress/src/main.rs |
Updated main entry point to support new CLI structure and database testing modes |
test-suite/gateway-stress/src/cli.rs |
Completely restructured CLI with new subcommands and argument parsing |
test-suite/gateway-stress/src/config.rs |
Reorganized config structure to support both blockchain and database configurations |
test-suite/gateway-stress/src/db/ |
New module implementing database stress testing functionality |
test-suite/gateway-stress/src/decryption/ |
Updated to support fhevm v0.9.0 API changes and added shared types |
test-suite/gateway-stress/src/blockchain/ |
Refactored existing blockchain functionality into dedicated manager |
test-suite/gateway-stress/Cargo.toml |
Updated dependencies to fhevm v0.9.0 and added database dependencies |
Files not reviewed (7)
- test-suite/gateway-stress/.sqlx/query-2f49a66126dda8f3e5f043ad8fa119691568ca3216e1a04715aa02322bf3723d.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-6007239279928f6691a5284666e99fb6f020f20264c157500dbad47d7ec3dfa9.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-6471556ae0071cc8896a01ad0f2f350416bf00d6d617422422f2368f5ec7c826.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-7eb5ee37fa8e57c641712b895a5f59e0c484429e78626ccd5ad8b6d55a12267b.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-a9ac11a0896006a03fd4260810c31bf236ebe89054f4c7e981490c596799585b.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-affa510bdee616839e36215c598d07a20ca7af56c37fca94c0c1759dc2eba8ea.json: Language not supported
- test-suite/gateway-stress/.sqlx/query-d6597f8cda1d06ba8a5adedc650047a8646bfa1f3c666f2cf0f29257268b3542.json: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
ed0c8c1 to
8df0b68
Compare
ce1b29e to
9e255be
Compare
Closes https://github.com/zama-ai/kms-internal/issues/2751
Also adjust the tooling to the
0.9.0fhevmrelease.The CLI now offers 4 commands:
gw: stress testing by sending decryption requests to the Gateway chainbench-gw: benchmarking decryption by sending decryption requests to the Gateway chaindb: stress testing by inserting decryption requests directly to the Connectors' DBsbench-gw: benchmarking decryption by inserting decryption requests directly to the Connectors' DBsRun example: