Skip to content

Set ha_set_id in DASH_HA_SCOPE_TABLE#95

Merged
yue-fred-gao merged 2 commits intosonic-net:masterfrom
yue-fred-gao:set_ha_set_id
Aug 1, 2025
Merged

Set ha_set_id in DASH_HA_SCOPE_TABLE#95
yue-fred-gao merged 2 commits intosonic-net:masterfrom
yue-fred-gao:set_ha_set_id

Conversation

@yue-fred-gao
Copy link
Collaborator

why

ha_set_id is added to DASH_HA_SCOPE_TABLE. hamgrd needs to pass ha_set_id in DASH_HA_SCOPE_CONFIG_TABLE

what this PR does

  • passes ha_set_id in DASH_HA_SCOPE_CONFIG_TABLE to DASH_HA_SCOPE_TABLE
  • we used to extract ha-set id from key of DASH_HA_SCOPE_CONFIG_TABLE. Since ha_set_id is a field in the table, we don't need to extract it from the key
  • add owner to DASH_HA_SET_TABLE according to HLD but the field is not set since dashhaorch is not using it

@mssonicbld
Copy link

/azp run

@yue-fred-gao yue-fred-gao requested a review from zjswhhh July 31, 2025 20:28
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@zjswhhh zjswhhh left a comment

Choose a reason for hiding this comment

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

lgtm

@mssonicbld
Copy link

Cherry-pick PR to msft-202506: Azure/sonic-dash-ha.msft#14

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.

5 participants