sonic-swss icon indicating copy to clipboard operation
sonic-swss copied to clipboard

Read switch_id of fabric switch from config_db

Open ysmanman opened this issue 1 year ago • 2 comments

What I did Read switch_id of fabric switch from config_db

Why I did it Fix https://github.com/sonic-net/sonic-buildimage/issues/18558

How I verified it

Details if related

ysmanman avatar Apr 05 '24 22:04 ysmanman

/Azp run Azure.sonic-swss

arlakshm avatar Apr 26 '24 18:04 arlakshm

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Apr 26 '24 18:04 azure-pipelines[bot]

/Azwp run Azure.sonic-swss

kenneth-arista avatar May 06 '24 20:05 kenneth-arista

/Azpw run Azure.sonic-swss

kenneth-arista avatar May 06 '24 21:05 kenneth-arista

/Azp run Azure.sonic-swss

arlakshm avatar May 08 '24 17:05 arlakshm

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 08 '24 17:05 azure-pipelines[bot]

@arlakshm , looks like its a genuine VS failure.. please check my comment

prsunny avatar May 08 '24 18:05 prsunny

@arlakshm , looks like its a genuine VS failure.. please check my comment

Thanks @prsunny . I will add switch_id in config_db of virtual chassis.

ysmanman avatar May 13 '24 21:05 ysmanman

Can you please fix coverage?

prsunny avatar May 21 '24 18:05 prsunny

Can you please fix coverage?

Hi @prsunny updated PR with unit test.

ysmanman avatar May 21 '24 23:05 ysmanman

/Azp run Azure.sonic-swss

ysmanman avatar May 22 '24 01:05 ysmanman

Commenter does not have sufficient privileges for PR 3102 in repo sonic-net/sonic-swss

azure-pipelines[bot] avatar May 22 '24 01:05 azure-pipelines[bot]

/azpw run Azure.sonic-swss

ysmanman avatar May 22 '24 01:05 ysmanman

/AzurePipelines run Azure.sonic-swss

mssonicbld avatar May 22 '24 01:05 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 22 '24 01:05 azure-pipelines[bot]

/Azp run Azure.sonic-swss

arlakshm avatar May 22 '24 16:05 arlakshm

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 22 '24 16:05 azure-pipelines[bot]

/azpw run Azure.sonic-swss

ysmanman avatar May 23 '24 03:05 ysmanman

/AzurePipelines run Azure.sonic-swss

mssonicbld avatar May 23 '24 03:05 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 23 '24 03:05 azure-pipelines[bot]

@arlakshm @prsunny All checks passed. Please help reviewing and merging the PR.

ysmanman avatar May 25 '24 18:05 ysmanman

/azpw run Azure.sonic-swss

ysmanman avatar May 29 '24 23:05 ysmanman

/AzurePipelines run Azure.sonic-swss

mssonicbld avatar May 29 '24 23:05 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 29 '24 23:05 azure-pipelines[bot]