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

NAS-107309 / 12.1 / Add ZSTD and ZSTD-FAST to WebUI #4559

Merged
merged 1 commit into from
Aug 21, 2020

Conversation

PrivatePuffin
Copy link
Contributor

@PrivatePuffin PrivatePuffin commented Aug 20, 2020

Openzfs2.0 adds ZSTD and ZSTD-fast compression.
See: openzfs/zfs/pull/10278

This commit adds the following compression values to be accepted by the pool middleware:

  • ZSTD
  • ZSTD-5
  • ZSTD-7
  • ZSTD-FAST

These levels give an acceptable and balanced spread of both compression ratio and performance.
(see graphs in openzfs/zfs/pull/10278 )

Notes:

Signed-off-by: Kjeld Schouten-Lebbing kjeld@schouten-lebbing.nl

Openzfs2.0 adds ZSTD and ZSTD-fast compression.
See: openzfs/zfs/pull/10278

This commit adds the following compression values to be accepted by the pool middleware:
- ZSTD
- ZSTD-5
- ZSTD-7
- ZSTD-FAST

These levels give an acceptable and balanced spread of both compression ratio and performance.
(see graphs in openzfs/zfs/pull/10278 )

Notes:
- Requires an update to OpenZFS before being able to be used
- Might also want to backport to 12U1 or even 12RC1
- Related middleware changes are also required, see: truenas/middleware/pull/5517

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
@erincodepirate
Copy link
Contributor

note: this should not be merged into master and 12.0 until the related openzfs is mergd in

@PrivatePuffin
Copy link
Contributor Author

@Erinix It does, I put that scentence in the description in bold on both PR's to prevent any mistakes!

@PrivatePuffin
Copy link
Contributor Author

PrivatePuffin commented Aug 21, 2020

@freqlabs You might want to update ZoF (Guess you have write access) and the pkg...
(that way we can test it and make sure there aren't any unforseen issues)

@ghost
Copy link

ghost commented Aug 21, 2020

@Ornias1993 Ports PRs are open, pending manager approval just merged. The next nightly build will have ZSTD in OpenZFS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants