Skip to content

[release/8.0] Fix SBoM generation#15603

Merged
hoyosjs merged 1 commit intodotnet:release/8.0from
garath:mistucke/net8-change-sbom
Mar 7, 2025
Merged

[release/8.0] Fix SBoM generation#15603
hoyosjs merged 1 commit intodotnet:release/8.0from
garath:mistucke/net8-change-sbom

Conversation

@garath
Copy link
Copy Markdown
Member

@garath garath commented Mar 6, 2025

Same as #15578 but for the release/8.0 branch.

1ES templates generates SBoM's for all artifacts uploaded - only issue is it does it in the root of the artifact that's uploaded inside the _manifest folder. If there's a manifest in the uploaded artifact, it will clobber the SBOM with an SBOM for the SBOM. The quick and dirty solution for now: generate the SBoM in a subdirectory of the upload root.

@garath garath changed the title Fix SBoM generation [release/8.0] Fix SBoM generation Mar 6, 2025
enableSbom: true
PackageVersion: 7.0.0
BuildDropPath: '$(Build.SourcesDirectory)/artifacts'
ManifestDirPath: $(Build.ArtifactStagingDirectory)/sbom
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: nothing is really using this

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original change was passign it down to the sbom step. This is not needed for correctness and it's fine to leave out of servicing changes

@hoyosjs hoyosjs merged commit 4b23ca9 into dotnet:release/8.0 Mar 7, 2025
11 checks passed
akoeplinger added a commit that referenced this pull request Mar 14, 2025
#15603 accidentally removed parts of the line, causing errors when running the script.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants