Skip to content

[action] [PR:12796] [Nokia-7215] Skip test_dynamic_acl on Nokia-7215 platform#12826

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12796
May 13, 2024
Merged

[action] [PR:12796] [Nokia-7215] Skip test_dynamic_acl on Nokia-7215 platform#12826
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12796

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Skip test_dynamic_acl on Nokia-7215 platform. This feature cannot be supported on Nokia-7215 before 202411 branch cut-over.

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

Skip test_dynamic_acl on Nokia-7215 platform. This feature cannot be supported on Nokia-7215 before 202411 branch cut-over.

How did you do it?

Use conditional mark.

How did you verify/test it?

Verified on Nokia-7215 M0 testbed.

=============================================== short test summary info ================================================
SKIPPED [18] generic_config_updater/test_dynamic_acl.py: Skip on unsupported platforms
SKIPPED [4] generic_config_updater/test_dynamic_acl.py:1166: Skip on unsupported platforms
=========================================== 22 skipped, 3 warnings in 45.82s ===========================================

Any platform specific information?

Nokia-7215 platform specific.

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

Documentation

…12796)

What is the motivation for this PR?
Skip test_dynamic_acl on Nokia-7215 platform. This feature cannot be supported on Nokia-7215 before 202411 branch cut-over.

How did you do it?
Use conditional mark.
@mssonicbld
Copy link
Collaborator Author

Original PR: #12796

@mssonicbld mssonicbld merged commit 769ff66 into sonic-net:202311 May 13, 2024
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