Add msr-tools package from upstream#17149
Conversation
|
@MadhurAggarwal -- I've merged your PR to azldev; can you try updating this PR? |
2f7207f to
cd55e9f
Compare
Thanks! |
There was a problem hiding this comment.
Pull request overview
Adds the Fedora-imported msr-tools component/spec to the Azure Linux repo so the rdmsr / wrmsr utilities are available as a packaged dependency (e.g., for LISA validation scenarios).
Changes:
- Added the
msr-toolscomponent to the Fedora-import list (components.toml) and introduced its lockfile. - Added rendered spec artifacts for
msr-tools(spec, sources hash, and the upstream patch used by the spec).
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| specs/m/msr-tools/sources | Adds the source tarball hash entry for the rendered component. |
| specs/m/msr-tools/msr-tools.spec | Introduces the rendered RPM spec for building/packaging msr-tools. |
| specs/m/msr-tools/bz1268604-increase-max-procs.patch | Adds the patch referenced by the spec to increase supported CPU index range. |
| locks/msr-tools.lock | Adds the azldev-managed lock pinning the imported upstream commit and input fingerprint. |
| base/comps/components.toml | Registers msr-tools as an unmodified Fedora-imported component (needs ordering fix per stored comment). |
e044c04 to
8915631
Compare
8915631 to
9c01a49
Compare
|
|
@christopherco Did you have any concerns around adding this new package? I'm supportive. |
No concerns on my side |
christopherco
left a comment
There was a problem hiding this comment.
Changes LGTM. Please squash your changes together into 1 commit, so we can rebase-merge.
9c01a49 to
087056f
Compare
|
I've squashed the commits into 1 and rebased it to the latest 4.0 |
chatted with @reubeno offline and this change looks good
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
rdmsr/wrmsrverify_hyperv_platform_idwhich skips due to this missing packageChange Log
Does this affect the toolchain?
NO
Associated issues
WorkItem: https://dev.azure.com/mariner-org/mariner/_workitems/edit/18998/
Test Methodology
Local RPM Build:

Koji build: https://purple-grass-0478bae1e.7.azurestaticapps.net/workflow/06fc9f45-c344-4fdd-c90b-08deb2c7ade9