Skip to content

[action] [PR:16357] [sanity_check][bgp] Enhance sanity check recover for bgp default route missing#16386

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/16357
Jan 13, 2025
Merged

[action] [PR:16357] [sanity_check][bgp] Enhance sanity check recover for bgp default route missing#16386
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/16357

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Sometimes exabgp in ptf would be in incorrect status by stress testing, hence add restarting exabgp before re-announce routes in sanity check.

How did you do it?

  1. Restart exabgp before re-announce routes
  2. Add try catch to handle failed to re-announce issue

How did you verify/test it?

Run test with sanity check

Any platform specific information?

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

Documentation

…e missing (sonic-net#16357)

What is the motivation for this PR?
Sometimes exabgp in ptf would be in incorrect status by stress testing, hence add restarting exabgp before re-announce routes in sanity check.

How did you do it?
Restart exabgp before re-announce routes
Add try catch to handle failed to re-announce issue
How did you verify/test it?
Run test with sanity check
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #16357

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit d75c9b6 into sonic-net:202411 Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants