[sonic-swss] update submodule#5537
Closed
madhanmellanox wants to merge 1 commit intosonic-net:masterfrom
Closed
Conversation
Contributor
|
fyi #5533 |
Contributor
|
Closing in favour of #5533 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
- Why I did it
Updating submodule for sonic-swss to get the changes to Azure master. The following were the commits that were part of this submodule.
remove FDB entires of BridgePort before removing the BridgePort 037aee61dc26048dd4d761c1bac184be6ffcc79c
fixed function naming and other print related information 0060a3328177105deb834d39e07479270c48d49b
fixed indentation issue 82e467e956d015c09f933b1bc039b51cf151e287
Fixing function name in portsorch.h c165697df2124d971a1c80254e1100c760e99fde
added function call to flushFdbEntries() also in removeDefaultBridgePorts() function 7dd0610ccf1662d716b6a946fcb1e011089691bb
fixed indentation issues 6c87e0fb71724a5f364d1a988cf233db81d3a01a
instead of a new member function for portsorch used fdborch member function c5ab0dc2fc849f1e63b533f7e8b7ef973a7fefc5
moved flushFDBEntries member function of fdborch from private to public 09c5a41368d12aa966752e9fd149c9103af5675a
- How I did it
Merged fix for removal of bridgeport defect to Azure master. Now, updating the submodule changes.
- How to verify it
To test the fix, create a MAC entry for the VLAN configured on its member port by sending untagged traffic to the port, it will create an invalid MAC entry, which is the defect itself. Try to remove the only member port from VLAN and see if the invalid MAC entry is getting removed.
- Which release branch to backport (provide reason below if selected)
- Description for the changelog
The changes are in sonic-swss/orchagent/portsorch.cpp and sonic-swss/orchagent/fdborch.h.
- A picture of a cute animal (not mandatory but encouraged)