Skip to content

[action] [PR:3825] [ssw][ha] consume new ha_scope fields#150

Merged
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/3825
Sep 10, 2025
Merged

[action] [PR:3825] [ssw][ha] consume new ha_scope fields#150
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/3825

Conversation

@mssonicbld
Copy link
Collaborator

What I did
Address the minor changes in HA HLD:

  1. reject ha_scope object createion if ha_set is not created first.
  2. get ha_set_id mapping from ha_scope table
  3. get vip from ha_scope table

Dependency: sonic-net/sonic-dash-api#44

Why I did it
To align with HLD change.

How I verified it

  • UTs.
  • load on DPU, program objects (pending)

Details if related

<!--
Please make sure you have read and understood the contribution guildlines:
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

1. Make sure your commit includes a signature generted with `git commit -s`
2. Make sure your commit title follows the correct format: [component]: description
3. Make sure your commit message contains enough details about the change and related tests
4. Make sure your pull request adds related reviewers, asignees, labels

Please also provide the following information in this pull request:
-->

**What I did**
Address the minor changes in HA HLD:
1. reject ha_scope object createion if ha_set is not created first.
2.  get ha_set_id mapping from ha_scope table
3. get vip from ha_scope table

Dependency:  sonic-net/sonic-dash-api#44

**Why I did it**
To align with HLD change.

**How I verified it**
* UTs.
* load on DPU, program objects (pending)

**Details if related**
@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-swss#3825

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit a5a8fed into Azure:202506 Sep 10, 2025
5 of 8 checks passed
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.

1 participant