Skip to content
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

Aggregator stress test refactoring #1186

Merged
merged 3 commits into from Aug 29, 2023

Conversation

jpraynaud
Copy link
Member

@jpraynaud jpraynaud commented Aug 29, 2023

Content

This PR includes a refactoring of the aggregator stress test tool: we have created a stress_test sub-module to make the code easier to understand and to maintain

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Issue(s)

Relates to #1155

@jpraynaud jpraynaud force-pushed the ensemble/1155-stress-test-refactoring branch from 4bf483c to eb46957 Compare August 29, 2023 16:05
@github-actions
Copy link

Test Results

    3 files  ±0    17 suites  ±0   5m 11s ⏱️ ±0s
671 tests ±0  671 ✔️ ±0  0 💤 ±0  0 ±0 
727 runs  ±0  727 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit eb46957. ± Comparison against base commit a2222e0.

@jpraynaud jpraynaud temporarily deployed to testing-preview August 29, 2023 16:21 — with GitHub Actions Inactive
@jpraynaud jpraynaud merged commit 9630459 into main Aug 29, 2023
27 checks passed
@jpraynaud jpraynaud deleted the ensemble/1155-stress-test-refactoring branch August 29, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants