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

[counterpoll] Add new FC group for port buffer drop counters. #1024

Merged
merged 4 commits into from
Aug 28, 2020

Conversation

mykolaf
Copy link
Collaborator

@mykolaf mykolaf commented Aug 3, 2020

Signed-off-by: Mykola Faryma [email protected]

Add new FC group for port buffer drop counters.

See sonic-net/sonic-buildimage#5093
- What I did

- How I did it

- 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)

@mykolaf

This comment has been minimized.

@mykolaf mykolaf marked this pull request as ready for review August 10, 2020 07:37
@mykolaf

This comment has been minimized.

@mykolaf
Copy link
Collaborator Author

mykolaf commented Aug 11, 2020

retest this please

1 similar comment
@lguohan
Copy link
Contributor

lguohan commented Aug 12, 2020

retest this please

@lguohan
Copy link
Contributor

lguohan commented Aug 12, 2020

need unit test for the newly added feature.

Copy link
Contributor

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add unit coverage for the new feature added.

@daall
Copy link
Contributor

daall commented Aug 20, 2020

@mykolaf can this be cherry-picked in via #1009?

@mykolaf
Copy link
Collaborator Author

mykolaf commented Aug 21, 2020

@mykolaf can this be cherry-picked in via #1009?

@daall No, due to rename of sonic-utilities-tests -> test I believe this does not apply cleanly. Or if it applies, the mock_tables is only updated in the old folder.

Volodymyr Samotiy added 2 commits August 27, 2020 14:40
* Fix review comments

Signed-off-by: Volodymyr Samotiy <[email protected]>
…o b201911

Signed-off-by: Volodymyr Samotiy <[email protected]>

Conflicts:
	sonic-utilities-tests/mock_tables/config_db.json
@lgtm-com
Copy link

lgtm-com bot commented Aug 27, 2020

This pull request introduces 5 alerts when merging a9563de into b0ac781 - view on LGTM.com

new alerts:

  • 5 for Unused import

@daall daall merged commit ddf84d9 into sonic-net:201911 Aug 28, 2020
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Fix error when running 'show lldp table' or 'show lldp neighbor'
through SSH command. (sonic-net#1067)
[201911]: Multi asic show interface support (sonic-net#1070)
[counterpoll] Add new FC group for port buffer drop counters
(sonic-net#1024)
[201911] show interface portchannel support for Multi ASIC
(sonic-net#1071)
Fix a typo in mellanox_buffer_migrator (sonic-net#1090)
Fix pfcwd stats crash with invalid queue name (sonic-net#1077)
[PFCWD] Fix issue with "pfcwd show stats" command during SONiC init
(sonic-net#1018)
enable watchdog before running platform specific reboot plugin
(sonic-net#1037)
Add namespace of the process in the coredump filename. (sonic-net#1091)
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.

3 participants