Skip to content
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

Add Snappi-Ixload package for docker-sonic-mgmt #20808

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

albertovillarreal-keys
Copy link

Adding additional python packages needed for IxLoad Snappi for DASH smartswitch testcases.

Why I did it

Test cases for DASH smartswitch will require added python packages

Work item tracking
  • Microsoft ADO (number only):

How I did it

Built a Docker image for sonic-mgmt on my local setup.

How to verify it

Ran an Ixload and IxNetwork snappi test to validate.

Which release branch to backport (provide reason below if selected)

None

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • 202405

Description for the changelog

Added python packages ipaddress, macaddress, and snappi_ixload.

Link to config_db schema for YANG module changes

na

A picture of a cute animal (not mandatory but encouraged)

Adding additional python packages needed for IxLoad Snappi for DASH smartswitch testcases.
removing flask module
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Pull request contains merge conflicts.

@albertovillarreal-keys albertovillarreal-keys marked this pull request as ready for review January 22, 2025 13:26
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@albertovillarreal-keys albertovillarreal-keys changed the title Snappi-Ixload Snappi-Ixload for docker-sonic-mgmt Jan 22, 2025
Copy link
Contributor

@r12f r12f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's update the PR title to be more descriptive. The change looks good to me.

@r12f r12f requested a review from yxieca January 22, 2025 21:13
@albertovillarreal-keys albertovillarreal-keys changed the title Snappi-Ixload for docker-sonic-mgmt Add Snappi-Ixload package for docker-sonic-mgmt Jan 29, 2025
ipython==5.4.1 \
ixload \
ixnetwork-restpy==1.0.64 \
ixnetwork-open-traffic-generator==0.0.79 \
jinja2==2.10.1 \
snappi[ixnetwork,convergence]==0.5.11 \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's already a version for this on L195, pls remove

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls make sure there are no duplicate packages

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

4 participants