Skip to content

[snmp] Add wait portchannel up in test_snmp_fdb#13982

Merged
wangxin merged 1 commit intosonic-net:masterfrom
yaqiangz:azure-master_fix_test_snmp_fdb
Aug 7, 2024
Merged

[snmp] Add wait portchannel up in test_snmp_fdb#13982
wangxin merged 1 commit intosonic-net:masterfrom
yaqiangz:azure-master_fix_test_snmp_fdb

Conversation

@yaqiangz
Copy link
Copy Markdown
Contributor

@yaqiangz yaqiangz commented Aug 5, 2024

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?

In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?

Add time wait for portchannel up

How did you verify/test it?

Run test on t0 and m0. passed

Any platform specific information?

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

Documentation

@yaqiangz yaqiangz changed the title [snmp] Add wait pc up in test_snmp_fdb [snmp] Add wait portchannel up in test_snmp_fdb Aug 6, 2024
@yaqiangz yaqiangz marked this pull request as ready for review August 6, 2024 09:07
@yaqiangz
Copy link
Copy Markdown
Contributor Author

yaqiangz commented Aug 6, 2024

Hi @wangxin could you please help to merge this PR?

@wangxin wangxin merged commit 5b60f18 into sonic-net:master Aug 7, 2024
@yaqiangz
Copy link
Copy Markdown
Contributor Author

yaqiangz commented Aug 7, 2024

Hi @yxieca @bingwang-ms could you please help to approve the backport request?

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 8, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 8, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202405: #14048

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202311: #14047

mssonicbld pushed a commit that referenced this pull request Aug 9, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
mssonicbld pushed a commit that referenced this pull request Aug 9, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
arista-hpandya pushed a commit to arista-hpandya/sonic-mgmt that referenced this pull request Oct 2, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
vikshaw-Nokia pushed a commit to vikshaw-Nokia/sonic-mgmt that referenced this pull request Oct 23, 2024
What is the motivation for this PR?
In some scenario that this case would fail because dummy MAC count miss portchannel data. Hence add time wait

How did you do it?
Add time wait for portchannel up

How did you verify/test it?
Run test on t0 and m0. passed
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.

5 participants