coreos / coreos-assembler

Tooling container to assemble CoreOS-like systems
https://coreos.github.io/coreos-assembler/
Apache License 2.0
335 stars 165 forks source link

cmd-compress: use level 10 for zstd compression #3721

Closed dustymabe closed 3 months ago

dustymabe commented 7 months ago

Based on the discussion in https://github.com/coreos/fedora-coreos-tracker/issues/1660 level 10 seems to give us a good speedup versus size tradeoff.

jlebon commented 4 months ago

Maybe let's flesh out the plan in https://github.com/coreos/fedora-coreos-tracker/issues/1660 first?

dustymabe commented 4 months ago

Maybe let's flesh out the plan in coreos/fedora-coreos-tracker#1660 first?

I think we could merge this without fleshing out the plan, since this doesn't really change the default to zstd, it just changes the default for zstd.

but ideally, yeah we'd figure out what we want to do in general. thoughts on how to make progress? should I retag the tracker issue with meeting? maybe we are still at this point: https://github.com/coreos/fedora-coreos-tracker/issues/1660#issuecomment-1933023388