-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add sbom scanning command #87
base: main
Are you sure you want to change the base?
Conversation
I know we talked about this project offline, but nobody outside you and I were part of that conversation and it's not stored anywhere, so the PR description for something like this is a good place to expound on the plan/purpose we discussed. 😅 It's also an important place to put into context which part of what we discussed this is, and why adding it here in the production repository now (without all the rest of what we talked about in place/ready) helps or unblocks you. 🙇 ❤️ For example:
Where "XXX" is information I don't currently have -- I have some technical/nit review comments I can make about this PR, but without that broader context telling me how this PR fits into that larger picture (and why it's being submitted by itself), I'm likely missing important information about why you've made the choices you did in the implementation you've proposed. 🙈 ❤️ |
5312672
to
2eadba3
Compare
2eadba3
to
e49263e
Compare
Sorry, I just noticed you updated the description (there's no GitHub notification for that, nor entry in the event log on the PR 😭). This is really good "what", but I'm still missing the "why" for this PR's changes specifically -- ie, if we merge this and commit to maintain it here, what does that unlock for you? This isn't the full solution to generating SBOMs in a detached way, and as we've discussed that's going to be a much larger project that needs to involve the full lifecycle management, so as-is this is going to be unused code for us for now (and will continue to be for the near-term future), so I'm missing what proposing this change here does beyond adding more red tape/review for your PoC/experimentation work? Can you please elaborate on that? For context, when I'm making larger interconnected changes like this and working on proving them out, I'll use a forked branch that adds my changes and rebase it periodically while I continue experimenting and integrating elsewhere, patching the other parts to point at my forked copy in the experimentation phases. In this specific case, that might look like the GitHub workflow you're working with doing Does that help explain what I'm looking for / missing better? 🙇 ❤️ |
Bringing the code in earlier allows us to parallelize the different parts of the process instead of doing it linearly and would be very similar to how feature flagging works. Additionally, this would allow proving that large portions of the code for signing and creating sbom etc work even if the final workflows are not in place. We can then create pipelines in a fork that use the production scripts generating SBOMs for production images but sending them to a seperate destination. Note also that I also am providing two possible approach we can take for this. |
This adds a command to generate SBOM independently of the build which has several advantages: