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

CASMNET-2301-fixing-VRF-test #633

Merged
merged 7 commits into from
Apr 1, 2025
Merged

Conversation

marumugam-hpe
Copy link
Contributor

@marumugam-hpe marumugam-hpe commented Mar 18, 2025

Summary and Scope

This change was made to allow customization of VRF name used for customers. Only canu test was modified.

Previously canu test had a hard coded Customer VRF. This was both wrong in that the PoR name is CSM, and didn't allow for customization. The new code defaults the VRF to CSM but allows command-line based customization.

This code also fixes a logic issue in the VRF neighbors test. Previously the test checked for a return of No OSPF Neighbors from the command line. The original test allowed errors where no VRF was configured to show up as a test success. The new logic catches both no neighbors and misconfiguration errors more effectively, though the framework does not allow dissecting which specific error occurred.

Issues and Related PRs

https://jira-pro.it.hpe.com:8443/browse/CASMNET-2301

Testing

Tested on spine switch 001/002 on Surtur.

@marumugam-hpe marumugam-hpe requested a review from a team as a code owner March 18, 2025 22:14
@spillerc-hpe
Copy link
Contributor

I installed your RPM on surtur, there is a second OSPF test that has Customer hard coded.

| 45 | sw-spine-002    | OSPF vrf Customer EXSTART Neighbors          | PASS     |                                                                                            |

Looks like your change is also needed here

- name: OSPF vrf Customer EXSTART Neighbors
task: show ip ospf neighbors vrf Customer
test: ncontains
pattern: EXSTART
err_msg: |
OSPF neighbors found in EXSTART state.
This is commonly caused by a MTU mismatch between neighbors.
Execute "show ip interface" to see the MTU of L3 interfaces.
device:
- leaf
- leaf-bmc
- cdu
- spine
pre_install: True

Somewhat distressingly the test passes because even though the Customer VRF isn't configured, the output does indeed not contain EXSTART.

sw-spine-001# show ip ospf neighbors vrf Customer
VRF Customer is not configured.

@rustydb rustydb mentioned this pull request Mar 28, 2025
3 tasks
Copy link
Contributor

@trad511 trad511 left a comment

Choose a reason for hiding this comment

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

@marumugam-hpe good work. In the future a bit more description in the PR is appropriate. I correct it in this one.

@trad511 trad511 merged commit 5156f5f into main Apr 1, 2025
21 checks passed
@trad511 trad511 deleted the CASMNET-2301-fix-customer-vrf-test branch April 1, 2025 19:31
rustydb added a commit to Cray-HPE/csm that referenced this pull request Apr 1, 2025
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.

4 participants