Skip to content

[action] [PR:9446] fix tcpdump issue in bgp update timer testing#9472

Merged
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/9446
Aug 16, 2023
Merged

[action] [PR:9446] fix tcpdump issue in bgp update timer testing#9472
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/9446

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Fix test_bgp_update_timer.py issue on T0 and T1, which caused by background running tcpdump issue.

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Fix test_bgp_update_timer.py issue on T0 and T1, which caused by background running tcpdump.
In script using "nohup <> &" running without waiting for tcpdump tool real running on DUT.
So in some failure case, bgp update sendout but tcpdump still not ready to capture packets and cause failure with saying bgp update not found.

How did you do it?

Wait until tcpdump tool running on DUT before continue testing

How did you verify/test it?

Keep running 10 times on KVM environment without failure.

Any platform specific information?

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

Documentation

* fix tcpdump issue in bgp update timer testing
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #9446

Copy link
Copy Markdown
Contributor

@jcaiMR jcaiMR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jcaiMR jcaiMR requested a review from StormLiangMS August 16, 2023 06:37
@mssonicbld mssonicbld merged commit d4049ed into sonic-net:202205 Aug 16, 2023
@mssonicbld mssonicbld deleted the cherry/202205/9446 branch February 4, 2024 09:24
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