Skip to content

Enhance getNeighborEntry to return neighbor entry for system neighbor#2024

Closed
ysmanman wants to merge 1 commit intosonic-net:masterfrom
ysmanman:evrflow-voq
Closed

Enhance getNeighborEntry to return neighbor entry for system neighbor#2024
ysmanman wants to merge 1 commit intosonic-net:masterfrom
ysmanman:evrflow-voq

Conversation

@ysmanman
Copy link
Copy Markdown
Contributor

@ysmanman ysmanman commented Nov 12, 2021

What I did
Enhance getNeighborEntry to return neighbor entry for system neighbor

Why I did it
getNeighborEntry is not able to handle system neighbor today. This breaks features like Everflow in VOQ systems.

How I verified it
We verified the change by testing Everflow in VOQ system (e.g., mirror DIP is resolved to system neighbor).

Details if related

@ysmanman ysmanman requested a review from prsunny as a code owner November 12, 2021 00:14
@prsunny prsunny requested a review from abdosi November 12, 2021 17:15
@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Copy Markdown
Contributor

@abdosi, can you take a look

@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Copy Markdown
Contributor

arlakshm commented May 4, 2022

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Copy Markdown
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented Aug 17, 2022

#2392

@rlhui
Copy link
Copy Markdown
Contributor

rlhui commented Nov 11, 2022

is this one still needed or can be closed?

@ysmanman
Copy link
Copy Markdown
Contributor Author

is this one still needed or can be closed?

Hi @rlhui, #2392 should fix the same issue the PR addresses. I am closing it.

@ysmanman ysmanman closed this Nov 11, 2022
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.

4 participants