Skip to content

add 595TRD1 drivers#666

Open
rahulait wants to merge 1 commit intoNVIDIA:mainfrom
rahulait:add-595-drivers
Open

add 595TRD1 drivers#666
rahulait wants to merge 1 commit intoNVIDIA:mainfrom
rahulait:add-595-drivers

Conversation

@rahulait
Copy link
Copy Markdown
Contributor

@rahulait rahulait commented Mar 24, 2026

@rahulait rahulait force-pushed the add-595-drivers branch 2 times, most recently from 3fabe7f to a1ac4ee Compare March 25, 2026 03:28
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Mar 25, 2026

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
Copy link
Copy Markdown
Contributor Author

/ok to test a1ac4ee

@JunAr7112 JunAr7112 self-requested a review March 30, 2026 17:13
run: |
if [[ "${{ inputs.dist }}" =~ ^(rhel|rocky) ]]; then
echo "CVE_UPDATES=openssl python3-urllib3 libarchive libxml2 pam python3 sqlite-libs gnupg2" >> $GITHUB_ENV
echo "CVE_UPDATES=openssl python3-urllib3 libarchive libxml2 pam python3 platform-python python3-libs curl p11-kit lz4-libs vim-minimal sqlite-libs gnupg2 gnutls glib2 libblkid libfdisk libmount libsmartcols libuuid util-linux util-linux-core openldap" >> $GITHUB_ENV
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.

Are we able to simplify this to list just the top lvl packets? (ie util-linux glib2 gnupg2 gnutls openldap libxml2 libarchive sqlite pam vim-minimal p11-kit). I believe the dependent packets should automatically be pulled. Otherwise LGTM

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Currently, we do:

yum update -y ${CVE_UPDATES}

Dependencies are upgraded only if strictly required. Hence, I am adding individual list so that they are upgraded if any newer version is available.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I ended up bumping the base image used for rocky9 and also updated the base images early during the build process in #672 and #673. Now, I don't see any new package installed by CVE_UPDATES as everything is already up to date. Hence removed all the entries as they are no longer needed.

Logs from a run which had base images updated, so none of the listed package was updated.
rhel8 run: https://github.com/NVIDIA/gpu-driver-container/actions/runs/23829447442/job/69459543075?pr=666#step:9:2471
rhel9 run: https://github.com/NVIDIA/gpu-driver-container/actions/runs/23829447442/job/69459543060?pr=666#step:9:3079
rhel10 run: https://github.com/NVIDIA/gpu-driver-container/actions/runs/23829447442/job/69459543065?pr=666#step:9:4132
rocky9 run: https://github.com/NVIDIA/gpu-driver-container/actions/runs/23829447442/job/69459543071?pr=666#step:9:3022

@rahulait
Copy link
Copy Markdown
Contributor Author

/ok to test 9020177

@rahulait rahulait self-assigned this Mar 31, 2026
@rahulait
Copy link
Copy Markdown
Contributor Author

/ok to test 2f39f55

@rahulait
Copy link
Copy Markdown
Contributor Author

rahulait commented Apr 1, 2026

/ok to test f3b9236

@rahulait rahulait closed this Apr 1, 2026
@rahulait rahulait reopened this Apr 1, 2026
@rahulait
Copy link
Copy Markdown
Contributor Author

rahulait commented Apr 1, 2026

Re-opened as I had accidentally fat-fingered and closed the PR.

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
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