fix: RA2.2 README still references Network Operator v26.1.0

[andrewwhitecdw]

This PR corrects the documentation in profiles/spectrum-x-ra2.2/README.md: RA2.2 README still references Network Operator v26.1.0.

Changes

• profiles/spectrum-x-ra2.2/README.md: update the prerequisite and Helm installation example to Network Operator v26.4.0.
• tests/test_spectrum_x_readme_versions.sh: add a regression check that fails if the README references v26.1.0 or omits v26.4.0.
• Makefile: add a shell-tests target and include it in the ci target so the regression check runs locally.
• .github/workflows/ci.yml: add a "Run shell regression tests" step to the test job so the check runs in CI.

Details

--- a/profiles/spectrum-x-ra2.2/README.md
+++ b/profiles/spectrum-x-ra2.2/README.md
@@ -1,3 +1,3 @@
-2. NVIDIA Network Operator v26.1.0 or later
-...
-  --version v26.1.0 \\
+2. NVIDIA Network Operator v26.4.0 or later
+...
+  --version v26.4.0 \\

Tests

make shell-tests

Result: PASS

go test ./pkg/networkoperatorplugin/... -count=1

Result: PASS

Contributor guidelines

Per this repo's CONTRIBUTING.md:

• All commits are signed off (Signed-off-by trailer, DCO).

[greptile-apps]

Greptile Summary

This PR updates the Spectrum-X RA2.2 documentation to require Network Operator v26.4.0 and adds automated regression coverage.

• Updates both the prerequisite and Helm installation version.
• Adds a shell regression-test target and runs it from CI.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
profiles/spectrum-x-ra2.2/README.md	Updates RA2.2 Network Operator references from v26.1.0 to v26.4.0.
tests/test_spectrum_x_readme_versions.sh	Adds assertions preventing the RA2.1 version from returning and requiring the RA2.2 version.
Makefile	Adds a shell-tests target and incorporates it into the aggregate CI target.
.github/workflows/ci.yml	Runs the shell regression-test target in the GitHub Actions test job.

_{Reviews (2): Last reviewed commit: "fix: RA2.2 README still references Netwo..." | Re-trigger Greptile}

[greptile-apps]

Regression test bypasses automation

The new README assertions are not invoked by any CI workflow or Make test target, so future version regressions can pass CI unless someone runs this script manually.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

[andrewwhitecdw]

@greptile-apps Thanks for the review. I wired the new shell regression test into both the Makefile (make shell-tests, now part of make ci) and the CI workflow test job, and made the script executable. make shell-tests and the Go tests both pass. Amended into the existing commit (6350e41).

[greptile-apps]

Everything looks good. The Makefile shell-tests target correctly globs tests/*.sh and runs each script with bash, ci now depends on shell-tests, and the CI workflow has a dedicated step for it. The script was added with set -euo pipefail so failures will propagate correctly.

One minor nit: the shell-tests target in the Makefile invokes scripts with bash "$$script" rather than relying on the shebang, which is fine and actually more explicit — no issue there.

LGTM. This is ready to merge.