Skip to content

[mirrororch] Fix a bug where ERSPAN used LAG as the destination port and linked down the first LAG member - #2547

Open
inspurSDN wants to merge 1 commit into
sonic-net:masterfrom
inspurSDN:inspur-erspan-support-lag-destination-port
Open

inspurSDN wants to merge 1 commit into
sonic-net:masterfrom
inspurSDN:inspur-erspan-support-lag-destination-port

Conversation

@inspurSDN

@inspurSDN inspurSDN commented Nov 24, 2022 •

Copy link
Copy Markdown
Contributor

[mirrororch] Fix a bug where ERSPAN used LAG as the destination port and linked down the first LAG member.
[portsorch] Add a function to return a linked-up member of an input LAG.

  • What I did
    When a destination port belongs to a LAG in an ERSPAN session,
    detect the LAG member that is linked down and switch the destination port to a LAG member that is linked up.

  • Why I did it
    Current ERSPAN is unaware of the LAG member port linking up or down.

  • How to verify it
    The community tests and manual tests pass.

Signed-off-by: sdn@inspur.com

@inspurSDN
inspurSDN requested a review from prsunny as a code owner November 24, 2022 10:12
@linux-foundation-easycla

linux-foundation-easycla Bot commented Nov 24, 2022 •

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: inspurSDN / name: InspurSDN (2741b7f)

@inspurSDN
inspurSDN force-pushed the inspur-erspan-support-lag-destination-port branch 6 times, most recently from 6c775ab to 5759dca Compare December 1, 2022 02:27
@prsunny
prsunny requested a review from bingwang-ms December 5, 2022 22:17
@prsunny

prsunny commented Dec 5, 2022

Copy link
Copy Markdown
Collaborator

@inspurSDN , do you've an HLD to link here?

@inspurSDN
inspurSDN force-pushed the inspur-erspan-support-lag-destination-port branch from 5759dca to 97c36d5 Compare December 6, 2022 10:52
@inspurSDN inspurSDN changed the title [mirrororch] ERSPAN supports LAG as a destination port. [mirrororch] Fix a bug where ERSPAN used LAG as the destination port and linked down the first LAG member Dec 6, 2022
@inspurSDN
inspurSDN force-pushed the inspur-erspan-support-lag-destination-port branch from 97c36d5 to 6f1d24a Compare December 7, 2022 02:27
â€Ķand linked down the first LAG member.

[portsorch] Add a function to return a linked-up member of an input LAG.

- What I did
When a destination port belongs to a LAG in an ERSPAN session,
detect the LAG member that is linked down and switch the destination port to a LAG member that is linked up.

- Why I did it
Current ERSPAN is unaware of the LAG member port linking up or down.

- How to verify it
The community tests and manual tests pass.

Signed-off-by: sdn@inspur.com
@inspurSDN
inspurSDN force-pushed the inspur-erspan-support-lag-destination-port branch from 6f1d24a to 4557a0c Compare December 22, 2022 06:46
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.

2 participants