Skip to content

Use TABLE for DASH_HA_ tables - #572

Merged
hdwhdw merged 5 commits into
sonic-net:masterfrom
zjswhhh:2_9
Feb 16, 2026
Merged

hdwhdw merged 5 commits into
sonic-net:masterfrom
zjswhhh:2_9

Conversation

@zjswhhh

@zjswhhh zjswhhh commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Why I did it

There are 8 instances of hamgrd, which will all consume the ha set config table. If we use producer state table, the table change will only be consumed by one of them.

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it

Check table prefix.

How to verify it

Tested on SSW testbed with gnmi_set cmd.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@zjswhhh zjswhhh changed the title TABLE Use TABLE for DASH_HA_ tables Feb 12, 2026
@zjswhhh
zjswhhh marked this pull request as ready for review February 12, 2026 01:58
prsunny
prsunny previously approved these changes Feb 13, 2026

@prsunny prsunny left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm, can you address copilot comments? @hdwhdw , can you review as well?

hdwhdw
hdwhdw previously approved these changes Feb 13, 2026

@hdwhdw hdwhdw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@zjswhhh
zjswhhh dismissed stale reviews from hdwhdw and prsunny via 5268668 February 16, 2026 20:26
@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Jing Zhang <zhangjing@microsoft.com>
Signed-off-by: Jing Zhang <zhangjing@microsoft.com>
Signed-off-by: Jing Zhang <zhangjing@microsoft.com>
Signed-off-by: Jing Zhang <zhangjing@microsoft.com>
Signed-off-by: Jing Zhang <zhangjing@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@zjswhhh

zjswhhh commented Feb 16, 2026

Copy link
Copy Markdown
Contributor Author

Hi @hdwhdw - addressed copilot's comments. Please help review and merge. Thanks!

@hdwhdw
hdwhdw enabled auto-merge (squash) February 16, 2026 20:32
@hdwhdw
hdwhdw merged commit aa7d5e1 into sonic-net:master Feb 16, 2026
9 checks passed
@zjswhhh
zjswhhh deleted the 2_9 branch February 17, 2026 04:56
@mssonicbld

Copy link
Copy Markdown
Contributor

Cherry-pick PR to msft-202506: Azure/sonic-gnmi.msft#168

@zjswhhh

zjswhhh commented Feb 17, 2026

Copy link
Copy Markdown
Contributor Author

Hi @vmittal-msft - can you please approve for 202511?

@mssonicbld

Copy link
Copy Markdown
Contributor

Cherry-pick PR to 202511: #580

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.

6 participants