Skip to content

LLR: Link Layer Retry feature yang and docker script changes#27028

Open
rminnikanti wants to merge 1 commit into
sonic-net:masterfrom
rminnikanti:llr
Open

LLR: Link Layer Retry feature yang and docker script changes#27028
rminnikanti wants to merge 1 commit into
sonic-net:masterfrom
rminnikanti:llr

Conversation

@rminnikanti
Copy link
Copy Markdown
Contributor

HLD: sonic-net/SONiC#2098

What I did

  • Yang implementation as per the HLD and respective tests
  • docker-orchagent changes, to start llrmgrd and include it into supervisord monitoring.
  • docker-sonic-vs changes to include llrmgrd and respective supervisord config changes
  • In DVS, all the daemons are explicitly started in start.sh added llrmgrd in it.

Why I did it

HLD: sonic-net/SONiC#2098

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

  • Verified by building a docker-sonic-vs.gz image.
  • llrmgrd automatically running and all the sonic-swss DVS tests pass

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

  • master

Description for the changelog

Link to config_db schema for YANG module changes

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

- Yang implementation as per the HLD and respective tests
- docker-orchagent changes, to start llrmgrd and include it into
  supervisord monitoring.
- docker-sonic-vs changes to include llrmgrd and respective supervisord
  config changes
- In DVS, all the daemons are explicitly started in start.sh added
  llrmgrd in it.

Signed-off-by: Ravi Minnikanti <rminnikanti@marvell.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

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.

2 participants