[SSSD] [PATCH] DNS site support 1st wave - generic SRV lookup plugin

Pavel Březina pbrezina at redhat.com
Tue Apr 2 13:55:51 UTC 2013


On 03/27/2013 03:00 PM, Jakub Hrozek wrote:
> On Wed, Mar 27, 2013 at 11:22:49AM +0100, Pavel Březina wrote:
>> Later, we can place the plugin architecture inside a module that
>> knows about SSSD and is placed above resolver. You said you want to
>> decouple fail over to fail over -> cache -> resolver. This is nice
>> idea.
>>
>> The plugin architecture can be put inside the cache, because cache
>> is responsible for resolving expired uris and _srv_ and it does not
>> really say that is has to use resolver directly.
>
> OK, this would be the best approach. After more discussion on #sssd
> with Pavel and Simo we agreed that Pavel will carry on with this
> approach for the time being.
>
> The only two changes in these patches would be better namespacing of
> the functions used and unifying the resolver initialization with what
> I'm working on wrt dynamic DNS updates.
>
> In the next release, we will create the additional layer and move
> the plugins there. I moved https://fedorahosted.org/sssd/ticket/1083
> which is tracking that effort to NEEDS_TRIAGE so we can put it into
> an appropriate milestone tomorrow.

Hi,
I'm sending new patch set.

I created async_resolv_utils.c and put there functions for domain
detection and for resolving SRV with fallback discovery domains. This is
similar to resolve_srv_send and resolve_get_domain_send from the first
patchset.

resolve_srv_send() was renamed to fo_discover_srv_send() and it wraps
resolv_discover_srv_send() so that it returns fo_server_info instead of
ares.

The plugin was renamed to fo_resolve_srv_dns_send(). It is a little bit
more complicated since it performs domain detection (originally in
resolve_srv_send()) if the domain is not provided from fail over.

Hostname is passed via plugin context. It is currently retrieved via
gethostname() so the current behaviour does not change. Later, when the
plugin initialization will be moved to provider code, the provider will
decide what hostname will be used (ipa_hostname for example).

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-resolv-add-resolv_get_domain-request-to-resolv-utils.patch
Type: text/x-patch
Size: 7182 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-resolv-add-resolv_discover_srv-request-to-resolv-uti.patch
Type: text/x-patch
Size: 7109 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-DNS-sites-support-SRV-lookup-plugin-interface.patch
Type: text/x-patch
Size: 4756 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-DNS-sites-support-SRV-DNS-lookup-plugin.patch
Type: text/x-patch
Size: 16576 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-DNS-sites-support-replace-SRV-lookup-code-with-a-plu.patch
Type: text/x-patch
Size: 16697 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-DNS-sites-support-use-SRV-DNS-lookup-plugin-for-all-.patch
Type: text/x-patch
Size: 3815 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130402/c3ed7cab/attachment-0005.bin>


More information about the sssd-devel mailing list