Removed other PR checks

Author: veenaskumar-cisco

aci-preupgrade-validation-script.py

@@ -6029,70 +6029,7 @@ def apic_downgrade_compat_warning_check(cversion, tversion, **kwargs):
 
     return Result(result=result, headers=headers, data=data, recommended_action=recommended_action, doc_url=doc_url)
 
-@check_wrapper(check_title="Tacacs server unresponsive check")
-def tacacs_server_unresponsive_check(fabric_nodes, tversion, username, password, **kwargs):
-    result = PASS
-    headers = ['APIC_Name', 'count']
-    data = []
-    recommended_action = "Contact Cisco TAC for Support before upgrade"
-    doc_url = "https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#tacacs-server-unresponsive-check"
-
-    if not tversion:
-        return Result(result=MANUAL, msg=TVER_MISSING)
-    if tversion.older_than("6.1(4h)"):
-        controllers = [node for node in fabric_nodes if node['fabricNode']['attributes']['role'] == 'controller']
-        if not controllers:
-            return Result(result=ERROR, msg="No fabricNode of APIC. Is the cluster healthy?", doc_url=doc_url)
-        has_error = False
-        for controller in controllers:
-            try:
-                connection = Connection(controller['fabricNode']['attributes']['address'])
-                connection.username = username
-                connection.password = password
-                connection.connect()
-                connection.cmd('cd /var/log/dme/log && zgrep -c "AAA server is unresponsive or too slow to respond" nginx.bin.log')
-                count = int(connection.output.strip())
-                if(count > 0):
-                    data.append([controller['fabricNode']['attributes']['name'], count])
-            except Exception as e:
-                has_error = True
-                data.append([controller['fabricNode']['attributes']['name'], str(e)])
-                
-        connection.close()
-
-        if has_error:
-            result = ERROR
-        elif data:
-            result = FAIL_O
-        return Result(result=result,headers=headers,data=data,recommended_action=recommended_action,doc_url=doc_url)
-    else:
-        return Result(result=PASS, msg=VER_NOT_AFFECTED)
-
 
-@check_wrapper(check_title='Auto Firmware Update on Switch Discovery')
-def auto_firmware_update_on_switch_check(cversion, tversion, **kwargs):
-    result = PASS
-    headers = ["Auto Firmware Update Status", "Default Firmware Version", "Upgrade Target Version"]
-    data = []
-    recommended_action = 'Disable Auto Firmware Update before the upgrade as a precaution. See the reference doc for details.'
-    doc_url = 'https://datacenter.github.io/ACI-Pre-Upgrade-Validation-Script/validations/#auto-firmware-update-on-switch-discovery'
-
-    if not tversion or not cversion:
-        return Result(result=MANUAL, msg=TVER_MISSING)
-
-    if tversion.older_than("6.0(3a)") or (
-        cversion.newer_than("6.0(3a)") or (cversion.major1 == "5" and cversion.newer_than("5.2(8a)"))
-    ):
-        return Result(result=NA, msg=VER_NOT_AFFECTED)
-
-    fwrepop = icurl("mo", "uni/fabric/fwrepop.json")
-    if fwrepop and fwrepop[0]["firmwareRepoP"]["attributes"]["enforceBootscriptVersionValidation"] == "yes":
-        data.append(["Enabled", fwrepop[0]["firmwareRepoP"]["attributes"]["defaultSwitchVersion"], str(tversion)])
-        result = MANUAL
-
-    return Result(result=result, headers=headers, data=data, recommended_action=recommended_action, doc_url=doc_url)
-
-
 @check_wrapper(check_title="svccoreCtrlr excessive entries check")
 def svccoreCtrlr_excessive_entries_check(tversion, **kwargs):
     result = PASS

docs/docs/validations.md

@@ -193,6 +193,8 @@ Items                                           | Defect       | This Script
 [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:
+[Policydist configpushShardCont Crash][d28]     | CSCwp95515   | :white_check_mark: | 
+[svccoreCtrlr excessive entries check][d30]     | CSCws8423    | :white_check_mark: | 
 [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:
 
@@ -224,6 +226,7 @@ Items                                           | Defect       | This Script
 [d26]: #stale-pconsra-object
 [d27]: #isis-dteps-byte-size
 [d28]: #policydist-configpushshardcont-crash
+[d30]: #svccoreCtrlr-excessive-entries-check
 [d29]: #auto-firmware-update-on-switch-discovery