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 support for DPU scope DPU driven HA #599

Merged
merged 2 commits into from
Jul 16, 2024
Merged

Conversation

mukeshmv
Copy link
Collaborator

@mukeshmv mukeshmv commented Jul 1, 2024

This PR adds SAI API support for the DPU scope DPU driven Dash HA mode described in this PR.
sonic-net/SONiC#1710

In this mode, the HA scope / HA state is at the DPU level. Also the DPU vendor SDK owns the HA state machine and drives state transitions on its own by directly communicating with its HA pair underneath the SAI layer. The DPU SDK advertises HA state changes as SAI event notifications.

@mukeshmv mukeshmv force-pushed the dpu_scope_ha_3 branch 2 times, most recently from e4b9c41 to a4256a0 Compare July 2, 2024 05:53
@mukeshmv mukeshmv changed the title Add support for DPU driven DPU scope HA attributes Add support for DPU scope DPU driven HA Jul 2, 2024
@r12f r12f requested review from r12f and marian-pritsak July 3, 2024 20:44
Copy link
Collaborator

@r12f r12f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added some minor comments in the HLD. Will merge once resolved. thanks Mukesh!

documentation/high-avail/ha-api-hld.md Outdated Show resolved Hide resolved
documentation/high-avail/ha-api-hld.md Outdated Show resolved Hide resolved
documentation/high-avail/ha-api-hld.md Outdated Show resolved Hide resolved
documentation/high-avail/ha-api-hld.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@r12f r12f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@r12f r12f merged commit 222ef40 into sonic-net:main Jul 16, 2024
5 checks passed
mukeshmv added a commit to mukeshmv/SAI that referenced this pull request Jul 17, 2024
mukeshmv added a commit to mukeshmv/SAI that referenced this pull request Jul 23, 2024
Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604

Signed-off-by: mukeshmv <[email protected]>
mukeshmv added a commit to mukeshmv/SAI that referenced this pull request Jul 23, 2024
Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604

Signed-off-by: mukeshmv <[email protected]>
mukeshmv added a commit to mukeshmv/SAI that referenced this pull request Jul 24, 2024
Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604

Signed-off-by: mukeshmv <[email protected]>
mukeshmv added a commit to mukeshmv/SAI that referenced this pull request Jul 24, 2024
Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604

Signed-off-by: mukeshmv <[email protected]>
prsunny pushed a commit to opencomputeproject/SAI that referenced this pull request Aug 6, 2024
Add Dash DPU scope DPU driven HA attributes (#2055)

Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604
siqbal1986 pushed a commit to siqbal1986/SAI that referenced this pull request Sep 30, 2024
Add Dash DPU scope DPU driven HA attributes (opencomputeproject#2055)

Brings in SAI API changes from these DASH PRs
sonic-net/DASH#599
sonic-net/DASH#604

Signed-off-by: siqbal1986 <[email protected]>
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.

3 participants