Skip to content

Add fabric port data for vs test, and start fabricmgrd in vs environment#16791

Merged
lguohan merged 1 commit intosonic-net:masterfrom
jfeng-arista:master-fabric-port-data-for-vstest
Nov 21, 2023
Merged

Add fabric port data for vs test, and start fabricmgrd in vs environment#16791
lguohan merged 1 commit intosonic-net:masterfrom
jfeng-arista:master-fabric-port-data-for-vstest

Conversation

@jfeng-arista
Copy link
Contributor

Add fabric port data for vs test, and start fabricmgrd in vs environment.

This PR depends on sonic-net/sonic-sairedis#1301

sonic-net/sonic-swss#2920 needs this one merge first.

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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)

@jfeng-arista
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@arlakshm arlakshm left a comment

Choose a reason for hiding this comment

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

lgtm

@arlakshm
Copy link
Contributor

Hi @lguohan, can you please help merge this PR.

@arlakshm
Copy link
Contributor

arlakshm commented Nov 1, 2023

Hi @lguohan can you please help merge this PR.

1 similar comment
@arlakshm
Copy link
Contributor

Hi @lguohan can you please help merge this PR.

@lguohan lguohan merged commit 6dfaf5e into sonic-net:master Nov 21, 2023
rlhui pushed a commit that referenced this pull request Feb 8, 2024
The fabricmgr daemon started in vs environment for testing from #16791, we now start the daemon in product code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants