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

bring up dash-ha containers in smartswitch #21621

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

yue-fred-gao
Copy link
Contributor

@yue-fred-gao yue-fred-gao commented Feb 4, 2025

Why I did it

dash-ha containers running hamgrd and swbusd are responsible for DASH HA management in smartswitch platform. We need one dash-ha container per DPU per HLD.

Work item tracking
  • Microsoft ADO (number only):

How I did it

A few integral parts are added.

  1. dash-container including Dockerfile.j2, supervisord.conf etc.
  2. Makefiles to build dash-ha and docker-dash-ha
  3. dash-ha service script including files/build_templates/per_namespace/dash-ha.service.j, files/scripts/dash-ha.sh and the common container startup script files/build_templates/docker_image_ctl.j2

How to verify it

This is built and tested in cisco smartswitch platform

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

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
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

 - Verified on MtFuji
 - swbusd connect to each other in the same switch
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yue-fred-gao yue-fred-gao marked this pull request as ready for review February 20, 2025 14:58
@yue-fred-gao yue-fred-gao changed the title Add sonic dash ha bring up dash-ha containers in smartswitch Feb 21, 2025
@r12f r12f self-requested a review February 24, 2025 18:25
@KrisNey-MSFT
Copy link

hi @r12f and @yue-fred-gao, should we clear out the conflict for this one?

@yue-fred-gao
Copy link
Contributor Author

hi @r12f and @yue-fred-gao, should we clear out the conflict for this one?

Resolved. Thanks!

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f requested review from erer1243, Pterosaur and zjswhhh March 13, 2025 20:57
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

4 participants