Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NewValidation: CSCwf58763 #164

Open
welkin-he opened this issue Sep 24, 2024 · 2 comments
Open

NewValidation: CSCwf58763 #164

welkin-he opened this issue Sep 24, 2024 · 2 comments
Assignees

Comments

@welkin-he
Copy link
Collaborator

(use upvote 👍 for attentions)

Validation Type

[x] - Bug

What needs to be validated

fabricRsDecommissionNode mo with node-id match any existing spine switches, while upgrading to 5.2(3e) and above unitl 6.0(3d)

Why it needs to be validated

The stale mo can trigger spine get removed from coop adjacency list for all leaf switches, while keep the spine participating forwarding decision without the latest insight of the fabric.

Additional context

Add any other context about the feature request here.

@monrog2
Copy link
Collaborator

monrog2 commented Sep 25, 2024

#166 asked to check for fabricRsDecommissionNode in general.

CSCwf58763 specifically says to flag for DNs for node IDs that exist, but may be currently in a different pod compared to its current location.

"In this example, Node 401/402 are Spines in Pod 2 but they are stale as if they are in Pod 1.

node-1# moquery fabricRsDecommissionNode | egrep "^dn|^_svc"
_svc                : policydist
dn                  : uni/fabric/outofsvc/rsdecommissionNode-[topology/pod-1/node-402]
_svc                : policydist
dn                  : uni/fabric/outofsvc/rsdecommissionNode-[topology/pod-1/node-401]
_svc                : policymgr
dn                  : uni/fabric/outofsvc/rsdecommissionNode-[topology/pod-1/node-401]
_svc                : policymgr
dn                  : uni/fabric/outofsvc/rsdecommissionNode-[topology/pod-1/node-402]

"

@welkin-he
Copy link
Collaborator Author

cool, thanks for putting two validation together.
Here is the initial commit for peer view.
https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script/tree/welkin-issue-164

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

No branches or pull requests

2 participants