ci: update OFED version to doca3.3.0-26.01-0.9.5.0.839-0#2639
ci: update OFED version to doca3.3.0-26.01-0.9.5.0.839-0#2639nvidia-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 CI PR replaces the OFED/DOCA driver version string across all example CRs and
Confidence Score: 4/5The change is mechanically consistent across all 8 files, but rolls the driver back to an older DOCA/OFED release rather than forward — human confirmation that this downgrade is deliberate is recommended before merging. Every file is updated identically and no logic, schema, or configuration is broken. The only concern is that the automated CI job produced a version regression (3.4.0 → 3.3.0, 26.04 → 26.01) rather than a forward bump, which could indicate the job picked up the wrong build or was triggered for an older release branch. hack/release.yaml — contains both Mofed and MofedStigFips entries and is the canonical source of truth for the version; worth a second look to confirm the intended target version. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[hack/release.yaml] -->|Mofed version| B["doca3.4.0-26.04-0.8.6.0-0 ➜ doca3.3.0-26.01-0.9.5.0.839-0"]
A -->|MofedStigFips version| B
C[7x example NicClusterPolicy CRs] -->|ofedDriver.version| B
B --> D{Intended downgrade?}
D -->|Yes| E[Safe to merge]
D -->|No| F[Revert needed]
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 |
There was a problem hiding this comment.
Version downgrade, not an upgrade
This PR rolls the OFED/DOCA version back from doca3.4.0-26.04-0.8.6.0-0 to doca3.3.0-26.01-0.9.5.0.839-0 — both the DOCA major version (3.4 → 3.3) and the MLNX_OFED stream (26.04 → 26.01) regress. The PR title calls it an "update," but given that the source is an automated internal CI job it is worth confirming that this downgrade is intentional and that the newer 3.4.0 driver was deliberately superseded rather than accidentally reverted. The same rollback is applied identically across all 7 example CRs and release.yaml.
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.