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

threshold feature #1313

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

Conversation

PrabhuSreenivasan
Copy link
Contributor

@PrabhuSreenivasan PrabhuSreenivasan commented Dec 16, 2020

Signed-off-by: Prabhu Sreenivasan prabhu.sreenivasan@broadcom.com

Threshold feature support
changes extracted from #665

- What I did
Added CLI commands for threshold feature

- How I did it
Added config, show and clear commands

- How to verify it

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

- Related PRs -
sonic-net/sonic-swss-common#433
sonic-net/sonic-swss#1545
Azure/sonic-tam#13
sonic-net/sonic-buildimage#6222

Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2021

This pull request introduces 7 alerts when merging 1774da7 into c7c01e4 - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 2 for Wrong name for an argument in a class instantiation

Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com>
Signed-off-by: Prabhu Sreenivasan <prabhu.sreenivasan@broadcom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants