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 watermarkstat CLI script for unconfigured PG counters #2220

Closed
wants to merge 4 commits into from
Closed

Fix watermarkstat CLI script for unconfigured PG counters #2220

wants to merge 4 commits into from

Commits on Jun 14, 2022

  1. Fix watermarkstat CLI script.

    Since PG/Queue counters are created only if they are configured in the switch, it is not enough to relay only on the first entry in the DB.
    We need to go over all configured counters, check what is the max configured, and build the table accordingly.
    
    Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
    Shlomi Bitton committed Jun 14, 2022
    Configuration menu
    Copy the full SHA
    409da5f View commit details
    Browse the repository at this point in the history

Commits on Jun 15, 2022

  1. Fix index issue

    Shlomi Bitton committed Jun 15, 2022
    Configuration menu
    Copy the full SHA
    a6b6d38 View commit details
    Browse the repository at this point in the history
  2. Fix indentation issue

    Shlomi Bitton committed Jun 15, 2022
    Configuration menu
    Copy the full SHA
    4741bf6 View commit details
    Browse the repository at this point in the history
  3. Fix min value for table header

    Shlomi Bitton committed Jun 15, 2022
    Configuration menu
    Copy the full SHA
    e67e683 View commit details
    Browse the repository at this point in the history