Hi Alan,

>
> All shares of a dataset should report the same label and drive letter,
> and I should be able to set sharesmb=off and still have other shares
> report that drive letter and label.

I see what you mean.

> For example:
>
> ? ?zfs create -o casesensitivity=mixed tank/photos
> ? ?zfs sharesmb=off
>
> ? ?sharemgr create -P smb photogroup
> ? ?sharemgr add-share -r public -s tank/photos/public photogroup
> ? ?sharemgr add-share -r private -s tank/photos/private photogroup
>
> I now have two shares (public and private) and sharesmb=off.
>
> If I were to set a label using sharesmb that would cause zfs to share
> tank/photos, which I may not want shared.

I do not understand that. Why wouldn't you just set label with sharemgr than ?

> The current syntax doesn't allow sharesmb=off,label=photos, and if we
> change the syntax of sharesmb it would be inconsistent with sharenfs and
> shareiscsi.

To the contrary - sharenfs does allow such syntax, and it is widely used:

pronto:/home/imp> zfs get sharenfs tank/export/install
NAME                 PROPERTY  VALUE                SOURCE
tank/export/install  sharenfs  ro,anon=0            local

> Suppose I did want to share tank/photos and set some options:
>
> ? ?zfs sharesmb=name=photos,csc=auto,oplocks=disabled,label=z:photos,
>
> Here you would be applying name and csc to the tank/photos share but not
> to the public and private shares, but the oplocks and label values would
> apply to the public and private shares. ?It may cause confusion to mix
> share specific options with dataset wide options in this format.

I do not see how can it causes any confusion. To my understanding it
does exactly what you want, but may be I missing something here.

Anyway, why don't we just always use "final component of the
mountpoint property" as you've suggested and need a special attribute
to override the value ? After all windows users do not have an option
to report volume label other than it is set on the disk.

-- 
Regards,
        Cyril

Reply via email to