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

feat(ci): add image scanning workflow #1161

Merged
merged 37 commits into from
Apr 21, 2024
Merged

Conversation

p5
Copy link
Member

@p5 p5 commented Apr 19, 2024

Creates a reusable workflow to scan images. This scan generates a SBOM and checks that SBOM for known vulnerabilities.

The only way I was able to merge the outputs of each matrix run was to use artifacts. It's certainly not clean, but it works.

We needed to exclude some directories during the SBOM generation because Syft crashes when scanning too large images.

This PR only enables the scans for Bluefin until we finalise it. Once we confirm everything is working, I will apply the same to Aurora and possibly Bazzite.

@p5 p5 changed the title [WIP] feature: add scaffolding for image scanning workflow [WIP] feature: add image scanning workflow Apr 19, 2024
@p5 p5 changed the title [WIP] feature: add image scanning workflow feature: add image scanning workflow Apr 20, 2024
@p5 p5 changed the title feature: add image scanning workflow feat: add image scanning workflow Apr 20, 2024
@p5 p5 marked this pull request as ready for review April 20, 2024 13:09
@p5 p5 requested a review from castrojo as a code owner April 20, 2024 13:09
Copy link
Member Author

@p5 p5 Apr 20, 2024

Choose a reason for hiding this comment

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

IMO we should centralise this configuration in ublue-os/main or somewhere before implementing scanning in other repos. There's nothing bluefin-specific in here, so no need to copy & paste it.

Copy link
Member

Choose a reason for hiding this comment

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

I think we can start here as a proof of concept.

We could then move it as it's own repo and have it be a callable action if we genericize it enough.

Copy link
Member Author

@p5 p5 Apr 21, 2024

Choose a reason for hiding this comment

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

Yeah, I plan to add things like uploading the SERIF files to GitHub, upload SBOMs to R2 etc. But that all comes later. Right now, I just want to build the files and push them to job artifacts in only this repo.

None of this will be specific to ublue and/or bluefin

@p5 p5 requested review from bsherman and noelmiller April 20, 2024 18:29
@p5 p5 changed the title feat: add image scanning workflow feat(ci): add image scanning workflow Apr 20, 2024
@m2Giles m2Giles added this pull request to the merge queue Apr 21, 2024
Merged via the queue into main with commit 244b269 Apr 21, 2024
73 checks passed
@m2Giles m2Giles deleted the sbom-generation-image-scanning branch April 21, 2024 20:10
awesomekyle pushed a commit to awesomekyle/bluefin that referenced this pull request Apr 24, 2024
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.

2 participants