Skip to content

[action] [PR:96] Add vip_v4 and vip_v6 to DPU DASH_HA_SCOPE_TABLE#15

Merged
prabhataravind merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/96
Aug 31, 2025
Merged

[action] [PR:96] Add vip_v4 and vip_v6 to DPU DASH_HA_SCOPE_TABLE#15
prabhataravind merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/96

Conversation

@mssonicbld
Copy link
Collaborator

why

There is a mismatch between DASH_HA_SCOPE_TABLE in DPU sonic-db and SAI HA_SCOPE definition. vip_v4 and vip_v6 exist in the latter but not in DASH_HA_SCOPE_TABLE. DashHaOrch has to implement complicated logic to get vip_v4 and vip_v6 from DASH_HA_SET_TABLE. To simplify DashHaOrch design, vip_v4 and vip_v6 are added to DASH_HA_SCOPE_TABLE. Accordingly, hamgrd needs to adapt to it.

what this PR does

  1. Update DASH_HA_SCOPE_TABLE until it receives DASH_HA_SET_CONFIG from corresponding ha_set actor
  2. Copy vip_v4 and vip_v6 from DASH_HA_SET_CONFIG to DASH_HA_SCOPE_TABLE
  3. Update UT

### why
There is a mismatch between DASH_HA_SCOPE_TABLE in DPU sonic-db and SAI HA_SCOPE definition. vip_v4 and vip_v6 exist in the latter but not in DASH_HA_SCOPE_TABLE. DashHaOrch has to implement complicated logic to get vip_v4 and vip_v6 from DASH_HA_SET_TABLE. To simplify DashHaOrch design, vip_v4 and vip_v6 are added to DASH_HA_SCOPE_TABLE. Accordingly, hamgrd needs to adapt to it.

### what this PR does
1. Update DASH_HA_SCOPE_TABLE until it receives DASH_HA_SET_CONFIG from corresponding ha_set actor
2. Copy vip_v4 and vip_v6 from DASH_HA_SET_CONFIG to DASH_HA_SCOPE_TABLE
3. Update UT
@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-dash-ha#96

@mssonicbld
Copy link
Collaborator Author

/azp run

@prabhataravind prabhataravind merged commit 9884abd into Azure:202506 Aug 31, 2025
2 checks passed
prabhataravind pushed a commit to prabhataravind/sonic-dash-ha.msft that referenced this pull request Sep 8, 2025
Unless clippy is disabled for some specific reason, I think we should
have it be enabled. It pointed out a couple of errors in swss-common
that I fixed.
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.

2 participants