Adding additional set of pass cases in tofino (#25)

* Add LAG hash seed test-case (opencomputeproject#184)

Signed-off-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* adding additional create remove cases (#14)

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* removing switch tunnel

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* resolving PR comments and merge conflict

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* resolving precommit checks

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* reverting sai and l2 basic

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Fixed u8/u16 and range Thrift conversions (opencomputeproject#188)

Signed-off-by: Andriy Kokhan <andriy.kokhan@gmail.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Updated PTF use-case. Enabled PTF TCs run from CI/CD (opencomputeproject#189)

Signed-off-by: Andriy Kokhan <andriy.kokhan@gmail.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Fix oper status check (opencomputeproject#192)

Signed-off-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Added support for string SAI_NULL_OBJECT_ID for Thrift RPC (opencomputeproject#194)

Signed-off-by: Vinod Kumar <vikumar7ks@gmail.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Remove unnecessary adding BPs to default VLAN on init (opencomputeproject#193)

Signed-off-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Adding new DASH TCs for create, set and remove API (opencomputeproject#195)

Signed-off-by: Vinod Kumar <vikumar7ks@gmail.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Adding new DASH TCs for create, set and remove API 2nd set (opencomputeproject#197)

Signed-off-by: Vinod Kumar <vikumar7ks@gmail.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Fixed set_sku_mode() (opencomputeproject#198)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Added FDB basic UTs (opencomputeproject#199)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Added LAG basic UTs (opencomputeproject#200)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Added FDB flush UTs (opencomputeproject#201)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* [CI/CD] Fixed checks what Docker images have to be rebuild (opencomputeproject#206)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Updated to SAI v1.13+ (opencomputeproject#205)

* Added nlohmann-json3-dev as new sairedis dependency
* Extended list of unsupported types in get_by_type()
---------
Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Fixed pytest warnings (opencomputeproject#207)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Refactored Redis RPC I/O failure processing (opencomputeproject#208)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Removed obsoleted exec option. Please use "--testbed" instead (opencomputeproject#209)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Add generic SAI Thrift server (opencomputeproject#203)

Signed-off-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* Fixed "--traffic" option behavior (opencomputeproject#210)

Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* line ending change

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>

* adding additional tofino vs pass cases

* adding skip all

* pytest

---------

Signed-off-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
Signed-off-by: Andriy Kokhan <andriy.kokhan@gmail.com>
Signed-off-by: Vinod Kumar <vikumar7ks@gmail.com>
Signed-off-by: Andriy Kokhan <andriy.kokhan@plvision.eu>
Co-authored-by: Yurii Lisovskyi <yurii.lisovskyi@plvision.eu>
Co-authored-by: Andriy Kokhan <andriy.kokhan@gmail.com>
Co-authored-by: vikumarks <119973184+vikumarks@users.noreply.github.com>
Co-authored-by: Andriy Kokhan <andriy.kokhan@plvision.eu>

tests/api/test_acl_counter.py

@@ -1,6 +1,14 @@
 from pprint import pprint
+import pytest
 
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
 
+
+@pytest.mark.npu
 class TestSaiAclCounter:
     # object with parent SAI_OBJECT_TYPE_ACL_TABLE
 

tests/api/test_acl_entry.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiAclEntry:
     # object with parent SAI_OBJECT_TYPE_ACL_TABLE
 

tests/api/test_acl_range.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiAclRange:
     # object with no parents
 

tests/api/test_acl_table.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiAclTable:
     # object with no parents
 

tests/api/test_acl_table_group.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiAclTableGroup:
     # object with no parents
 

tests/api/test_acl_table_group_member.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiAclTableGroupMember:
     # object with parent SAI_OBJECT_TYPE_ACL_TABLE_GROUP SAI_OBJECT_TYPE_ACL_TABLE
 

tests/api/test_ars.py

@@ -0,0 +1,35 @@
+from pprint import pprint
+import pytest
+
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
+class TestSaiArs:
+    # object with no attributes
+
+    def test_ars_create(self, npu):
+        commands = [
+            {
+                'name': 'ars_1',
+                'op': 'create',
+                'type': 'SAI_OBJECT_TYPE_ARS',
+                'attributes': [],
+            }
+        ]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values create =======')
+        pprint(results)
+
+    def test_ars_remove(self, npu):
+        commands = [{'name': 'ars_1', 'op': 'remove'}]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values remove =======')
+        pprint(results)

tests/api/test_ars_profile.py

@@ -0,0 +1,35 @@
+from pprint import pprint
+import pytest
+
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
+class TestSaiArsProfile:
+    # object with no attributes
+
+    def test_ars_profile_create(self, npu):
+        commands = [
+            {
+                'name': 'ars_profile_1',
+                'op': 'create',
+                'type': 'SAI_OBJECT_TYPE_ARS_PROFILE',
+                'attributes': [],
+            }
+        ]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values create =======')
+        pprint(results)
+
+    def test_ars_profile_remove(self, npu):
+        commands = [{'name': 'ars_profile_1', 'op': 'remove'}]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values remove =======')
+        pprint(results)

tests/api/test_buffer_pool.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiBufferPool:
     # object with no parents
 

tests/api/test_counter.py

@@ -0,0 +1,36 @@
+from pprint import pprint
+
+import pytest
+
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
+class TestSaiCounter:
+    # object with no attributes
+
+    def test_counter_create(self, npu):
+        commands = [
+            {
+                'name': 'counter_1',
+                'op': 'create',
+                'type': 'SAI_OBJECT_TYPE_COUNTER',
+                'attributes': [],
+            }
+        ]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values create =======')
+        pprint(results)
+
+    def test_counter_remove(self, npu):
+        commands = [{'name': 'counter_1', 'op': 'remove'}]
+
+        results = [*npu.process_commands(commands)]
+        print('======= SAI commands RETURN values remove =======')
+        pprint(results)

tests/api/test_debug_counter.py

@@ -1,6 +1,16 @@
 from pprint import pprint
 
+import pytest
 
+
+@pytest.fixture(scope='module', autouse=True)
+def skip_all(testbed_instance):
+    testbed = testbed_instance
+    if testbed is not None and len(testbed.npu) != 1:
+        pytest.skip('invalid for {} testbed'.format(testbed.name))
+
+
+@pytest.mark.npu
 class TestSaiDebugCounter:
     # object with no parents