Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[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

Merged
merged 10 commits into from
Mar 13, 2025

Conversation

haruna99
Copy link

Fixes Issue dotnet/arcade#15562

Description

This PR solves the SBOM generation issue in the runtime repo

@haruna99 haruna99 requested a review from hoyosjs March 13, 2025 06:39
@haruna99 haruna99 self-assigned this Mar 13, 2025
@Copilot Copilot AI review requested due to automatic review settings March 13, 2025 06:39
Copy link
Contributor

@Copilot Copilot AI left a 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

Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-meta
See info in area-owners.md if you want to be subscribed.

@haruna99 haruna99 added the Servicing-approved Approved for servicing release label Mar 13, 2025
@hoyosjs hoyosjs changed the title fix SBOM issues for runtime [release/9.0] fix SBOM issues for runtime Mar 13, 2025
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a 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

cc @carlossanlop

@carlossanlop carlossanlop added Servicing-approved Approved for servicing release and removed Servicing-approved Approved for servicing release labels Mar 13, 2025
@carlossanlop carlossanlop merged commit d9d0ae0 into dotnet:release/9.0 Mar 13, 2025
150 of 151 checks passed
@hoyosjs hoyosjs added Servicing-approved Approved for servicing release and removed Servicing-approved Approved for servicing release labels Mar 13, 2025
@hoyosjs
Copy link
Member

hoyosjs commented Mar 13, 2025

/backport to release/8.0

Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/13846556831

Copy link
Contributor

@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!

@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Meta Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants