Skip to content

Unsupported hardware check#364

Open
dhaselva wants to merge 3 commits intodatacenter:v4.1.0-devfrom
dhaselva:dhaselva/N2k_fex
Open

Unsupported hardware check#364
dhaselva wants to merge 3 commits intodatacenter:v4.1.0-devfrom
dhaselva:dhaselva/N2k_fex

Conversation

@dhaselva
Copy link
Copy Markdown

@dhaselva dhaselva changed the base branch from master to v4.1.0-dev April 9, 2026 10:16
@monrog2
Copy link
Copy Markdown
Collaborator

monrog2 commented Apr 29, 2026

address review and paste in run results based on pytests.

@dhaselva
Copy link
Copy Markdown
Author

Comment thread docs/docs/validations.md
Comment on lines +270 to +276
The script checks the presence of deprecated hardware in the fabric.

Refer the [Release Note 15.0(1) of ACI switches][3] to see the list of unsuporrted hardware for your desired target versions. Prior upgrading to target version, replace the unsupported hardware elements in your fabric with other supported hardware.

Contact cisco TAC for further assistance.


Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add some lines here explaining that the check is now populated with supported.unsupported HW from all release notes.

Comment thread docs/docs/validations.md Outdated
Comment on lines +5 to +6
"dn": "invalid-dn-format",
"model": "N2K-C2232PP-10GE"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do we have this documented with the BE?

Comment thread aci-preupgrade-validation-script.py
eqptExtCh = "eqptExtCh.json"
eqptSupC = "eqptSupC.json"

@pytest.mark.parametrize(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

add 1 test with the unformatted DN to prove the check works/output is formatted ok with unformatted DN

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