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 for ecn counter increment for ECT marked packets #16356

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sreejithsreekumaran
Copy link
Contributor

@sreejithsreekumaran sreejithsreekumaran commented Jan 6, 2025

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Verify if ingress congestion experienced markets pkts (ECT bit b11) are causing the ecn counter increment on the DUT as expected on congestion

How did you do it?

Wrote snappi test to cause congestion on the port

How did you verify/test it?

Injected frames with ECT bit set and caused congestion on the egress port.

Verified that the DUT is counting the ECN marking and the counter is incrementing.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sreejithsreekumaran
Copy link
Contributor Author

T0/T1 results

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============================================================================================================================================ PASSES =============================================================================================================================================
_______________________________________________________________________________________________________________________ test_ecn_marking_port_toggle[multidut_port_info0] _______________________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent0] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent1] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent2] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent3] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent4] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent5] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent6] _____________________________________________________________________________________________________________
____________________________________________________________________________________________________________ test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent7] _____________________________________________________________________________________________________________
_____________________________________________________________________________________________________________________ test_ecn_marking_ect_marked_pkts[multidut_port_info0] _____________________________________________________________________________________________________________________
---------------------------------------------------------------------------------------------------- generated xml file: /run_logs/ixia/19154/2025-01-06-11-28-56/tr_2025-01-06-11-28-56.xml ----------------------------------------------------------------------------------------------------
INFO:root:Can not get Allure report URL. Please check logs
------------------------------------------------------------------------------------------------------------------------------------ live log sessionfinish -------------------------------------------------------------------------------------------------------------------------------------
11:48:35 __init__.pytest_terminal_summary         L0067 INFO   | Can not get Allure report URL. Please check logs
==================================================================================================================================== short test summary info ====================================================================================================================================
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_port_toggle[multidut_port_info0]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent0]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent1]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent2]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent3]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent4]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent5]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent6]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_lossless_prio[multidut_port_info0-test_flow_percent7]
PASSED snappi_tests/multidut/ecn/test_multidut_ecn_marking_with_snappi.py::test_ecn_marking_ect_marked_pkts[multidut_port_info0]
========================================================================================================================= 10 passed, 13 warnings in 1177.08s (0:19:37) ==========================================================================================================================

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.

2 participants