Skip to content

ci: update OFED version to doca3.3.3-26.01-2.0.0.0.631-0 - #3573

Closed
nvidia-ci-cd wants to merge 1 commit into
masterfrom
ci/update-ofed-version-26.01-2.0.0.0.631-0
Closed

nvidia-ci-cd wants to merge 1 commit into
masterfrom
ci/update-ofed-version-26.01-2.0.0.0.631-0

Conversation

@nvidia-ci-cd

Copy link
Copy Markdown
Collaborator

Created by an internal DOCA CI job.

Signed-off-by: svc-cloud-orch-ci <svc-cloud-orch-ci@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Oct 23, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@greptile-apps

greptile-apps Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[Medium impact] The change reaches example cluster policies and the OFED release workflow.

Fix the webhook’s rejection of the selected driver version before merging.

Findings

  1. P1 Webhook rejects the new version ▶

Summary

Changes the DOCA driver version from doca3.6.0-26.10-059000-0 to doca3.3.3-26.01-2.0.0.0.631-0.

  • Example policies now select the new OFED driver build.
  • Release defaults now select the new OFED driver build.

Reviews (1) · Last reviewed commit: "ci: update OFED version to doca3.3.3-26...." · Reviewed by Greptile

Comment thread hack/release.yaml
image: doca-driver
repository: nvcr.io/nvstaging/mellanox
version: doca3.6.0-26.10-059000-0
version: doca3.3.3-26.01-2.0.0.0.631-0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Webhook rejects the new version

With validation enabled, the webhook rejects doca3.3.3-26.01-2.0.0.0.631-0. isValidOFEDVersion accepts four dotted build-number groups followed by a hyphen, or at least six digits; 2.0.0.0.631 matches neither. All seven updated examples therefore fail to create or update a NicClusterPolicy with “invalid OFED version.” Extend the validator and add a test for this format before selecting it.

@rollandf rollandf closed this Oct 25, 2026
@github-actions
github-actions Bot deleted the ci/update-ofed-version-26.01-2.0.0.0.631-0 branch October 27, 2026 13:21
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.

2 participants