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 HA session parameter Bounceback IP #631

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

marian-pritsak
Copy link
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 <[email protected]>
@marian-pritsak
Copy link
Collaborator Author

Depends on opencomputeproject/SAI#2090

@r12f
Copy link
Collaborator

r12f commented Oct 20, 2024

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

@KrisNey-MSFT
Copy link
Collaborator

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

@KrisNey-MSFT
Copy link
Collaborator

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

@r12f
Copy link
Collaborator

r12f commented Jan 22, 2025

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
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).

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