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

[WIP] [SDFAB-512] Design testing framework for ASTF-mode TRex for linerate TCP testing #368

Open
wants to merge 31 commits into
base: main
Choose a base branch
from

Conversation

dariusgrassi
Copy link
Contributor

@dariusgrassi dariusgrassi commented Aug 26, 2021

  • Implement basic ASTF test class and ASTF HTTP profile
  • Create new TRex class for ASTFClient
  • Add optional --trex-astf-mode flag to activate TRex in ASTF mode

* Create new TRex class for ASTFClient
* Add astf flag option to activate TRex in ASTF mode
* Implement basic ASTF test class and ASTF HTTP profile
@dariusgrassi dariusgrassi self-assigned this Aug 26, 2021
@dariusgrassi dariusgrassi added the WIP Do not merge! label Aug 26, 2021
@codecov
Copy link

codecov bot commented Aug 26, 2021

Codecov Report

Merging #368 (36c0118) into main (2df880d) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 36c0118 differs from pull request most recent head 8aa46ef. Consider uploading reports for the commit 8aa46ef to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##               main     #368      +/-   ##
============================================
+ Coverage     67.09%   67.12%   +0.02%     
- Complexity      552      554       +2     
============================================
  Files            60       60              
  Lines          3951     3951              
  Branches        401      401              
============================================
+ Hits           2651     2652       +1     
  Misses         1080     1080              
+ Partials        220      219       -1     
Impacted Files Coverage Δ
...atumproject/fabric/tna/stats/StatisticDataKey.java 100.00% <0.00%> (+3.44%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2df880d...8aa46ef. Read the comment docs.

Darius Grassi and others added 3 commits August 26, 2021 16:49
* Remove `argparse` section of HTTP ASTF profile. This removes the
ability to call the profile from the CLI, but solves annoying bugs
related to passing the tunables from the test to the profile
* Remove `astf_path.py` file
* Fix small issues with test file
Darius Grassi and others added 16 commits September 2, 2021 15:15
* astf_tcp_flow: add switch port configuration and capture receive port
* http_eflow: match style of http_eflow profile instead of http_eflow2
* Pass astf param to start_stateless call in ptf_runner
* Combine ASTF profile and client code into one test file
* Create new ASTF base class not including any STL calls
* Revert trex-config changes
* Change to 1 core so there is only 1 flow generated (if 16 core, 16
parallel flows are created)
* Cleanup test file
statistics for a specified port every second
* Runtime is based on specified duration is test file
* Generate expected amount of traffic
* Use a multiplier/speedup of 1
* Only specify traffic rate as part of stream definition
* Only use doRunTest convention if multiple parameters are being tested
Darius Grassi added 5 commits September 30, 2021 14:25
* Split sanity test into two tests: unshaped 40G ports, and shaped 1G
egress port
* Add pydoc for both tests
* Normalize tests to match TRex INT test structure
* Add verification of average rates to tests
@dariusgrassi dariusgrassi removed the WIP Do not merge! label Oct 1, 2021
@dariusgrassi dariusgrassi changed the title [WIP] [SDFAB-512] Design testing framework for ASTF-mode TRex for linerate TCP testing [SDFAB-512] Design testing framework for ASTF-mode TRex for linerate TCP testing Oct 1, 2021
@onf-bot
Copy link
Collaborator

onf-bot commented Oct 28, 2021

Can one of the admins verify this patch? For help please reach out to [email protected]. The list of admins can be updated in the Jenkins configuration.

@ccascone ccascone added the 🍩 merge Do not merge label Jan 20, 2022
@ccascone ccascone changed the title [SDFAB-512] Design testing framework for ASTF-mode TRex for linerate TCP testing [WIP] [SDFAB-512] Design testing framework for ASTF-mode TRex for linerate TCP testing Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍩 merge Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants