Re: [Patch 0/9] NFS Mount Configuration File

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Steve Dickson wrote:
> Chuck Lever wrote:
>> On Mar 18, 2009, at Mar 18, 2009, 9:07 AM, Steve Dickson wrote:
>>>>>> This can become challenging to troubleshoot if there are these
>>>>>> semi-hidden options (cf. selinux).
>>>>> Why? 'mount -v' clearly shows what options are being used and then
>>>>> of course 'cat /proc/mounts' will show all the mount options.
>>>> Because we expect certain default settings for these mount options.  If
>>>> they can be different on every system and every mount point, then you
>>>> have to remember to go back and check /etc/fstab, and your configuration
>>>> file, and so on.
>>> Yes... If you do change something... the default settings may or may
>>> no longer
>>> be in effect... But I have faith that most admins would be able realize
>>> this... If they change something... things are going to change! :-)
>> The point is that these settings are changed so infrequently (or are
>> managed by a team of admins) that it will be too easy to overlook or
>> forget this kind of change.  You are asking administrators to look in
>> one more place when tracking down mount behavior, and I don't see a
>> clear benefit for the added complexity.
> No. I am not asking admins to do ANYTHING! It will be up to them if/when
> to used this configuration file... I'm just giving them an another way to 
> set NFS mounts options... They choose use so be it.. In not that ok too..
> 
>> It seems to me that automounter maps provide exactly this kind of
>> functionality.  Why are they inadequate for this task?  If they are
>> inadequate, why not improve automounter?
> Well, configuring the automounter is a very complex endeavour. Figuring out 
> how to set mount options globally, per server and per mount point is not an 
> easy task but with this configuration file it is.. Plus, to use the auto
> maps you have to use the automounter... There are no such requirement to use
> the configuration file. 
> 
>>>>>> I don't get why the per-mount section is even needed -- currently, the
>>>>>> mount options in /etc/fstab are used automatically if no options are
>>>>>> specified on the command line.
>>>>> Sure... this is option redundant with /etc/fstab but why not allow
>>>>> people
>>>>> configure all the NFS mounts in one spot? Why make them edit different
>>>>> files?
>>>> Indeed... why make them edit /etc/fstab and some other config file?
>>> But the /etc/fstab file can't set global or per server options...
>> Yes, however you can change /etc/fstab with a global editor command.
>>
>>>> We have /etc/fstab for static mounts and maps for automounter.  It's
>>>> already bad enough, and now we are considering another layer of
>>>> complexity.  Plus, the internal syntax of the mount command
>>>> configuration file is unlike either /etc/fstab or automounter maps.
>>> The format follows the already existing /etc/idmap.conf file. I
>>> basically reused the parsing routine that were already in nfs-utils.
>>> And I must admit, they were very well written...
>> Since we attempt to follow the example of other implementations
>> (especially the reference implementation, Solaris) why are we not
>> implementing a config file format that is used by other O/Ses?
> Suggestions are always welcomed... The fact that the parsing code
> already existed in nfs-utils I thought was a plus... 
> 
>> I've looked for a similar facility on Solaris 10, and I don't find it
>> mentioned in either mount(1M) or mount_nfs(1M).
> /etc/default/nfs has a similar look an feel... but it does not the
> same feature set.
> 
>>> While its true the format does not follow either the /etc/fstab file
>>> or the autofs maps, I contend its a format that is much more
>>> straightforward... Basically uncomment things... and again I have
>>> faith in the admins that they will be able to quickly figure it out...
>>> What is really need is a GUI... but that's for another day... ;-)
>> Whether it is straightforward or not, it is a new and additional format
>> that has to be understood.
> This format is not new... It used by a number other configuration files
> in /etc/.... and if there was one standard format for Linux configuration
> files I would adhere to it... But that is simply not the case... 
>  
>> It would make more sense if we were introducing this global config
>> feature for _all_ file systems (via the mount command, not by the
>> subcommands).  A GUI would follow the current trend of system-config-*
>> and many other system utilities, such as NetworkManager.
> I think this would be a bit too ambitious for what I'm trying to do...
> 
>>>>>>> This is the first step toward moving the default NFS version from 3
>>>>>>> to 4
>>>>>>> on the client.
>>>>>> I would think that the _only_ step is to implement the version
>>>>>> fallback
>>>>>> logic; ie. if the server doesn't support NFSv4, then use NFSv3, then
>>>>>> NFSv2.  Why can't an admin simply specify a fixed NFS version
>>>>>> (nfsvers=3) if there is a problem with NFSv4?
>>>>> They can... Nfsvers=3. When that is set, there will be no negotiation.
>>>> That's exactly my point.
>>>>
>>>> If an admin has a problem with using a particular version, they can
>>>> specify exactly what they want, today.
>>> Not globally... they have to change every single mount in both /etc/fstab
>>> and the autofs maps... verse making the change in one place.
>> If an admin cares enough to manage global changes, he will already use
>> automounter and avoid /etc/fstab.
> Again, this give them just another way of doing things...
> 
> 
>>>> People who don't care now probably are not likely to care if they get
>>>> NFSv4,
>>>> as long as NFSv4 is working as well as NFSv3.
>>> This is an assumption I don't think we should make.
>> Why not?  The point of a default setting is that it works well in most
>> cases.  If NFSv4 doesn't work well in most cases, then we have bigger
>> problems.  Otherwise, people who care about the minor differences
>> already have specific settings and configurations to deal with them.
> The key words being "most cases"... With this change the default settings
> can be manipulated to work in all cases... combining both groups you
> are alluding to.  
>  
>>>>>> It seems to me that if the admin hasn't specified nfsvers=, then she
>>>>>> does
>>>>>> not care which NFS version is used.
>>>>> True. And then the highest version the server supports
>>>>> will be negotiated. Having a Max/Min version will allow a the
>>>>> client to control that negotiation... Say the want v4 but not
>>>>> v4.1 ? Or may they only v41?
>>>> If they care that much, why not just say nfsvers=4 and be done with it?
>>> v4 has had years more testing than v4.1.
>> You are suggesting that one cannot choose specifically among NFSv4 minor
>> versions with a mount option.  I truly truly hope this is not the case.
> No. Of course there is a mount option to specify v4 and v4.1. But giving
> admins a way of added one line to one configuration that will either make
> all mounts v4 or v4.1. Something I would thing would be a bit handy... 
> 
>>>>>>> Having a configuration file which allows users to define
>>>>>>> the maximum and minimum NFS versions that should be negotiated is the
>>>>>>> right thing to do... IMHO.. Even though this patch does not does not
>>>>>>> do that, it does lay the ground work for that type of functionality
>>>>> Well I think most admins do want complete control over which NFS
>>>>> version will or will not be used... esp when it comes to v4 and v4.1
>>>> I honestly don't see that.  Admins want the system to work, and not be
>>>> bothered with the trivial details.  They want either "give me anything
>>>> that works" or "give me this specific setting."  And we have the ability
>>>> to do that already.
>>> Yes... But for this part "give me this specific setting" having a way to
>>> configure a setting, globally, per server, or per mount point is a
>>> good thing... imho...
>> No one has demonstrated yet that what we have today cannot provide the
>> same effect as a global setting.  An automounter map provides this
>> functionality.  What problem, exactly, are we trying to address here?
> Complexity... Not having to use the automounter and getting the similar 
> type of configurability is not a bad thing... imho... 
> 
>>>> Even without a config file, Linux is open source.  If an admin cares
>>>> that much about exactly how the system works, she will build it herself,
>>>> and possibly modify the source code.
>>> Please... how many admins do you know that are also kernel
>>> developers... :-)
>> You don't have to be a kernel developer, you just have to be able to
>> read C and run "make".  That's been the whole argument for open source
>> since the very beginning: it enables people to modify the code to make
>> it do exactly what they want.
> I thought open source argument was to write the patch once and the
> send it upstream so you only have to do it once... but this is 
> not really relevant to this discussion... the "open source argument"
> that is...  
> 
>> Any admin worth his or her NaCl never relies on default settings.
> Which is exactly the point of having a configuration file where
> they can easily be manipulate the defaults... 
> 
> 
>>>> I think the problem I have with this (besides the complexity) is that if
>>>> we must add more ways of avoiding NFSv4, then is NFSv4 really ready to
>>>> be made the default?
>>> Yes, I see your point... But I do think its not a bad thing to give the
>>> admins the final say on how and what they want to run... because they are
>>> the ones doing the front line support.. not us..
>>>
>>> I firmly believe if the transition to V4 is seamless and uneventful,
>>> having a configuration file will be non-issue... But if its not, then
>>> having a configuration file will be a good thing...
>> OK, then, what are we doing to make the transition seamless?  
> Please see the Dynamic Pseudo Root discussion on the NFSv4 mailing list.
> 
>> Do we know where the pain points are?  
> I've only identified the mounting pain points which I'm trying to
> address with this configuration file.
>  
>> Do we have them documented (something that for example Fedora release notes can refer to)?
> No, not yet.
>   
>> Are we giving our users real tools for managing the transition, like tools to convert ACLs?
> Yes, I'm trying giving them a configuration file that they use to define the defaults
> that work best in their environments. A tool to convert ACLS? Please see nfs4-acl-tools
> tar ball on the CITI web page... Remember, patches are always welcome! :-)
> 
>> Our defaults should be settings that work well out of the box in most environments.
>> If that isn't the case yet for NFSv4, then we shouldn't make it the default.
> Believe me.. with all the push back I'm getting from both the client and 
> server side... I being to think the Linux community simply does not want 
> move forward... but unfortunately I can accept that as option... which 
> the the reason I will continue to push for changes that will move us forward... 
This comment was uncalled for and I hope it does not imped on the productive
dialogue we were having...    

steved.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux