BuildNvidiaDriverContainers.sh: fix processing of cuda RPM version #12243
Conversation
This file contains 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
merge 3.0-dev into 3.0
Signed-off-by: Muhammad Falak R Wani <[email protected]> Co-authored-by: Paco Huelsz <[email protected]> Co-authored-by: Andrew Phelps <[email protected]> Co-authored-by: alejandro-microsoft <[email protected]> Co-authored-by: CBL-Mariner Servicing Account <[email protected]> Co-authored-by: SeanDougherty <[email protected]> Co-authored-by: Henry Beberman <[email protected]> Co-authored-by: Henry Li <[email protected]> Co-authored-by: Henry Li <[email protected]> Co-authored-by: Bala <[email protected]> Co-authored-by: Muhammad Falak R Wani <[email protected]> Co-authored-by: suresh-thelkar <[email protected]> Co-authored-by: Dan Streetman <[email protected]> Co-authored-by: Mandeep Plaha <[email protected]> Co-authored-by: jslobodzian <[email protected]> Co-authored-by: George Mileka <[email protected]> Co-authored-by: CBL-Mariner-Bot <[email protected]> Co-authored-by: lanzeliu <[email protected]> Co-authored-by: Karim Eldegwy <[email protected]> Co-authored-by: osamaesmailmsft <[email protected]> Co-authored-by: Tobias Brick <[email protected]> Co-authored-by: abadawi-msft <[email protected]> Co-authored-by: Mykhailo Bykhovtsev <[email protected]> Co-authored-by: sindhu-karri <[email protected]> Co-authored-by: Rohit Rawat <[email protected]> Co-authored-by: Dan Streetman <[email protected]> Co-authored-by: Nan Liu <[email protected]> Co-authored-by: Neha Agarwal <[email protected]> Co-authored-by: Adit Jha <[email protected]> Co-authored-by: Nan Liu <[email protected]> Co-authored-by: ypanch <[email protected]> Co-authored-by: Trung <[email protected]> Co-authored-by: Rakshaa Viswanathan <[email protected]> Co-authored-by: Rakshaa Viswanathan <[email protected]> Co-authored-by: Daniel McIlvaney <[email protected]> Co-authored-by: Chris Gunn <[email protected]> Co-authored-by: Cameron E Baird <[email protected]> Co-authored-by: Pawel Winogrodzki <[email protected]> Co-authored-by: Adub17030MS <[email protected]> Co-authored-by: Sam Meluch <[email protected]> Co-authored-by: Sam Meluch <[email protected]> Co-authored-by: himaja-kesari <[email protected]> Co-authored-by: Aurélien Bombo <[email protected]> Co-authored-by: Lanze Liu <[email protected]> Co-authored-by: ms-mahuber <[email protected]> Co-authored-by: Kanika Nema <[email protected]>
A recent rebranding change #8238 changed the repo name used by the containerized-rpmbuild container for mariner to azl-3.0-daily-build, but didn't change the default tdnf argument, which continued to use mariner-3.0-daily-build. This results in tdnf not working in the container. This fix updates the tdnf argument, basing it on the string in the repo file, so we don't have to copy/paste it later.
…ET option to suppress it (#8408) The containerized-rpmbuild scripts suppress most output from our tools/make commands, which means you have no idea what it's doing, whether it's hung, or sometimes even what caused an error. This change switches the default to print all this output to stdout, which a new option, QUIET to suppress it.
Co-authored-by: Henry Li <[email protected]>
…nd remove audit messages (#8459)
Co-authored-by: Sam Meluch <[email protected]> Co-authored-by: Sam Meluch <[email protected]>
…dev (#8544) Co-authored-by: Rachel Menge <[email protected]>
Co-authored-by: lanzeliu <[email protected]>
This reverts commit c36b428.
Update kernel-64k to include kexec_sig verification. Bump all other relevant kernel specs
This reverts commit dd652a9.
Co-authored-by: Binu Jose Philip <[email protected]>
Tested via buddy build.
Co-authored-by: Binu Jose Philip <[email protected]>
microsoft-github-policy-service
bot
added
the
3.0-dev
|
created from wrong branch; closing |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
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?
BuildNvidiaDriverContainers.shgenerates a stringCOMPONENT_VERSIONto pass togenerate-container-sbom.shwhen generating an SBOM.COMPONENT_VERSIONtypically looks like this550.54.15-3_6.6.57.1.7, but recently a new package "cuda-open" was added, causing this variable to contain both version numbers separated by a newline:This causes an incorrect parameter to be passed to the SBOM script.
Change Log
BuildNvidiaDriverContainers.shDoes this affect the toolchain?
NO
Associated issues
Test Methodology