fix: create sbom scanner without tmpfs in windows#6093
fix: create sbom scanner without tmpfs in windows#6093olljanat wants to merge 1 commit intomoby:masterfrom
Conversation
|
|
||
| runscan := llb.Image(scanner).Run(runOpts...) | ||
| runscan.AddMount("/tmp", llb.Scratch(), llb.Tmpfs()) | ||
| if runtime.GOOS != "windows" { |
There was a problem hiding this comment.
let's not use runtime, we created integration.UnixOrWindows() for this. I'd suggest something like this:
integration.UnixOrWindows(
runscan.AddMount("/tmp", llb.Scratch(), llb.Tmpfs()),
runscan.AddMount("/tmp", llb.Scratch()),
)
There was a problem hiding this comment.
Actually this is not good. /testutil/integration should only be used in the tests, not in this frontend code.
There was a problem hiding this comment.
Yes, I was thinking same but do you want it to be moved example to /util/… in this PR or refactor it later?
There was a problem hiding this comment.
oops, I thought this was a test. Must just been my foggy brain. If that's the case then your original code was correct, please revert and I approve.
Really sorry for going around...
There was a problem hiding this comment.
Revert and then just add Tonis' recommendation on using a cache mount. I think other than doing a TODO, let's do it within this PR...
|
If we can't use As follow-up (tracking issue) we should support |
Signed-off-by: Olli Janatuinen <olli.janatuinen@gmail.com>
Maybe but not sure how to use it in this context? |
|
Just a +1 here. We would really like proper support for Windows in this tool. Is there any way we can help to move that forward. ? |
4 participants
Windows does not support tmpfs so SBOM scanner create failed.
Related to docker/buildkit-syft-scanner#142