Add RHEL (dnf) support to install-automation-components#64
Open
Add RHEL (dnf) support to install-automation-components#64
Conversation
The MI355X bare-metal runners use RHEL (with dnf), not Ubuntu or UBI minimal. The OS detection only handled apt-get and microdnf, causing all TEST and LM-EVAL jobs on MI355X to fail with "unknown OS". Add dnf-based detection and package installation for bare-metal RHEL environments, matching the package set installed on Ubuntu (curl, git, jq, wget, gcc, gcc-c++, lld, gh). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The MI355X bare-metal runner's GHA user doesn't have passwordless sudo. Instead of blindly running `sudo dnf install`, check if each required tool is already available and only install missing ones. On a properly provisioned runner, this skips the sudo call entirely. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
dnf, not Ubuntu (apt-get) or UBI minimal (microdnf)install-automation-componentsonly handled those two, causing all TEST and LM-EVAL jobs on MI355X to fail with"unknown OS"dnfdetection and full package installation for bare-metal RHEL (curl, git, jq, wget, gcc, gcc-c++, lld, gh)Context
This unblocks ROCm 7 MI355X validation (INFERENG-5254). The existing
rhel-updatesbranch had a partial fix (gh CLI only) but didn't install the full package set.Test plan
dnfavailablev1.25.0-betatag pointing to this commit🤖 Generated with Claude Code