Updated the if condition for maintenance release,6.1(5e) and variable name for impacted nodes.Also, updated pytest test cases

Author: Harinadh-Saladi

aci-preupgrade-validation-script.py

@@ -6059,7 +6059,7 @@ def n9k_c9408_model_lem_count_check(tversion, fabric_nodes, **kwargs):
     headers = ["Node ID", "Switch Model", "LEM Model", "LEM Count"]
     data = []
     recommended_action = (
-    "Upgrade from pre-16.1(2f) to 16.1(2f) or later on N9K-C9408 with 6 or more N9K-X9400-16W LEMs can result in boot loop. Choose a different version."
+        "Upgrade from pre-16.1(2f) to 16.1(2f) or later on N9K-C9408 with 6 or more N9K-X9400-16W LEMs can result in boot loop. Choose a different version."
     )
     doc_url = 'https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#n9k-c9408-with-6-n9k-x9400-16w-lems'
 
@@ -6437,4 +6437,4 @@ def main(_args=None):
         msg = "Abort due to unexpected error - {}".format(e)
         prints(msg)
         log.error(msg, exc_info=True)
-        sys.exit(1)
+        sys.exit(1)

docs/docs/validations.md

@@ -187,14 +187,18 @@ Items                                           | Defect       | This Script
 [Nexus 950X FM or LC Might Fail to boot after reload][d20] | CSCvg26013   | :white_check_mark: | :no_entry_sign:
 [Stale Decommissioned Spine][d21]               | CSCwf58763   | :white_check_mark: | :no_entry_sign:
 [N9K-C9408 Platform Model][d22]                 | CSCwk77800   | :white_check_mark: | :no_entry_sign:
-[N9K-C9408 with 6+ N9K-X9400-16W LEMs][d29]     | CSCws82819   | :white_check_mark: | :no_entry_sign:
 [PBR High Scale][d23]                           | CSCwi66348   | :white_check_mark: | :no_entry_sign:
 [Standby Sup Image Sync][d24]                   | CSCwi66348   | :white_check_mark: | :no_entry_sign:
 [Observer Database Size][d25]                   | CSCvw45531   | :white_check_mark: | :no_entry_sign:
 [Stale pconsRA Object][d26]                     | CSCwp22212   | :warning:{title="Deprecated"} | :no_entry_sign:
 [ISIS DTEPs Byte Size][d27]                     | CSCwp15375   | :white_check_mark: | :no_entry_sign:
+<<<<<<< HEAD
 [Policydist configpushShardCont Crash][d28]     | CSCwp95515   | :white_check_mark: | :no_entry_sign:
 [Auto Firmware Update on Switch Discovery][d29] | CSCwe83941   | :white_check_mark: | :no_entry_sign:
+=======
+[Policydist configpushShardCont Crash][d28]     | CSCwp95515   | :white_check_mark: | 
+[N9K-C9408 with 6+ N9K-X9400-16W LEMs][d29]     | CSCws82819   | :white_check_mark: | :no_entry_sign:
+>>>>>>> Updated the if condition for maintenance release,6.1(5e) and variable name for impacted nodes.Also, updated pytest test cases
 
 [d1]: #ep-announce-compatibility
 [d2]: #eventmgr-db-size-defect-susceptibility

tests/checks/n9k_c9408_model_lem_count_check/test_n9k_c9408_model_lem_count_check.py

@@ -36,7 +36,7 @@
             [],
             script.VER_NOT_AFFECTED,
         ),
-        # Version not affected (higher than 6.2(1g))
+        # Applicable version but no N9K-C9408 nodes found
         (
             {eqptLC_api: read_data(dir, "eqptLC_6_node.json")},
             "6.1(2f)",
@@ -54,7 +54,7 @@
             [],
             "",
         ),
-        # Applicable version, exactly 5 LEMs -> PASS
+        # Applicable version, C9408 exists, with <=5 LEMs -> PASS
         (
             {eqptLC_api: read_data(dir, "eqptLC_5_node.json")},
             "6.2(1g)",
@@ -63,6 +63,15 @@
             [],
             "",
         ),
+        # Applicable version with 6 LEMs -> FAIL_O
+        (
+            {eqptLC_api: read_data(dir, "eqptLC_6_node.json")},
+            "6.2(1g)",
+            read_data(dir, "fabricNode_n9k_c9408.json"),
+            script.FAIL_O,
+            [["101", "N9K-C9408", "N9K-X9400-16W", 6]],
+            "",
+        ),
         # Applicable mid-train version 6.1(5e), less than 6 LEMs on C9408 -> PASS
         (
             {eqptLC_api: read_data(dir, "eqptLC_5_node.json")},
@@ -81,14 +90,14 @@
             [["101", "N9K-C9408", "N9K-X9400-16W", 6]],
             "",
         ),
-        # Applicable mid-train version 6.1(5e), more than 6 LEMs on C9408 -> FAIL_O
+        # Version not affected (fixed after 6.1(5e))
         (
-            {eqptLC_api: read_data(dir, "eqptLC_7_node.json")},
-            "6.1(5e)",
+            {eqptLC_api: read_data(dir, "eqptLC_6_node.json")},
+            "6.1(5f)",
             read_data(dir, "fabricNode_n9k_c9408.json"),
-            script.FAIL_O,
-            [["101", "N9K-C9408", "N9K-X9400-16W", 7]],
-            "",
+            script.NA,
+            [],
+            script.VER_NOT_AFFECTED,
         ),
         # Applicable version, 6 LEMs on C9408 -> FAIL_O
         (
@@ -99,15 +108,6 @@
             [["101", "N9K-C9408", "N9K-X9400-16W", 6]],
             "",
         ),
-        # Applicable version, more than 6 LEMs on C9408 -> FAIL_O
-        (
-            {eqptLC_api: read_data(dir, "eqptLC_7_node.json")},
-            "6.1(2f)",
-            read_data(dir, "fabricNode_n9k_c9408.json"),
-            script.FAIL_O,
-            [["101", "N9K-C9408", "N9K-X9400-16W", 7]],
-            "",
-        ),
         # Count only C9408 nodes and only matching LEM model
         (
             {eqptLC_api: read_data(dir, "eqptLC_mixed.json")},