Skip to content

[action] [PR:10938] Enhance the test route flap#10948

Merged
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/10938
Dec 5, 2023
Merged

[action] [PR:10938] Enhance the test route flap#10948
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/10938

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)
test_route_flap is flaky, sometimes errored out due to missed_frr_routes check, which is a transient log print, we should ignore it during route flap test.

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?

Enhance robust of test_route_flap

How did you do it?

Ignore error which is expected.

How did you verify/test it?

Run manual test.

03:02:53 test_route_flap.test_route_flap L0384 INFO | End
PASSED [100%]

Any platform specific information?

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

Documentation

What is the motivation for this PR?
Enhance robust of test_route_flap

How did you do it?
Ignore error which is expected.

How did you verify/test it?
Run manual test.

03:02:53 test_route_flap.test_route_flap L0384 INFO | End
PASSED [100%]
@mssonicbld mssonicbld requested a review from jcaiMR as a code owner December 5, 2023 08:24
@mssonicbld mssonicbld mentioned this pull request Dec 5, 2023
7 tasks
@mssonicbld
Copy link
Collaborator Author

Original PR: #10938

@mssonicbld mssonicbld merged commit 3a44774 into sonic-net:202305 Dec 5, 2023
@mssonicbld mssonicbld deleted the cherry/202305/10938 branch February 4, 2024 09:30
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Why I did it
Provide GNMI native write interface for configuration.

How I did it
Add configuration parameters for GNMI native write.

How to verify it
Check build pipeline.
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