Skip to content

[action] [PR:10697] Fix suppress fib failure#10703

Merged
StormLiangMS merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/10697
Nov 10, 2023
Merged

[action] [PR:10697] Fix suppress fib failure#10703
StormLiangMS merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/10697

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

  • 201911
  • 202012
  • 202205
  • 202305

Approach

What is the motivation for this PR?

bgp suppress fib enabled by default, which could fail below two cases
test_bgp_route_without_suppresstest_bgp_route_without_suppress
test_credit_loop

How did you do it?

To disable the bgp suppress fib at the beginning of test. And enhance the restore_bgp_suppress_fib function

How did you verify/test it?

Run sonic-mgmt test

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #10697

What is the motivation for this PR?
bgp suppress fib enabled by default, which could fail below two cases
test_bgp_route_without_suppresstest_bgp_route_without_suppress
test_credit_loop

How did you do it?
To disable the bgp suppress fib at the beginning of test. And enhance the restore_bgp_suppress_fib function

How did you verify/test it?
Run sonic-mgmt test
@StormLiangMS StormLiangMS merged commit 806f961 into sonic-net:202305 Nov 10, 2023
@mssonicbld mssonicbld deleted the cherry/202305/10697 branch February 4, 2024 09:37
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