msquic icon indicating copy to clipboard operation
msquic copied to clipboard

Standard SecNetPerf Scenarios

Open nibanks opened this issue 1 year ago • 1 comments

Description

Adds a new secnetperf cmd line arg -scenario: which standardizes a few common test cases. This then simplifies the secnetperf scripts for netperf to just use these scenarios.

Testing

CI/CD

Documentation

Updated help text

nibanks avatar Oct 10 '24 15:10 nibanks

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.87%. Comparing base (c8d66e8) to head (410e492). Report is 35 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4607      +/-   ##
==========================================
- Coverage   86.98%   85.87%   -1.12%     
==========================================
  Files          56       56              
  Lines       17354    17354              
==========================================
- Hits        15096    14903     -193     
- Misses       2258     2451     +193     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 11 '24 12:10 codecov[bot]

@ProjectsByJackHe where are we on this? Is netperf ready to merge this?

nibanks avatar Nov 06 '24 10:11 nibanks

@ProjectsByJackHe where are we on this? Is netperf ready to merge this?

Doing some local dry runs to ensure these changes do not break compatibility.

Tracking: https://github.com/microsoft/netperf/issues/401

ProjectsByJackHe avatar Nov 11 '24 19:11 ProjectsByJackHe

@ProjectsByJackHe where are we on this? Is netperf ready to merge this?

Doing some local dry runs to ensure these changes do not break compatibility.

Tracking: microsoft/netperf#401

Almost ready for merge! Once https://github.com/microsoft/netperf/pull/414 gets merged first, validation passes, then this PR is ready for merge.

ProjectsByJackHe avatar Nov 19 '24 02:11 ProjectsByJackHe