[Freeipa-devel] [PATCH] 133 Fix creation of reverse records for IPA masters during install

[Martin Kosek]

On 05/03/2013 03:03 PM, Jan Cholasta wrote:
> Hi,
> 
> the attached patch fixes <https://fedorahosted.org/freeipa/ticket/3600>.
> 
> Honza
> 

This would fix the behavior, yes. But I think that it would be even better to
fix add_ptr_rr function to avoid unconditionally adding trailing dot to fqdn:

def add_ptr_rr(zone, ip_address, fqdn, dns_backup=None):
    name = get_reverse_record_name(zone, ip_address)
    add_rr(zone, name, "PTR", fqdn+".", dns_backup)

Otherwise we may get hit by this issue next time if we normalize fqdn before
passing it to add_ptr_rr. add_ns_rr already does this check:

def add_ns_rr(zone, hostname, dns_backup=None, force=True):
    if not hostname.endswith('.'):
        hostname += '.'


Martin