Skip to content

store os release name and use it than recalculating from tag which is errorprone - #2244

Merged
rahulait merged 1 commit into
NVIDIA:mainfrom
rahulait:fix-issue-2239
Apr 6, 2026
Merged

rahulait merged 1 commit into
NVIDIA:mainfrom
rahulait:fix-issue-2239

Conversation

@rahulait

@rahulait rahulait commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor

Description

Fixes #2239

Instead of re-calculating os release from the tag (which is errorprone as there can be different combinations and all might not be covered by code), store the os-release when its fetched from node label and then re-use it where required.

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

@rahulait rahulait changed the title store os release name and use it than recalculating from tag which is… store os release name and use it than recalculating from tag which is errorprone Mar 24, 2026
@rahulait rahulait added this to the v26.3.1 milestone Mar 25, 2026
@rahulait rahulait self-assigned this Mar 31, 2026
Comment thread controllers/object_controls.go Resolved
@copy-pr-bot

copy-pr-bot Bot commented Apr 3, 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.

@rahulait

rahulait commented Apr 3, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 092582b

Comment thread controllers/object_controls_test.go Resolved
@tariq1890

Copy link
Copy Markdown
Contributor

Looks like the unit tests are failing

Comment thread controllers/object_controls.go Resolved
@rahulait

rahulait commented Apr 6, 2026

Copy link
Copy Markdown
Contributor Author

Looks like the unit tests are failing

Should be fixed now.

@rahulait

rahulait commented Apr 6, 2026

Copy link
Copy Markdown
Contributor Author

/cherry-pick release-26.3

@rahulait
rahulait merged commit 4587047 into NVIDIA:main Apr 6, 2026
19 checks passed
@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

🤖 Backport PR created for release-26.3: #2268 ⚠️ (has conflicts)

nicolerenee added a commit to frecklelabs/k8s that referenced this pull request Jul 1, 2026
The Talos version-parsing fix (NVIDIA/gpu-operator#2244, backported via
NVIDIA/gpu-operator#2268) is contained in the v26.3.3 release, so the
temporary ghcr.io/nvidia/gpu-operator:670be908 override is no longer
needed.

Co-Authored-By: Claude Fable 5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Latest relese does not work on Talos Linux because of incorrect version parsing

2 participants