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

github: do not generate SBOM from source #23161

Merged
merged 1 commit into from
Jan 22, 2023
Merged

Conversation

aanm
Copy link
Member

@aanm aanm commented Jan 18, 2023

Generating SBOM from source takes a long time and due to a bug [1] it fills out the GH runner disk. Thus we will not be generating the SBOM from source until the bug is fixed.

[1] kubernetes-sigs/bom#202

Fixes: b11a065 ("build: Generate SBOM during image release")
Signed-off-by: André Martins andre@cilium.io

@aanm aanm added priority/high This is considered vital to an upcoming release. release-blocker/1.13 This issue will prevent the release of the next version of Cilium. needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Jan 18, 2023
@aanm aanm requested review from a team as code owners January 18, 2023 13:19
@aanm aanm requested a review from nbusseneau January 18, 2023 13:19
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jan 18, 2023
@aanm aanm added release-note/misc This PR makes changes that have no direct user impact. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jan 18, 2023
@aanm aanm force-pushed the pr/remove-dirs-generation-sbom branch from bec93da to 7ace532 Compare January 18, 2023 16:10
@joestringer
Copy link
Member

What does it use if not the source? Does it just generate it from the compiled binaries?

@ldelossa ldelossa force-pushed the pr/remove-dirs-generation-sbom branch from 7ace532 to b6deb5d Compare January 19, 2023 21:22
@aanm
Copy link
Member Author

aanm commented Jan 19, 2023

What does it use if not the source? Does it just generate it from the compiled binaries?

@joestringer In this case it will generate it from the installed packages in the image. I'm not sure if it will generate it for the cilium binaries.

Generating SBOM from source takes a long time and due to a bug [1] it
fills out the GH runner disk. Thus we will not be generating the SBOM
from source until the bug is fixed.

[1] kubernetes-sigs/bom#202

Fixes: b11a065 ("build: Generate SBOM during image release")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/remove-dirs-generation-sbom branch from b6deb5d to 0bc4180 Compare January 19, 2023 23:35
@nbusseneau
Copy link
Member

This is likely voided by #23204.

@aanm
Copy link
Member Author

aanm commented Jan 22, 2023

None of the test failures are caused by the modification of the workflow files. Merging...

@aanm aanm merged commit cee3e46 into master Jan 22, 2023
@aanm aanm deleted the pr/remove-dirs-generation-sbom branch January 22, 2023 23:19
@aanm aanm mentioned this pull request Jan 23, 2023
19 tasks
@aanm aanm added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Jan 23, 2023
@aanm aanm removed the backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. label Jan 23, 2023
@aanm aanm added the backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. label Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. priority/high This is considered vital to an upcoming release. release-blocker/1.13 This issue will prevent the release of the next version of Cilium. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants