ci: update OFED version to doca3.3.0-26.01-0.9.5.0.840-0#2640
ci: update OFED version to doca3.3.0-26.01-0.9.5.0.840-0#2640nvidia-ci-cd wants to merge 1 commit into
Conversation
Signed-off-by: svc-cloud-orch-ci <svc-cloud-orch-ci@nvidia.com>
Greptile SummaryThis automated PR from the internal DOCA CI job updates the OFED/DOCA driver version string across all example CRs and
Confidence Score: 3/5The change is a version string replacement that goes to a numerically lower DOCA and OFED version; merging without confirming the intent carries risk of shipping a downgraded driver. Every changed file consistently replaces hack/release.yaml is the most impactful file since it controls both Mofed and MofedStigFips release entries; the 7 example CRs mirror the same change uniformly. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[DOCA Internal CI Job] -->|triggers PR| B[Version Update]
B --> C[hack/release.yaml\nMofed + MofedStigFips]
B --> D[example/crs/*.yaml\n7 NicClusterPolicy CRs]
C --> E[doca3.4.0-26.04-0.8.6.0-0\n→ doca3.3.0-26.01-0.9.5.0.840-0]
D --> E
E --> F[nvcr.io/nvstaging/mellanox\ndoca-driver image]
Reviews (1): Last reviewed commit: "ci: update OFED version to doca3.3.0-26...." | Re-trigger Greptile |
| image: doca-driver | ||
| repository: nvcr.io/nvstaging/mellanox | ||
| version: doca3.4.0-26.04-0.8.6.0-0 | ||
| version: doca3.3.0-26.01-0.9.5.0.840-0 | ||
| MofedStigFips: | ||
| image: doca-driver-stig-fips |
There was a problem hiding this comment.
Version appears to be a downgrade
The DOCA version is changing from doca3.4.0-26.04-0.8.6.0-0 to doca3.3.0-26.01-0.9.5.0.840-0, which is a step backward in both the DOCA major version (3.4.0 → 3.3.0) and the OFED version (26.04 → 26.01). If this is an intentional switch to a different (e.g., more stable) release branch rather than a rollback, a note in the PR description explaining the rationale would help reviewers confirm this is correct.
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!
Created by an internal DOCA CI job.