Skip to content

[Marvell-Teralynx] [Everflow_Policer] - Fix python3 error in ERSPAN Header Format #18306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gnanapriya27
Copy link
Contributor

@Gnanapriya27 Gnanapriya27 commented May 9, 2025

Description of PR

Fixed a pattern matching issue in case of everflow policer tests as part of checkMirroredFlow fixture, py3 upgrade is added for marvell platform specific ERSPAN Header format.

Summary:
Fix applicable for specific platform vendor.

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?

PY3 migration for vendor specific packet format to validate the test on marvell platform.

How did you do it?

Added byte conversion.

How did you verify/test it?

Verified by running all the everflow_policer PTF tests.

Any platform specific information?

Marvell specific platform

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

NA

Documentation

…SPAN Header Format

Signed-off-by: Gnanapriya Sethuramarajan <[email protected]>
Copy link

linux-foundation-easycla bot commented May 9, 2025

CLA Signed


The committers listed above are authorized under a signed CLA.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Gnanapriya27 Gnanapriya27 marked this pull request as draft May 9, 2025 09:27
@Gnanapriya27 Gnanapriya27 changed the title PY3 upgrade in everflow policer test for marvell-teralynx specific ERSPAN Header Format [Marvell-Teralynx] [Everflow_Policer] - Fix python3 error in ERSPAN Header Format May 9, 2025
@Gnanapriya27
Copy link
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Gnanapriya27 Gnanapriya27 marked this pull request as ready for review May 19, 2025 13:10
@Gnanapriya27
Copy link
Contributor Author

/apt run

@Gnanapriya27
Copy link
Contributor Author

/AzurePipelines run Azure.sonic-mgmt

Copy link

Commenter does not have sufficient privileges for PR 18306 in repo sonic-net/sonic-mgmt

@Gnanapriya27
Copy link
Contributor Author

/azp run

Copy link

Commenter does not have sufficient privileges for PR 18306 in repo sonic-net/sonic-mgmt

@Gnanapriya27
Copy link
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Gnanapriya27
Copy link
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-mgmt

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Gnanapriya27
Copy link
Contributor Author

@Blueve, @yxieca - Please review the PR

@yxieca yxieca requested a review from bingwang-ms June 6, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants