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

Add/Del lag_name_map item when adding and removing lag. #1124

Merged
merged 1 commit into from
Mar 4, 2020
Merged

Add/Del lag_name_map item when adding and removing lag. #1124

merged 1 commit into from
Mar 4, 2020

Conversation

JiangboHe
Copy link

What I did
The code is to add/del items in LAG_NAME_MAP_TABLE in COUNTERS_DB if a lag is added or removed.
The code need the below pull request:
#51 in Azure/sonic-py-swsssdk
read portchannel name from LAG_NAME_MAP_TABLE in COUNTERS_DB #51

Why I did it
For use of LAG_NAME_MAP_TABLE in COUNTERS_DB just like fdbshow.
I have create another pull request in Azure/sonic-utilities:
Show mac learned on lag interface #730

How I verified it

Details if related

@msftclas
Copy link

msftclas commented Nov 8, 2019

CLA assistant check
All CLA requirements met.

@JiangboHe JiangboHe changed the title Add/Del lag_name_map item according to lag adding and removing Add/Del lag_name_map item when adding and removing lag. Nov 13, 2019
@JiangboHe
Copy link
Author

Hi
I have review the failed checks. They are not caused by my commit.
Could you or the system please check my commit again?
Thanks.

@lguohan
Copy link
Contributor

lguohan commented Feb 13, 2020

retest this please

@lguohan
Copy link
Contributor

lguohan commented Feb 14, 2020

retest this please

1 similar comment
@lguohan
Copy link
Contributor

lguohan commented Feb 29, 2020

retest this please

@lguohan lguohan merged commit 574f199 into sonic-net:master Mar 4, 2020
abdosi pushed a commit that referenced this pull request Mar 4, 2020
The code is to add/del items in LAG_NAME_MAP_TABLE in COUNTERS_DB if a lag is added or removed.
The code need the below pull request:
#51 in Azure/sonic-py-swsssdk
read portchannel name from LAG_NAME_MAP_TABLE in COUNTERS_DB #51

For use of LAG_NAME_MAP_TABLE in COUNTERS_DB just like fdbshow.
I have create another pull request in Azure/sonic-utilities:
Show mac learned on lag interface #730
yxieca pushed a commit that referenced this pull request Apr 3, 2020
The code is to add/del items in LAG_NAME_MAP_TABLE in COUNTERS_DB if a lag is added or removed.
The code need the below pull request:
#51 in Azure/sonic-py-swsssdk
read portchannel name from LAG_NAME_MAP_TABLE in COUNTERS_DB #51

For use of LAG_NAME_MAP_TABLE in COUNTERS_DB just like fdbshow.
I have create another pull request in Azure/sonic-utilities:
Show mac learned on lag interface #730
jleveque added a commit to sonic-net/sonic-py-swsssdk that referenced this pull request Apr 8, 2020
…TERS_DB (#51)" (#57)" (#64)

This reverts commit 022d8fb.

Add #51 back in by reverting #57, in because issues should be addressed by the merge of sonic-net/sonic-swss#1124
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
#### Why I did it
sonic-swss-common lib will add depency to libyang soon, so need install libyang lib to prevent build and UT break.

#### How I did it
Modify azure pipeline to install libyang in azure pipeline steps.

#### How to verify it
Pass all UT.

#### Which release branch to backport (provide reason below if selected)

#### Description for the changelog
Modify azure pipeline to install libyang in azure pipeline steps.

#### Link to config_db schema for YANG module changes

#### A picture of a cute animal (not mandatory but encouraged)
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.

6 participants