Add novus 10g ci

.github/workflows/publish.yml

@@ -2,17 +2,45 @@ name: cicd
 
 on:
   push:
-    branches: [main]
+    # branches: [main]
   pull_request:
 
+env:
+  ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
+
 jobs:
-  cicd:
-    runs-on: [self-hosted, Linux, Ubuntu, x64]
-    strategy:
-      max-parallel: 1
-      matrix:
-        python-version: [python27, python38]
+  # test_novus_100g:
+  #   runs-on: [snappi-ixn-ci-novus100g]
+  #   steps:
+  #     - name: Checkout source
+  #       uses: actions/checkout@v2
+  #       with:
+  #         ref: ${{ env.BASE_BRANCH }}
+  #         # ref: ${{ github.head_ref }}
+  #         submodules: recursive
+  #     - name: Set python path
+  #       id: path
+  #       run: echo "::set-output name=pythonv::/home/github-runner/pyenv/.env/bin/python"
+  #     - name: Install dependencies
+  #       run: |
+  #         rm -rf .env
+  #         ${{steps.path.outputs.pythonv}} do.py setup
+  #         ${{steps.path.outputs.pythonv}} do.py init
+  #     - name: Build distribution
+  #       run: |
+  #         ${{steps.path.outputs.pythonv}} do.py dist
+  #     - name: Install package on clean env
+  #       run: |
+  #         rm -rf .env
+  #         ${{steps.path.outputs.pythonv}} do.py setup
+  #         ${{steps.path.outputs.pythonv}} do.py install
+  #         ${{steps.path.outputs.pythonv}} do.py init
+  #     - name: Run tests
+  #       run: |
+  #         TEST_USERNAME=${{secrets.TEST_USERNAME}} TEST_PASSWORD='${{secrets.TEST_PASSWORD}}' ${{steps.path.outputs.pythonv}} do.py test
 
+  test_novus_10g:
+    runs-on: [snappi-ixn-ci-novus10g]
     steps:
       - name: Checkout source
         uses: actions/checkout@v2
@@ -22,7 +50,7 @@ jobs:
           submodules: recursive
       - name: Set python path
         id: path
-        run: echo "::set-output name=pythonv::/home/otg/${{matrix.python-version}}/bin/python"
+        run: echo "::set-output name=pythonv::/home/github-runner/pyenv/.env/bin/python"
       - name: Install dependencies
         run: |
           rm -rf .env
@@ -39,7 +67,31 @@ jobs:
           ${{steps.path.outputs.pythonv}} do.py init
       - name: Run tests
         run: |
-          ${{steps.path.outputs.pythonv}} do.py test
+          TEST_USERNAME=${{secrets.TEST_USERNAME}} ${{steps.path.outputs.pythonv}} do.py test novus10g
+
+  publish_artifacts:
+    # runs-on: [snappi-ixn-ci-novus100g]
+    runs-on: [snappi-ixn-ci-novus10g]
+    needs: [test_novus_10g]
+    # needs: [test_novus_100g, test_novus_10g]
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ env.BASE_BRANCH }}
+          # ref: ${{ github.head_ref }}
+          submodules: recursive
+      - name: Set python path
+        id: path
+        run: echo "::set-output name=pythonv::/home/github-runner/pyenv/.env/bin/python"
+      - name: Install dependencies
+        run: |
+          rm -rf .env
+          ${{steps.path.outputs.pythonv}} do.py setup
+          ${{steps.path.outputs.pythonv}} do.py init
+      - name: Build distribution
+        run: |
+          ${{steps.path.outputs.pythonv}} do.py dist
       - name: Get package version
         id: get_version
         run: |
@@ -74,16 +126,17 @@ jobs:
         run: |
           ${{steps.path.outputs.pythonv}} do.py install_requests ${{steps.path.outputs.pythonv}}
           ${{steps.path.outputs.pythonv}} do.py check_release_flag ${{ steps.release.outputs.release_flag }} ${{ steps.get_version.outputs.version }}
-  cicd_snappitest:
-    runs-on: [self-hosted, Linux, Ubuntu, x64]
-    needs: cicd
-    steps:
-      - name: Trigger CI/CD snappi-tests
-        run: |
-          git clone https://github.com/open-traffic-generator/snappi-tests.git
-          mv version.txt snappi-tests
-          cd snappi-tests
-          git add version.txt
-          git commit --allow-empty -m "Trigger Snappi test from snappi-ixnetwork"
-          git remote set-url origin https://${{secrets.CI_TOKEN}}@github.com/open-traffic-generator/snappi-tests.git
-          git push origin main
+
+  # cicd_snappitest:
+  #   runs-on: [snappi-ixn-ci-novus100g]
+  #   needs: test_novus_100g
+  #   steps:
+  #     - name: Trigger CI/CD snappi-tests
+  #       run: |
+  #         git clone https://github.com/open-traffic-generator/snappi-tests.git
+  #         mv version.txt snappi-tests
+  #         cd snappi-tests
+  #         git add version.txt
+  #         git commit --allow-empty -m "Trigger Snappi test from snappi-ixnetwork"
+  #         git remote set-url origin https://${{secrets.CI_TOKEN}}@github.com/open-traffic-generator/snappi-tests.git
+  #         git push origin main

do.py

@@ -35,35 +35,47 @@ def lint():
     )
 
 
-def test():
+def test(card="novus100g"):
     coverage_threshold = 67
-    #     args = [
-    #         '--location="https://10.39.71.97:443"',
-    #         (
-    #             '--ports="10.39.65.230;6;1 10.39.65.230;6;2 10.39.65.230;6;3'
-    #             ' 10.39.65.230;6;4"'
-    #         ),
-    #         '--media="fiber"',
-    #         "tests",
-    #         '-m "not e2e and not l1_manual"',
-    #         '--cov=./snappi_ixnetwork --cov-report term'
-    #         ' --cov-report html:cov_report',
-    #     ]
-    args = [
-        '--location="https://otg-novus100g.lbj.is.keysight.com:5000"',
-        (
-            '--ports="otg-novus100g.lbj.is.keysight.com;1;1'
-            " otg-novus100g.lbj.is.keysight.com;1;2"
-            " otg-novus100g.lbj.is.keysight.com;1;5"
-            ' otg-novus100g.lbj.is.keysight.com;1;6"'
-        ),
+    username = os.environ.get("TEST_USERNAME", "admin")
+    psd = os.environ.get("TEST_PASSWORD", "admin")
+
+    if card == "novus100g":
+        args = [
+            '--location="https://snappi-ixn-ci-novus100g.lbj.is.keysight.com:5000"',
+            (
+                '--ports="snappi-ixn-ci-novus100g.lbj.is.keysight.com;1;1'
+                " snappi-ixn-ci-novus100g.lbj.is.keysight.com;1;2"
+                " snappi-ixn-ci-novus100g.lbj.is.keysight.com;1;5"
+                ' snappi-ixn-ci-novus100g.lbj.is.keysight.com;1;6"'
+            ),
+            "--speed=speed_100_gbps",
+        ]
+    elif card == "novus10g":
+        args = [
+            '--location="https://novus1-715849.ccu.is.keysight.com:5000"',
+            (
+                '--ports="novus1-715849.ccu.is.keysight.com;1;1'
+                " novus1-715849.ccu.is.keysight.com;1;2"
+                " novus1-715849.ccu.is.keysight.com;1;5"
+                ' novus1-715849.ccu.is.keysight.com;1;6"'
+            ),
+            "--speed=speed_10_gbps",
+        ]
+    else:
+        raise Exception("card %s is not supported for testing" % card)
+
+    args += [
         "--ext=ixnetwork",
-        "--speed=speed_100_gbps",
+        "--username=" + username,
+        "--psd='" + psd + "'",
         "tests",
         '-m "not e2e and not l1_manual and not uhd"',
         "--cov=./snappi_ixnetwork --cov-report term"
         " --cov-report html:cov_report",
     ]
+    print(args)
+
     run(
         [
             py() + " -m pip install pytest-cov",
@@ -244,11 +256,13 @@ def get_workflow_id():
 
 
 def check_release_flag(release_flag=None, release_version=None):
-    if release_flag == '1':
+    if release_flag == "1":
         with open("setup.py") as f:
             out = f.read()
-            snappi_convergence = re.findall(r"\"snappi_convergence==(.+)\"",out)[0]
-        release_version = release_version.replace('v', "")
+            snappi_convergence = re.findall(
+                r"\"snappi_convergence==(.+)\"", out
+            )[0]
+        release_version = release_version.replace("v", "")
         with open("version.txt", "w+") as f:
             f.write("version: {}\n".format(release_version))
             f.write("snappi_convergence: {}\n".format(snappi_convergence))

readme.md

@@ -1,7 +1,7 @@
 # snappi Extension for IxNetwork
 
 [![license](https://img.shields.io/badge/license-MIT-green.svg)](https://en.wikipedia.org/wiki/MIT_License)
-[![Project Status: Active – The project has reached a stable, usable state and is being actively developed.](https://www.repostatus.org/badges/latest/active.svg)](https://www.repostatus.org/#active)
+[![Project Status: Active - The project has reached a stable, usable state and is being actively developed.](https://www.repostatus.org/badges/latest/active.svg)](https://www.repostatus.org/#active)
 [![Build](https://github.com/open-traffic-generator/snappi-ixnetwork/workflows/Build/badge.svg)](https://github.com/open-traffic-generator/snappi-ixnetwork/actions)
 [![pypi](https://img.shields.io/pypi/v/snappi_ixnetwork.svg)](https://pypi.org/project/snappi_ixnetwork)
 [![python](https://img.shields.io/pypi/pyversions/snappi_ixnetwork.svg)](https://pypi.python.org/pypi/snappi_ixnetwork)
@@ -78,3 +78,4 @@ while True:
     if all([m.frames_tx == 10000 == m.frames_rx for m in res.flow_metrics]):
         break
 ```
+

setup.py

@@ -5,7 +5,7 @@
 import setuptools
 
 pkg_name = "snappi_ixnetwork"
-version = "0.9.1"
+version = "0.11.5"
 
 # read long description from readme.md
 base_dir = os.path.abspath(os.path.dirname(__file__))
@@ -37,7 +37,7 @@
     install_requires=["ixnetwork-restpy>=1.0.52"],
     extras_require={
         "testing": [
-            "snappi==0.9.1",
+            "snappi==0.11.5",
             "snappi_convergence==0.4.1",
             "pytest",
             "mock",

snappi_ixnetwork/capture.py

@@ -83,7 +83,9 @@ def config(self):
                 imports.append(capture)
         for capture_item in self._api.snappi_config.captures:
             if capture_item.format == "pcap":
-                self._api.warning("pcap format is not supported for IxNetwork, setting capture format to pcapng")
+                self._api.warning(
+                    "pcap format is not supported for IxNetwork, setting capture format to pcapng"
+                )
                 capture_item.format = "pcapng"
             if capture_item.port_names is None:
                 continue
@@ -348,7 +350,7 @@ def results(self, request):
         self._api._request("POST", url, payload)
 
         path = "%s/capture" % self._api._ixnetwork.Globals.PersistencePath
-        #Todo: Revert dc to merged capture after fix is available in 9.20
+        # Todo: Revert dc to merged capture after fix is available in 9.20
         url = "%s/files?absolute=%s&filename=%s" % (
             self._api._ixnetwork.href,
             path,