Skip to content

add CVE updates for compiled and precompiled ubuntu images#629

Open
rahulait wants to merge 2 commits intoNVIDIA:mainfrom
rahulait:add-cve-updates
Open

add CVE updates for compiled and precompiled ubuntu images#629
rahulait wants to merge 2 commits intoNVIDIA:mainfrom
rahulait:add-cve-updates

Conversation

@rahulait
Copy link
Contributor

@rahulait rahulait commented Mar 2, 2026

Updates GitHub Actions image-build workflows to apply additional CVE-related package upgrades for Ubuntu-based images, aligning compiled and precompiled builds with a consistent set of security updates.

Changes:

Add libssl3 to Ubuntu CVE_UPDATES.
Introduce Ubuntu CVE_UPDATES export for precompiled image build jobs in both workflows.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates GitHub Actions image-build workflows to apply additional CVE-related package upgrades for Ubuntu-based images, aligning compiled and precompiled builds with a consistent set of security updates.

Changes:

  • Add libssl3 to Ubuntu CVE_UPDATES for the compiled image workflow.
  • Introduce Ubuntu CVE_UPDATES export for precompiled image build jobs in both workflows.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/image.yaml Expands Ubuntu CVE update package list and applies it to the pre-compiled job as well.
.github/workflows/precompiled.yaml Adds Ubuntu CVE update package list for the scheduled precompiled image workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

use composite action so that future package list changes only need to happen in one file

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

if [[ "${{ inputs.dist }}" =~ ^(rhel|rocky) ]]; then
echo "CVE_UPDATES=openssl python3-urllib3 libarchive libxml2 pam python3 sqlite-libs gnupg2" >> $GITHUB_ENV
elif [[ "${{ inputs.dist }}" =~ "ubuntu" ]]; then
echo "CVE_UPDATES=gnupg libssl3" >> $GITHUB_ENV
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll review and update this list once #636 is merged as I need to experiment and update it as required.

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