Page MenuHomeFreeBSD
Differential D58175

Handbook - ZFS rework
ClosedPublic
Actions

Authored by carlavilla on Sat, Jul 11, 5:49 PM.
Tags
None
Referenced Files
F165288364: D58175.id182174.diff
Fri, Aug 7, 10:46 AM
F165288361: D58175.id182168.diff
Fri, Aug 7, 10:46 AM
F165288360: D58175.id181823.diff
Fri, Aug 7, 10:46 AM
F165288357: D58175.id181822.diff
Fri, Aug 7, 10:46 AM
F165288355: D58175.id181785.diff
Fri, Aug 7, 10:46 AM
F165288353: D58175.id.diff
Fri, Aug 7, 10:46 AM
F165288311: D58175.diff
Fri, Aug 7, 10:45 AM
F165285825: D58175.diff
Fri, Aug 7, 10:22 AM
View All 54 Files
Subscribers
delphij
vladlen

Details

Reviewers
bcr
michaelo
fernape
dbaio
allanjude
arrowd
Group Reviewers
Doc Committers
Commits
R9:653598764d3c: Handbook - ZFS: Rework
Summary
  • Add RAID-Z expansion, dRAID, special vdevs, hot spares with zfsd, TRIM, pool checkpoints, block cloning, fast dedup, bookmarks, holds, resumable/corrective send, project quotas, direct I/O, and boot environments
  • Add Jails and ZFS crossref target, encryption key loading at boot (zfskeys), and a Pool Properties section
  • Rewrite Tuning to drop removed sysctls (scrub_delay, vdev.cache.size, arc.meta_limit, …); delete the obsolete ZFS-on-i386 section
  • Switch examples to ada device names and GPT partitions
  • Fix the inverted reservation terminology entry, zfs scrub → zpool scrub, stale checksum/compression algorithm lists, and EOL version framing

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

carlavilla created this revision.Sat, Jul 11, 5:49 PM
Herald added a subscriber: delphij. · View Herald TranscriptSat, Jul 11, 5:49 PM
carlavilla requested review of this revision.Sat, Jul 11, 5:49 PM
carlavilla updated this revision to Diff 181822.Sun, Jul 12, 7:55 AM
bcr added a comment.Sun, Jul 12, 8:14 AM

Amazing to read through this rewrite. It is a much needed and substantial rewrite/update of most of the chapter. I only found this small nit.
Make sure to reference Bug 287148, Bug 261212, and maybe even Bug 263321. Thank you very much!

documentation/content/en/books/handbook/zfs/_index.adoc
970

Can you use ada0 ... for the device names in this example as well for consistency?

carlavilla updated this revision to Diff 181823.Sun, Jul 12, 9:03 AM
carlavilla marked an inline comment as done.
carlavilla added inline comments.
documentation/content/en/books/handbook/zfs/_index.adoc
970

Done, thanks!

bcr added a comment.Sun, Jul 12, 9:33 AM

Excellent, then let's have this in the handbook. Approved!

bcr accepted this revision.Sun, Jul 12, 9:34 AM

Macro shipit:

This revision is now accepted and ready to land.Sun, Jul 12, 9:34 AM
michaelo requested changes to this revision.Sun, Jul 12, 10:42 AM

One nit.

documentation/content/en/books/handbook/zfs/_index.adoc
106

What about NVMe? The names are different too.

This revision now requires changes to proceed.Sun, Jul 12, 10:42 AM
vladlen added a subscriber: vladlen.Mon, Jul 13, 5:12 PM
In D58175#1334411, @michaelo wrote:

One nit.

NVMe is a good point for a future improvement, but I don't think it's a blocker for this change.

documentation/content/en/books/handbook/zfs/_index.adoc
106

What about NVMe? The names are different too.

NVMe is a good point for a future improvement, but I don't think it's a blocker for this change.

carlavilla updated this revision to Diff 182168.Sat, Jul 18, 10:35 AM
carlavilla marked an inline comment as done.
carlavilla marked 2 inline comments as done.
carlavilla added inline comments.
documentation/content/en/books/handbook/zfs/_index.adoc
106

Fixed. Thanks for the review.

carlavilla added a comment.Sat, Jul 18, 11:13 AM

I'll make the commit this night or tomorrow

arrowd accepted this revision.Sat, Jul 18, 11:17 AM

Just finished reading this, great work! Thank you so much for working on this, @carlavilla !

michaelo accepted this revision.Sat, Jul 18, 12:51 PM
This revision is now accepted and ready to land.Sat, Jul 18, 12:51 PM
Closed by commit R9:653598764d3c: Handbook - ZFS: Rework (authored by carlavilla). · Explain WhySat, Jul 18, 1:26 PM
This revision was automatically updated to reflect the committed changes.
carlavilla added a commit: R9:653598764d3c: Handbook - ZFS: Rework.

Revision Contents
Changeset List

PathSize
documentation/
content/
en/
books/
handbook/
zfs/
2206 lines

Diff 182174

View Options

documentation/content/en/books/handbook/zfs/_index.adoc

Loading...