Skip to content

Start fabric mgr daemon in swss container.#17473

Merged
rlhui merged 1 commit intosonic-net:masterfrom
jfeng-arista:master-start-fabricmgr-daemon-in-sup
Feb 8, 2024
Merged

Start fabric mgr daemon in swss container.#17473
rlhui merged 1 commit intosonic-net:masterfrom
jfeng-arista:master-start-fabricmgr-daemon-in-sup

Conversation

@jfeng-arista
Copy link
Copy Markdown
Contributor

Why I did it

The fabricmgr daemon started in vs environment for testing from #16791, we now start the daemon in product code.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Start fabric mgr daemon in swss container .

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
Copy Markdown
Contributor Author

/Azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@jfeng-arista
Copy link
Copy Markdown
Contributor Author

the failure seems not related to the change

/Azpw run Azure.sonic-buildimage

@jfeng-arista
Copy link
Copy Markdown
Contributor Author

/Azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@jfeng-arista
Copy link
Copy Markdown
Contributor Author

/Azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@jfeng-arista
Copy link
Copy Markdown
Contributor Author

2023-12-13T21:09:29.8376752Z Test plan id: 657a0283cfacd86e78e3e907, status: EXECUTING, progress: 91.67%, elapsed: 5169 seconds
2023-12-13T21:09:29.8377841Z Operation failed with exception: Exception('Test plan id: 657a0283cfacd86e78e3e907, status: FAILED, result: FAILED, Elapsed 5229 seconds. Check https://elastictest.org/scheduler/testplan/657a0283cfacd86e78e3e907 for test plan status')
2023-12-13T21:09:29.8580108Z ##[error]Bash exited with code '3'.
2023-12-13T21:09:29.8610465Z ##[section]Finishing: Run test

failed as abvoe, not sure if this could related to my change as it is in t0.

I will retirgger the pipline tomorrow

@jfeng-arista
Copy link
Copy Markdown
Contributor Author

/Azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph judyjoseph requested a review from arlakshm December 14, 2023 21:41
arlakshm pushed a commit to sonic-net/sonic-mgmt that referenced this pull request Jan 24, 2024
…mands. (#10994)

Summary: Add mgmt test for testing fabric link isolation cli commands.

This change adds a test for the fabric link isolation command.
The test iterates over the fabric links on a linecard. It isolates and unisolates fabric links. Each time the state of the link is changed the state is checked in both CONFIG_DB and APPL_DB. The values in CONFIG_DB are updated by the fabric CLI commands and the values in APPL_DB are updated by the fabric manager daemon. The test check if the 2 major part work as expected.

need to wait sonic-net/sonic-buildimage#17473

Signed-off-by: Jie Feng <[email protected]>
@arlakshm
Copy link
Copy Markdown
Contributor

@lguohan, can you help merge this PR

@rlhui rlhui merged commit 5a20589 into sonic-net:master Feb 8, 2024
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