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

upgrade to snappi 0.11.14 #131

Merged

Conversation

chrispsommers
Copy link
Contributor

  • Upgrade snappi python client to 0.11.14 to support new ixiac controller ver 0.0.1-4064, which removes TLS 1.0 and 1.1 for security reasons (was flagged by MSFT in Ixia controller security risk (TLS 1.1 enabled) sonic-net/DASH#357)
  • Change client connection to ixia-c controller from port 443 to 8443, which is new default for the controller

Note, any test fixtures need to utilize correct version of ixia-c controller and ixia-c traffic engines in your test fixtures. This change only modifies the snappi library and connection port in SAI-Challenger. See snappi/ixia-c release notes for details.

This change is required to be pulled into DASH as a submodule in order to resolve #357

Note, some deprecation warnings will result from API changes in snappi. These can be seen when running DASH tests, for example see below. These should be migrated inside SAIChallenger as time allows.

[WARNING]: set_capture_state api is deprecated, Please use set_control_state with port.capture choice instead
[WARNING]: CaptureState is deprecated, Please use StatePortCapture instead

@andriy-kokhan andriy-kokhan merged commit cc78686 into opencomputeproject:main May 24, 2023
andriy-kokhan pushed a commit that referenced this pull request May 24, 2023
* Bump snappi to latest.
* Changed ixia-c controller port from 443 to 8443 per snappi/ixia-c upgrade.
andriy-kokhan pushed a commit that referenced this pull request May 24, 2023
* Bump snappi to latest.
* Changed ixia-c controller port from 443 to 8443 per snappi/ixia-c upgrade.
andriy-kokhan pushed a commit that referenced this pull request May 24, 2023
* Bump snappi to latest.
* Changed ixia-c controller port from 443 to 8443 per snappi/ixia-c upgrade.
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.

Ixia controller security risk (TLS 1.1 enabled)
2 participants