Skip to content

[action] [PR:8172] TestAclReboot test fix for Nokia-400G-T2 chassis#8759

Merged
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/8172
Jun 29, 2023
Merged

[action] [PR:8172] TestAclReboot test fix for Nokia-400G-T2 chassis#8759
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/8172

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)

  • TestAclWithReboot ACL test case is not waiting long enough for the interfaces to come up with respect to Nokia IXR7250E-36x400G DUT.
  • Due to this, ACL tests' call (under TestAclWithReboot) start thinking all interfaces are up and fails.

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?

  • This PR helps to fix the TestAclWithReboot ACL test cases w.r.t Nokia IXR7250E-36x400G platform.
  • Test cases are not waiting long enough for the interfaces to come up with respect to Nokia IXR7250E-36x400G DUT.

How did you do it?

  • If the platform is 'Nokia IXR7250E-36x400G', wait long enough and make sure all the interfaces and transceivers are UP & running.
  • Once the ports are up, then proceed with running the ACL test calls.

How did you verify/test it?

  • Ran all the ACL test cases against a multi-asic line card in a T2 chassis including Nokia IXR7250E-36x400G.

Any platform specific information?

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

Documentation

ACL-Apr27

* TestAclReboot test fix for Nokia-400G-T2 DUT

* Indentation changes

* Adding #noqa to import warning

* Revert "Adding #noqa to import warning"

This reverts commit ec6f765.

* Precommit error fixes
@mssonicbld
Copy link
Collaborator Author

Original PR: #8172

@mssonicbld mssonicbld merged commit 47c5f1c into sonic-net:202205 Jun 29, 2023
@mssonicbld mssonicbld deleted the cherry/202205/8172 branch February 4, 2024 08:27
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