add spc2 asic

Author: isabelmsft

generic_config_updater/field_operation_validators.py

@@ -31,10 +31,14 @@ def get_asic_name():
         hwsku = output.rstrip('\n')
         if asic_type == 'mellanox' or asic_type == 'vs':
             spc1_hwskus = asic_mapping["mellanox_asics"]["spc1"]
+            spc2_hwskus = asic_mapping["mellanox_asics"]["spc2"]
             spc3_hwskus = asic_mapping["mellanox_asics"]["spc3"]
             if hwsku.lower() in [spc1_hwsku.lower() for spc1_hwsku in spc1_hwskus]:
                 asic = "spc1"
                 return asic
+            if hwsku.lower() in [spc2_hwsku.lower() for spc2_hwsku in spc2_hwskus]:
+                asic = "spc2"
+                return asic
             if hwsku.lower() in [spc3_hwsku.lower() for spc3_hwsku in spc3_hwskus]:
                 asic = "spc3"
                 return asic

generic_config_updater/gcu_field_operation_validators.conf.json

@@ -18,6 +18,7 @@
         "rdma_config_update_validator": {
             "mellanox_asics": {
                 "spc1": [ "ACS-MSN2700", "ACS-MSN2740", "ACS-MSN2100", "ACS-MSN2410", "ACS-MSN2010", "Mellanox-SN2700", "Mellanox-SN2700-D48C8" ],
+                "spc2": [ "ACS-MSN3800", "Mellanox-SN3800-D112C8" ],
                 "spc3": [ "ACS-MSN4700", "ACS-MSN4600", "ACS-MSN4600C", "ACS-MSN4410", "Mellanox-SN4600C-D112C8", "Mellanox-SN4600C-C64" ]
             },
             "broadcom_asics": {
@@ -44,6 +45,7 @@
                         "operations": ["remove", "add", "replace"],
                         "platforms": {
                             "spc1": "20181100",
+			    "spc2": "20191100",
                             "spc3": "20220500",
                             "td2": "20181100",
                             "th": "20181100",
@@ -68,6 +70,7 @@
                         "operations": ["replace"],
                         "platforms": {
                             "spc1": "20191100",
+			    "spc2": "20191100",
                             "spc3": "20220500",
                             "td2": "",
                             "th": "20221100",
@@ -90,6 +93,7 @@
                         "operations": ["replace"],
                         "platforms": {
                             "spc1": "20181100",
+			    "spc2": "20191100",
                             "spc3": "20220500",
                             "td2": "20181100",
                             "th": "20181100",
@@ -105,6 +109,7 @@
                         "operations": ["replace"],
                         "platforms": {
                             "spc1": "20191100",
+			    "spc2": "20191100",
                             "spc3": "20220500",
                             "td2": "",
                             "th": "20221100",

tests/generic_config_updater/field_operation_validator_test.py

@@ -119,6 +119,14 @@ def test_get_asic_spc1(self, mock_popen, mock_get_sonic_version_info):
         mock_popen.return_value.communicate.return_value = ["Mellanox-SN2700-D48C8", 0]
         self.assertEqual(fov.get_asic_name(), "spc1")
 
+    @patch('sonic_py_common.device_info.get_sonic_version_info')
+    @patch('subprocess.Popen')
+    def test_get_asic_spc2(self, mock_popen, mock_get_sonic_version_info):
+        mock_get_sonic_version_info.return_value = {'asic_type': 'mellanox'}
+        mock_popen.return_value = mock.Mock()
+        mock_popen.return_value.communicate.return_value = ["ACS-MSN3800", 0]
+        self.assertEqual(fov.get_asic_name(), "spc2")
+    
     @patch('sonic_py_common.device_info.get_sonic_version_info')
     @patch('subprocess.Popen')
     def test_get_asic_spc3(self, mock_popen, mock_get_sonic_version_info):