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

Fix SNMP output having fewer unicast queues than expected #330

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Commits on Sep 5, 2024

  1. Fix SNMP output having fewer unicast queues than expected

    SNMP assumes the first half of the queues to be unicast and the second half to be multicast.
    This assumption was fine for most hwSkus but not new ones being added to
    support, i.e. x86_64-arista_7060x6
    
    Only tested on Arista hwSkus.
    justin-wong-ce committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    bf9207b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    664cd6b View commit details
    Browse the repository at this point in the history

Commits on Oct 9, 2024

  1. Added unit test to check #UC > half total queue behavior

    Added a unit test subcase and relevant mock tablesto check for SNMP
    behavior when there are more unicast queues than half of the total
    number of queues.
    
    This is to ensure SNMP does not operate on the false assumption where
    there is always a 50-50 split between unicast and multicast queues.
    justin-wong-ce committed Oct 9, 2024
    Configuration menu
    Copy the full SHA
    6f8f173 View commit details
    Browse the repository at this point in the history