Skip to content
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

Enable SBOM on all OneBranch pipelines #1906

Merged
merged 1 commit into from
Jan 23, 2022

Conversation

mikeandescavage
Copy link
Contributor

Which issue this PR addresses:

Fixes: https://dev.azure.com/msazure/AzureRedHatOpenShift/_workitems/edit/12997379/
Related: https://dev.azure.com/msazure/AzureRedHatOpenShift/_workitems/edit/12932704

What this PR does / why we need it:

This is a requirement driven by Microsoft

Test plan for issue:

N/A

Is there any documentation that needs to be updated for this PR?

N/A

Copy link
Collaborator

@bennerv bennerv left a comment

Choose a reason for hiding this comment

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

LGTM - waiting on green CI to merge

@bennerv
Copy link
Collaborator

bennerv commented Jan 18, 2022

/azp run e2e

@bennerv
Copy link
Collaborator

bennerv commented Jan 18, 2022

/azp run ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@ross-bryan ross-bryan left a comment

Choose a reason for hiding this comment

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

lgtm,
@mikeandescavage - do you know if these sboms are shipped anywhere or just stay in the pipelines?

@mikeandescavage
Copy link
Contributor Author

@bryanro92 - the SBOM manifests generated are included in the published artifacts of the build. Based on their documentation, it does not appear that it gets shipped anywhere else.

@hawkowl hawkowl merged commit c2ca901 into Azure:master Jan 23, 2022
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.

4 participants