Conversation
There was a problem hiding this comment.
Pull Request Overview
This pull request adds stress testing capabilities to the project by introducing the pytest-repeat plugin and a new GitHub Actions workflow for automated stress testing.
Key Changes:
- Added
pytest-repeat>=0.9.0as a development dependency to enable test repetition - Created a new
.github/workflows/stress.ymlworkflow for scheduled and manual stress testing - Configured the workflow to run tests 100 times by default with parallelization support
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
pyproject.toml |
Added pytest-repeat to dev dependencies for test repetition functionality |
uv.lock |
Lock file updated with pytest-repeat 0.9.4 package metadata and dependencies |
.github/workflows/stress.yml |
New workflow for running stress tests with configurable repeat count and parallel workers, supporting multiple Python versions and operating systems |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| jobs: | ||
| stress-test: | ||
| name: ${{ matrix.project }} stress test (python-${{ matrix.python-version }}, nats-server-${{ matrix.nats-server-version }}, ${{ matrix.os }}) | ||
| runs-on: ${{ matrix.os }} |
There was a problem hiding this comment.
The stress test job lacks a timeout setting. Running 100 test iterations (the default count) could potentially take a very long time, especially on slower runners. Consider adding a timeout-minutes setting to the job to prevent it from running indefinitely if tests hang or take unexpectedly long. For reference, the regular test workflow uses timeout-minutes: 20.
| runs-on: ${{ matrix.os }} | |
| runs-on: ${{ matrix.os }} | |
| timeout-minutes: 20 |
7e9b4ac to
5306e8b
Compare
Signed-off-by: Casper Beyer <casper@synadia.com>
5306e8b to
b0e1589
Compare
Signed-off-by: Casper Beyer <casper@synadia.com>
No description provided.