Skip to content

Add HA session parameter Bounceback IP#631

Open
marian-pritsak wants to merge 2 commits into
sonic-net:mainfrom
marian-pritsak:bounceback
Open

Add HA session parameter Bounceback IP#631
marian-pritsak wants to merge 2 commits into
sonic-net:mainfrom
marian-pritsak:bounceback

Conversation

@marian-pritsak

Copy link
Copy Markdown
Collaborator

Add bounceback IP which is a different IP from
peer IP, to emulate the same network path for data packets as for the SYN packets.

Add bounceback IP which is a different IP from
peer IP, to emulate the same network path for data
packets as for the SYN packets.

Signed-off-by: Marian Pritsak <marianp@mellanox.com>
@marian-pritsak

Copy link
Copy Markdown
Collaborator Author

Depends on opencomputeproject/SAI#2090

@r12f

r12f commented Oct 20, 2024

Copy link
Copy Markdown
Collaborator

hi Marian, do you mind to update the HLD and explain a bit more on what is this parameter doing?

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

hi @marian-pritsak - is an HLD update in play for this week?

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

@marian-pritsak opencomputeproject/SAI#2090 has been merged - is it ok to merge this one too?

@r12f

r12f commented Jan 22, 2025

Copy link
Copy Markdown
Collaborator

Hi Marian, do you mind explaining a bit more on what does this attribute do? @KrisNey-MSFT , maybe we can do it in one of our community call.

Junhua is helping bring up the SAI API in DASH to latest, but this difference is causing problems, since the attribute is not aware in DASH, do generated SAI will simply revert this.

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

Do not need to send HA back to DPU; this is an optimization. To preserve entropy and look as if it is coming from the same switch. @marian-pritsak will update the HA document in the SONiC repo. Attribute to support capability queries. If attribute not supported, then not enforced. (SAI API is already merged).

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

Hi @marian-pritsak - is this one ready for merge?

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

Hi @marian-pritsak , has the HA document been updated in the SONiC repo? @r12f , and/or @mukeshmv as Reviewers

@KrisNey-MSFT

Copy link
Copy Markdown
Collaborator

Hi @r12f and @mukeshmv - is this ready to review?

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