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

Count of ECN packets from Stats instead of counting from packet capture #541

Open
ANISH-GOTTAPU opened this issue Jun 8, 2022 · 3 comments

Comments

@ANISH-GOTTAPU
Copy link
Contributor

Is your feature request related to a use case? Please describe.
As of now in the test https://github.com/Azure/sonic-mgmt/blob/master/tests/snappi/ecn/test_dequeue_ecn_with_snappi.py , the count of ECN packets is calculated by capturing the packets and check for ECN bit.

The test works for 100 packets, but for scaling i.e capturing millions of packets has it's own limitations.

  1. Increase in process time.
  2. Hardware Limitation of packet capture on cards.

Describe the solution you'd like
A better way to calculate the count instead of capture (using custom stats filter).

@alpeshspatel
Copy link

Anish,
Additionally, we need the ability to clear the counter

@ANISH-GOTTAPU
Copy link
Contributor Author

@alpeshspatel : Clear counters is part of start transmit, if you start transmit on a flow the counters get reset and starts from 0 whereas clearing the counters during transmit is not available.

Can you explain the use case for clearing the counters during the transmit?

@alpeshspatel
Copy link

Anish - we should be ok if the counters are cleared on start of a flow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Not Started
Development

No branches or pull requests

2 participants