ci: update ib-kubernetes version to c89c59e (of branch network-operator-26.1.x)#2759
ci: update ib-kubernetes version to c89c59e (of branch network-operator-26.1.x)#2759nvidia-ci-cd wants to merge 1 commit into
Conversation
Signed-off-by: nvidia-ci-cd <svc-cloud-orch-gh@nvidia.com>
Greptile SummaryAutomated CI bump of the
Confidence Score: 4/5Safe to merge; all three version references are updated consistently and the target registry is staging. The update is fully consistent across all three files and is automated by a CI bot. The only notable concern is that a bare commit SHA replaces a descriptive version tag, which reduces traceability in changelogs and makes rollbacks less intuitive — but this does not affect runtime correctness as long as the image exists at that tag in the staging registry.
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[CI Workflow Run\n28871934641] -->|Triggered by| B[ib-kubernetes branch\nnetwork-operator-26.1.x]
B -->|Produces image at| C[nvcr.io/nvstaging/mellanox\n/ib-kubernetes:c89c59e]
C -->|Version bumped in| D[hack/release.yaml]
C -->|Version bumped in| E[example/crs/...cr-full.yaml]
C -->|Version bumped in| F[example/crs/...cr-full-ocp.yaml]
D & E & F -->|Previously| G[network-operator-v26.7.0-beta.2]
D & E & F -->|Now| H[c89c59e]
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[CI Workflow Run\n28871934641] -->|Triggered by| B[ib-kubernetes branch\nnetwork-operator-26.1.x]
B -->|Produces image at| C[nvcr.io/nvstaging/mellanox\n/ib-kubernetes:c89c59e]
C -->|Version bumped in| D[hack/release.yaml]
C -->|Version bumped in| E[example/crs/...cr-full.yaml]
C -->|Version bumped in| F[example/crs/...cr-full-ocp.yaml]
D & E & F -->|Previously| G[network-operator-v26.7.0-beta.2]
D & E & F -->|Now| H[c89c59e]
Reviews (1): Last reviewed commit: "ci: update ib-kubernetes version to c89c..." | Re-trigger Greptile |
| repository: nvcr.io/nvstaging/mellanox | ||
| sourceRepository: ib-kubernetes | ||
| version: network-operator-v26.7.0-beta.2 | ||
| version: c89c59e |
There was a problem hiding this comment.
Raw commit SHA used as version identifier
The version is set to a bare short Git SHA (c89c59e) rather than a named tag (e.g., the previous network-operator-v26.7.0-beta.2). Short SHAs are ambiguous across repositories and can collide; they also make changelog auditing and rollback harder because there is no semantic context about what changed. If the CI pipeline produces a properly tagged image (or will tag one before this lands in a release), prefer using that tag here instead.
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!
Automated CI update for component 'ib-kubernetes', created by GitHub actions reusable workflow run 28871934641 for release branch network-operator-26.1.x.