Skip to content

Remove hardcoded package version from NVIDIA image#3671

Merged
dirkmueller merged 3 commits intomainfrom
package-version-nvidia
Apr 15, 2026
Merged

Remove hardcoded package version from NVIDIA image#3671
dirkmueller merged 3 commits intomainfrom
package-version-nvidia

Conversation

@alexandrevicenzi
Copy link
Copy Markdown
Member

@alexandrevicenzi alexandrevicenzi commented Apr 14, 2026

Grab the released package version from SCC, the only true source of truth for all codestreams.
Save all known versions to package_versions.json.

bci-pushman pushed a commit that referenced this pull request Apr 14, 2026
Test build for #3671
0645713
bci-pushman pushed a commit that referenced this pull request Apr 14, 2026
Test build for #3671
039641d
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 14, 2026 •
edited
Loading

Created a staging project on OBS for 16.0: home:pushman:BCI:Staging:16.0:16.0-3671
Changes pushed to branch 16.0-3671 as commit 0645713068676c02c46ddb3aa6c79206d70ca4b8
Build succeeded ✅

Build Results

Repository containerkiwi in home:pushman:BCI:Staging:16.0:16.0-3671 for x86_64: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.0:16.0-3671 for aarch64: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.0:16.0-3671 for s390x: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.0:16.0-3671 for ppc64le: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerfile in home:pushman:BCI:Staging:16.0:16.0-3671 for x86_64: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.0:16.0-3671 for aarch64: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.0:16.0-3671 for s390x: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.0:16.0-3671 for ppc64le: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Build succeeded ✅

To run BCI-tests against this PR, use the following command:

OS_VERSION=16.0 TARGET=custom BASEURL=registry.opensuse.org/home/pushman/bci/staging/16.0/16.0-3671/ tox -- -n auto
The following images can be pulled from the staging project:
  • registry.opensuse.org/home/pushman/bci/staging/16.0/16.0-3671/containerfile/bci/rust:1.93
⚠️⚠️ Project and Branch have been deleted ⚠️⚠️

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 14, 2026 •
edited
Loading

Created a staging project on OBS for 16.1: home:pushman:BCI:Staging:16.1:16.1-3671
Changes pushed to branch 16.1-3671 as commit 039641d11a3f75d56de2f804408470a43037198c
Build succeeded ✅

Build Results

Repository containerkiwi in home:pushman:BCI:Staging:16.1:16.1-3671 for x86_64: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.1:16.1-3671 for aarch64: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.1:16.1-3671 for s390x: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerkiwi in home:pushman:BCI:Staging:16.1:16.1-3671 for ppc64le: current state: published
Build results:

package name status build log
rust-oldstable-image ⛔ excluded live log
rust-stable-image ⛔ excluded live log

Repository containerfile in home:pushman:BCI:Staging:16.1:16.1-3671 for x86_64: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.1:16.1-3671 for aarch64: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.1:16.1-3671 for s390x: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Repository containerfile in home:pushman:BCI:Staging:16.1:16.1-3671 for ppc64le: current state: published
Build results:

package name status build log
rust-oldstable-image ✅ succeeded live log
rust-stable-image ✅ succeeded live log

Build succeeded ✅

To run BCI-tests against this PR, use the following command:

OS_VERSION=16.1 TARGET=custom BASEURL=registry.opensuse.org/home/pushman/bci/staging/16.1/16.1-3671/ tox -- -n auto
The following images can be pulled from the staging project:
  • registry.opensuse.org/home/pushman/bci/staging/16.1/16.1-3671/containerfile/bci/rust:1.93
⚠️⚠️ Project and Branch have been deleted ⚠️⚠️

bci-pushman pushed a commit that referenced this pull request Apr 14, 2026
Test build for #3671
7c5d83c
dirkmueller
dirkmueller reviewed Apr 14, 2026
View reviewed changes
Comment thread src/bci_build/package/versions.py Outdated
dirkmueller
dirkmueller reviewed Apr 14, 2026
View reviewed changes
Comment thread src/bci_build/package/package_versions.json Outdated
dirkmueller
dirkmueller reviewed Apr 14, 2026
View reviewed changes
Comment thread src/bci_build/package/package_versions.json Outdated
"7": {
"latest": "6.4.0-150700.53.34",
"versions": [
"6.4.0-150700.51",
Copy link
Copy Markdown
Member

@dirkmueller dirkmueller Apr 14, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

do we ever need to keep the release numbers for anything other than the same version? just checking. because then we could also automatically cleanup the list as soon as the version part of the ver-release is changing. and we only need to record the release part as well (but that would be optional)

Copy link
Copy Markdown
Member Author

@alexandrevicenzi alexandrevicenzi Apr 14, 2026

Choose a reason for hiding this comment

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

I prefer to keep the full version, it is easier to understand. Unfortunately for AMD and NVIDIA we need all kernel versions ever released, but in theory, this will be needed only for a few selected packages, right now only kernel and its flavors.

We could change to only save 6.12.0 once and 160000.27 and else multiple times, but this gets trickier to handle IMHO.

Copy link
Copy Markdown
Member

@dirkmueller dirkmueller Apr 14, 2026

Choose a reason for hiding this comment

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

I prefer to keep the full version, it is easier to understand. Unfortunately for AMD and NVIDIA we need all kernel versions ever released

ok, but only for the code stream I believe (e.g. 16.0/16.1 share the same kernel, but 16.2 won't)

Copy link
Copy Markdown
Member Author

@alexandrevicenzi alexandrevicenzi Apr 15, 2026

Choose a reason for hiding this comment

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

I'm not sure we need to change anything here, the json is cleaner now, and only two packages have the release history, not that bad.

@alexandrevicenzi alexandrevicenzi marked this pull request as ready for review April 14, 2026 15:30
@alexandrevicenzi
Allow version history in package_versions.json
b0f067b 
If enabled, also track the previous versions of a given package.
@alexandrevicenzi
Support release version in package_versions.json
ed648be 
Allow package_versions.json to also include the release version.

For this to work properly, we need to fetch the package information from
SCC instead of OBS.
@alexandrevicenzi
Move kernel package to package_versions.json
84eab6f 
Remove the hardcoded kernel versions from the NVIDIA image and migrate
it to package_versions.json.
dirkmueller
dirkmueller approved these changes Apr 15, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@dirkmueller dirkmueller left a comment

Choose a reason for hiding this comment

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

Thanks!

@dirkmueller dirkmueller merged commit 3cdea6c into main Apr 15, 2026
11 checks passed
@dirkmueller dirkmueller deleted the package-version-nvidia branch April 15, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dirkmueller dirkmueller dirkmueller approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexandrevicenzi @dirkmueller