Skip to content

Added pre-upgrade validation for N9K-C9408 with more than 6 N9K-X9400-16W LEM's for the bug CSCws82819#354

Open
Harinadh-Saladi wants to merge 5 commits intodatacenter:masterfrom
Harinadh-Saladi:CSCws82819-c9408-boot-loop-check
Open

Added pre-upgrade validation for N9K-C9408 with more than 6 N9K-X9400-16W LEM's for the bug CSCws82819#354
Harinadh-Saladi wants to merge 5 commits intodatacenter:masterfrom
Harinadh-Saladi:CSCws82819-c9408-boot-loop-check

Conversation

@Harinadh-Saladi
Copy link

@Harinadh-Saladi Harinadh-Saladi commented Feb 25, 2026

  1. Added check for the bug CSCws82819 which has switch model N9K-C9408 with more than 6 N9K-X9400-16W LEM's in aci-preupgrade-validation-script.py
  2. Added 11 test cases including Pass, Fail and NA cases.
  3. Added documentation about the check and also about recommended action.

Tested in the lab and enclosed PASS, FAIL and NA scenarios.

CSCws82819_Logs.txt

CSCws82819_Pytest_Logs.txt

Note: FAIL and PASS logs enclosed in the attachment are simulated logs based on the available switch and line card model as we don't have the models N9K-C9408 and N9K-X9400-16W in our lab.

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 LEMs will result in boot loop. Do NOT proceed. Use fewer LEMs or choose a different version. "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modify the recommended action message

if tversion.older_than("6.1(2f)") or tversion.newer_than("6.2(1g)"):
return Result(result=NA, msg=VER_NOT_AFFECTED)

c9408_nodes = {}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use general variable match with existing structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants