Bug 211022 - [Patch] Add get/set share properties subsection to handbook's ZFS chapter (19)
Summary: [Patch] Add get/set share properties subsection to handbook's ZFS chapter (19)
Status: Closed FIXED
Alias: None
Product: Documentation
Classification: Unclassified
Component: Books & Articles (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Sevan Janiyan
URL: https://reviews.freebsd.org/D12009
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-07-12 01:29 UTC by Dan Bright
Modified: 2017-08-18 21:04 UTC (History)
2 users (show)

See Also:


Attachments
Patch: FreeBSD handbook ZFS chapter diff text file. (2.39 KB, patch)
2016-07-12 01:29 UTC, Dan Bright
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Bright 2016-07-12 01:29:19 UTC
Created attachment 172397 [details]
Patch: FreeBSD handbook ZFS chapter diff text file.

Add a subsection to the handbook's ZFS chapter that briefly outlines the method by which NFS and SMB share properties may be enabled and configured on a ZFS dataset.
Comment 1 Allan Jude freebsd_committer freebsd_triage 2017-08-12 17:06:59 UTC
Few comments:

sharesmb doesn't work on freebsd.

for sharenfs, you can avoid a lot of problems, and the need for quoting, by replacing the spaces in the sharenfs string with commas

zfs set sharenfs=-alldirs,-network,192.168.0.0,-mask,255.255.255.0 pool/dataset
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-08-18 19:32:14 UTC
A commit references this bug:

Author: sevan
Date: Fri Aug 18 19:31:46 UTC 2017
New revision: 50688
URL: https://svnweb.freebsd.org/changeset/doc/50688

Log:
  Add get/set share properties subsection to handbook's ZFS chapter.

  PR:		211022
  Submitted by:	Dan Bright <freebsd AT zaziork DOT com>
  Reviewed by:	allanjude
  Approved by:	bcr (mentor)
  Differential Revision:	https://reviews.freebsd.org/D12009

Changes:
  head/en_US.ISO8859-1/books/handbook/zfs/chapter.xml
Comment 3 Sevan Janiyan freebsd_committer freebsd_triage 2017-08-18 19:33:38 UTC
Committed, thank you for the patch & apologies for the delay.
Comment 4 Dan Bright 2017-08-18 21:04:29 UTC
(In reply to Sevan Janiyan from comment #3)

You're very welcome! No problem at all about the delay; never seem to be enough hours in the day, right... or in this case, days in the year ;-) Glad it was still of some use.