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

test: overlay requests/second onto load test graph #1470

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

btlghrants
Copy link
Contributor

Description

Based on some team conversation, this PR adds a plot of the "requests per second" on top of the watcher load test memory graph.

Related Issue

Relates to #1431

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@btlghrants btlghrants marked this pull request as ready for review November 21, 2024 16:21
@btlghrants btlghrants requested a review from a team as a code owner November 21, 2024 16:21
@btlghrants btlghrants self-assigned this Nov 21, 2024
Copy link

codecov bot commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.55%. Comparing base (beb40a9) to head (3209e0a).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1470   +/-   ##
=======================================
  Coverage   78.55%   78.55%           
=======================================
  Files          36       36           
  Lines        1716     1716           
  Branches      411      410    -1     
=======================================
  Hits         1348     1348           
  Misses        339      339           
  Partials       29       29           
---- 🚨 Try these New Features:

@btlghrants btlghrants added this to the v0.41.0 milestone Nov 21, 2024
@cmwylie19 cmwylie19 enabled auto-merge November 21, 2024 16:27
@cmwylie19 cmwylie19 added this pull request to the merge queue Nov 21, 2024
Merged via the queue into main with commit 3d6c811 Nov 21, 2024
46 checks passed
@cmwylie19 cmwylie19 deleted the load_test_req_per_sec branch November 21, 2024 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants