Skip to content

[T2] Multi-asic support for everflow IPV6 mirror-session config tests#18155

Closed
sanjair-git wants to merge 1 commit intosonic-net:masterfrom
sanjair-git:eflow-ipv6
Closed

[T2] Multi-asic support for everflow IPV6 mirror-session config tests#18155
sanjair-git wants to merge 1 commit intosonic-net:masterfrom
sanjair-git:eflow-ipv6

Conversation

@sanjair-git
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes # (issue)

  • This PR adds multi-asic support for mirror configuration in everfow tests.
  • It applies to 'erspan-ipv6' tests under 'test_everflow_testbed' and 'test_everflow_ipv6' test suites.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

  • All 'erspan ipv6' evverflow tests were failing for multi-asic duts with the following error.
E               acl_loader.set_session_name(session_name)
E             File "/usr/local/lib/python3.11/dist-packages/acl_loader/main.py", line 326, in set_session_name
E               raise AclLoaderException("Session %s does not exist" % session_name)
E           acl_loader.main.AclLoaderException: Session test_session_1 does not exist

How did you do it?

  • If the duthost has multiple asics, apply the mirror config on each asic.
  • And also add the 'ERSPAN' session type in the mirror config.

How did you verify/test it?

  • Ran the tests on T2 multi-asic dut and made sure tests are passing.

Any platform specific information?

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

Documentation

image
image

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

flake8 fix

[18155] Remove duplicate policer config
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sanjair-git
Copy link
Copy Markdown
Contributor Author

Trigger /azp failed. Can you please help in rerunning /azp run. Thanks

@tjchadaga
Copy link
Copy Markdown
Contributor

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sanjair-git
Copy link
Copy Markdown
Contributor Author

Closing this PR as the changes are being taken care in #19594 with some additional changes for erspan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants