Re: [PATCH] libnfsidmap: Query DNS for the the NFSv4 domain

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

 



Hey bud-

> On Aug 18, 2016, at 12:40 PM, Steve Dickson <SteveD@xxxxxxxxxx> wrote:
> 
> 
> 
> On 08/15/2016 11:16 AM, Chuck Lever wrote:
>>>>> On Aug 11, 2016, at 5:26 PM, Steve Dickson <SteveD@xxxxxxxxxx> wrote:
>>>>>>>> 
>>>>>>>> In domain_from_dns(), when at the hostname is a FQHN
>>>>>>>> query the DNS server for the _nfsv4idmapdomain TXT
>>>>>>>> record. If the record exists, use that as the
>>>>>>>> NFSv4 domain.
>>>>>>>> 
>>>>>>>> Note, this query will only happen if the domain name
>>>>>>>> is not set in the /etc/idmapd.conf
>>>>>> 
>>>>>> Is there a man page update that goes with this? The order in
>>>>>> which the library searches for the domain name should be
>>>>>> documented. idmapd.conf(5), maybe.
>>>> Yes... I should have made this an RFC patch since I just
>>>> wanted to get core out there for for comments... 
>>>> The man page needs to updated as well as the configure.ac.
>> To enable this feature IMO a better choice is to use a command
>> line option on nfsidmap and rpc.idmapd.
> I've be thinking about this... I don't think a command line 
> option is possible... How does a command line option tell a 
> internal library routine what to do? w/out creating new interface
> which is something I don't think we want to do...

I may have mentioned this elsewhere, but I changed my mind about this
already:

There's probably no need to have a command line option or a build-time
setting here, because setting the Domain = variable in /etc/idmapd.conf
is enough to disable the new query.


--
Chuck Lever



--
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