Skip to content

fix: RHEL update packages#256

Merged
rollandf merged 1 commit intoMellanox:mainfrom
rollandf:rhel-update
Feb 15, 2026
Merged

fix: RHEL update packages#256
rollandf merged 1 commit intoMellanox:mainfrom
rollandf:rhel-update

Conversation

@rollandf
Copy link
Member

No description provided.

Signed-off-by: Fred Rolland <frolland@nvidia.com>
@rollandf rollandf merged commit e03368b into Mellanox:main Feb 15, 2026
5 checks passed
@greptile-apps
Copy link

greptile-apps bot commented Feb 15, 2026

Greptile Summary

This PR adds dnf -y update to the RHEL_Dockerfile base stage, bringing it into consistency with the Ubuntu and SLES Dockerfiles which already perform distro updates. The change ensures RHEL-based images have up-to-date packages before installing driver dependencies.

  • Adds system package update in the base image build stage
  • Improves comment clarity by specifying "Perform distro update and install prerequirements"
  • Aligns RHEL build process with Ubuntu (apt-get upgrade) and SLES (zypper update) approaches

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The change is straightforward and mirrors existing patterns in Ubuntu_Dockerfile and SLES_Dockerfile. Adding package updates improves security posture by ensuring the base image has the latest patches. The change is minimal (2 lines) and follows established project conventions.
  • No files require special attention

Important Files Changed

Filename Overview
RHEL_Dockerfile Adds dnf -y update to align with Ubuntu and SLES Dockerfiles, ensuring packages are up-to-date in the base image

Last reviewed commit: 6c286b8

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

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