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

Add static route ASIC DB check for test_nhop_group test

Open echuawu opened this issue 1 year ago • 7 comments

Description of PR

Add the static route ASIC DB validation in cases of weak switch

Summary: Fixes # (issue)

Type of change

  • [x] Bug fix
  • [ ] Testbed and Framework(new/improvement)
  • [ ] Test case(new/improvement)

Back port request

  • [ ] 202012
  • [ ] 202205
  • [ ] 202305
  • [ ] 202311
  • [x] 202405

Approach

What is the motivation for this PR?

For some weak CPU switch, such as Mellanox SN2700 with A0 chip type. The fixed 15 seconds is not enough to handle a bunch of BGP route add/delete and static route add/delete. So the case would failed from time to time.

How did you do it?

Add a ASIC DB check for the static route validation.

How did you verify/test it?

Run it in internal regression.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

echuawu avatar Sep 09 '24 08:09 echuawu

/azpw run

echuawu avatar Sep 10 '24 02:09 echuawu

/AzurePipelines run

mssonicbld avatar Sep 10 '24 02:09 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 10 '24 02:09 azure-pipelines[bot]

/azpw run

echuawu avatar Sep 11 '24 07:09 echuawu

/AzurePipelines run

mssonicbld avatar Sep 11 '24 07:09 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 11 '24 07:09 azure-pipelines[bot]

@abdosi @prsunny Can you help review?

bingwang-ms avatar Oct 23 '24 01:10 bingwang-ms

Cherry-pick PR to 202405: https://github.com/sonic-net/sonic-mgmt/pull/15373

mssonicbld avatar Nov 06 '24 02:11 mssonicbld