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

Revert "build: Generate SBOM during image release" #23204

Merged
merged 1 commit into from Jan 20, 2023

Conversation

ldelossa
Copy link
Contributor

@ldelossa ldelossa commented Jan 19, 2023

This reverts commit b11a065.

SBOM generation in combination with updates to the GH runner broke
the CI for master. This reverts SBOM CI steps until we have a more
permanent fix.

Signed-off-by: Dylan Reimerink dylan.reimerink@isovalent.com

Revert "build: Generate SBOM during image release"

@ldelossa ldelossa requested review from a team as code owners January 19, 2023 23:00
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 19, 2023
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Jan 19, 2023
@ldelossa ldelossa added the kind/bug/CI This is a bug in the testing code. label Jan 19, 2023
@pchaigno pchaigno added the release-note/ci This PR makes changes to the CI. label Jan 19, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 19, 2023
@nbusseneau
Copy link
Member

Thanks. I think we should just revert b11a065, rather than make separate independent commits to address the issue. This way not only can we keep track of the revert w.r.t. original commit but we also remove the documentation bits, which helps protect us against potential users expecting an SBOM in release (if we don't re-enable SBOM by then).

This reverts commit b11a065.

SBOM generation in combination with updates to the GH runner broke
the CI for master. This reverts SBOM CI steps until we have a more
permanent fix.

Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
@nbusseneau nbusseneau added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jan 20, 2023
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Thanks!

Don't forget to update the PR title and description.

@nbusseneau nbusseneau changed the title CI: remove sbom for build image workflows Revert "build: Generate SBOM during image release" Jan 20, 2023
@nbusseneau
Copy link
Member

@cilium/docs-structure ACK is not required on this one since it's a revert, marked ready-to-merge.

@nebril nebril merged commit b7cd73e into master Jan 20, 2023
@nebril nebril deleted the ldelossa/disable-sbom branch January 20, 2023 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug/CI This is a bug in the testing code. kind/community-contribution This was a contribution made by a community member. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants