-
Notifications
You must be signed in to change notification settings - Fork 5k
[release/9.0] fix SBOM issues for runtime #113463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR addresses SBOM generation issues in the runtime repository. The changes disable SBOM generation at the job template level while explicitly adding a step to generate SBOM artifacts.
- Disabled SBOM generation by introducing the "enableSbom: false" flag.
- Added an explicit template invocation for generating SBOM artifacts.
Comments suppressed due to low confidence (1)
eng/pipelines/official/jobs/prepare-signed-artifacts.yml:13
- The 'enableSbom: false' flag disables SBOM generation at the job level, yet an explicit step later invokes SBOM generation. Please ensure that this configuration is intentional or adjust the logic so that these settings do not conflict.
enableSbom: false
Tagging subscribers to this area: @dotnet/area-meta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved. we will take this as tell mode
/backport to release/8.0 |
Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/13846556831 |
@hoyosjs backporting to "release/8.0" failed, the patch most likely resulted in conflicts: $ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch
Applying: update runtime repo to produce SBOM after signing artifacts
Using index info to reconstruct a base tree...
M eng/pipelines/official/jobs/prepare-signed-artifacts.yml
Falling back to patching base and 3-way merge...
Auto-merging eng/pipelines/official/jobs/prepare-signed-artifacts.yml
CONFLICT (content): Merge conflict in eng/pipelines/official/jobs/prepare-signed-artifacts.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 update runtime repo to produce SBOM after signing artifacts
Error: The process '/usr/bin/git' failed with exit code 128 Please backport manually! |
Fixes Issue dotnet/arcade#15562
Description
This PR solves the SBOM generation issue in the runtime repo