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

[Mellanox] Update hw-management service config #2019

Merged
merged 1 commit into from
Sep 6, 2018

Conversation

andriymoroz-mlnx
Copy link
Collaborator

Signed-off-by: Andriy Moroz c_andriym@mellanox.com

This is a clone of #1914
It was merged to master earlier

Signed-off-by: Andriy Moroz <c_andriym@mellanox.com>
@lguohan
Copy link
Collaborator

lguohan commented Sep 5, 2018

vs test failure due to test script issue.

@lguohan
Copy link
Collaborator

lguohan commented Sep 5, 2018

retest this please

@lguohan lguohan merged commit be56470 into sonic-net:201807 Sep 6, 2018
vivekrnv pushed a commit to vivekrnv/sonic-buildimage that referenced this pull request Jan 18, 2022
d9f3afe [fdbshow] Adding more options for fdbshow and show mac (sonic-net#1982)
902e14f Revert "Revert "[Barefoot] Added CLI to list/set P4 profile (sonic-net#1951)"" (sonic-net#2019)
5cc9dd5 Revert "Revert "[sonic-package-manager] support sonic-cli-gen and packages with YANG model (sonic-net#1650)" (sonic-net#1972)" (sonic-net#1994)

Signed-off-by: Vivek Reddy Karri <vkarri@s-build-sonic-01.mts.labs.mlnx>
qiluo-msft pushed a commit that referenced this pull request Jan 19, 2022
```
d9f3afe [fdbshow] Adding more options for fdbshow and show mac (#1982)
902e14f Revert "Revert "[Barefoot] Added CLI to list/set P4 profile (#1951)"" (#2019)
5cc9dd5 Revert "Revert "[sonic-package-manager] support sonic-cli-gen and packages with YANG model (#1650)" (#1972)" (#1994)
```
prsunny pushed a commit that referenced this pull request Apr 21, 2022
Swss Commit update:

1fd1dbf Add support for route flow counter (#2094)
d8fadc6 [QoS] Resolve an issue in the sequence where a referenced object removed and then the referencing object deleting and then re-adding (#2210)
eaf7264 [macsecorch]: MACsec with pfc (#2095)
a32b611 [azp]: Reduce diff coverage to 50% threshhold (#2227)
6301db7 [Code owner] Set owners for auto reviews (#2229)
d1fb3dd [BFD]Retry create BFD with different source UDP port on failure (#2225)
53620f3 [orchagent] add & remove port counters dynamically each time port was added or removed (#2019)
cf216be Change ERR to Notice for tunnel term create fail (#2219)
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
…t#1951)"" (sonic-net#2019)

Refer to sonic-net#1951 for details

The PR sonic-net#1951 was reverted in sonic-net#2016 because it was thought to be causing the build failure in sonic-swss TestWarmReboot UTs. But it seems the failures in TestWarmReboot are still occurring e.g. sonic-net#2017 [build](https://dev.azure.com/mssonic/build/_build/results?buildId=65653&view=logs&s=859b8d9a-8fd6-5a5c-6f5e-f84f1990894e)

I think we can reapply sonic-net#1951 and need to investigate TestWarmReboot tests instability
Ndancejic pushed a commit to Ndancejic/sonic-buildimage that referenced this pull request May 3, 2022
… added or removed (sonic-net#2019)

- What I did
Add support for adding & removing port counters dynamically each time port was added or removed the counters that were supported are:

1. pg watermark counters
2. pg drop counters
3. queue stat counters
4. queue watermark counters
5. debug counters
6. buffer drop counters and port stat counters are already supported to be added or removed each time port is added/removed

Implemented according to the - 'HLD document on add/remove ports dynamically feature' sonic-net/SONiC#900

- Why I did it
In order to support dynamically add or removed ports on sonic

- How I verified it
tested manually that the flex counters were added or removed correctly whenever we add or remove ports
added new test cases to the following vs tests:

test_flex_counters.py
test_drop_counters.py
test_pg_drop_counter.py

Co-authored-by: dprital <drorp@nvidia.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.

3 participants