[Freeipa-devel] [PATCH 0285] DNS: consolidate RR types in API and LDAP schema

[Jan Cholasta]

Dne 20.7.2015 v 15:33 Martin Babinsky napsal(a):
> On 07/17/2015 01:00 PM, Martin Basti wrote:
>> On 15/07/15 18:13, Petr Spacek wrote:
>>> On 15.7.2015 17:39, Martin Basti wrote:
>>>> On 15/07/15 17:38, Petr Spacek wrote:
>>>>> On 15.7.2015 17:33, Martin Basti wrote:
>>>>>> On 15/07/15 16:03, Martin Basti wrote:
>>>>>>> On 15/07/15 15:39, Petr Vobornik wrote:
>>>>>>>> On 07/15/2015 02:42 PM, Martin Basti wrote:
>>>>>>>>> Patch attached.
>>>>>>>>>
>>>>>>>>>        https://fedorahosted.org/freeipa/ticket/4934
>>>>>>>>>        https://fedorahosted.org/freeipa/ticket/5055
>>>>>>>>>
>>>>>>>> Why are you adding RP, APL, IPSEC, DHCID, HIP, SPF records? Is
>>>>>>>> there a plan
>>>>>>>> to use them? Can't we just not use unsupported records in
>>>>>>>> LDAPSearch'?
>>>>>>>> (which would fix #5055)
>>>>>>> I had discussion with Petr2, and we decided to do it this way,
>>>>>>> because these
>>>>>>> records are valid.
>>>>>>>
>>>>>>> Removing unsupported records from search changes the behavior of
>>>>>>> the DNS
>>>>>>> commands. Now IPA shows even unsupported records which cannot be
>>>>>>> modified by
>>>>>>> API. AFAIK we want to keep this behavior.
>>>>>>>
>>>>>> Updated patch attached.
>>>>>>
>>>>>> I forgot to remove DNSKEY from object class definition.
>>>>> Are you 100 % sure that it will not break on upgrade? Please
>>>>> double-check that
>>>>> with Thierry.
>>>>>
>>>> IT was my change before we decided that DNSKEY should not be there,
>>>> it is not
>>>> in git repo.
>>> I see, okay.
>>>
>> Updated patch attached.
>>
>>
>>
> ACK

Pushed to:
master: 5ea41abe9836c94579115f9b220a8205b15d520d
ipa-4-2: bb6498547e1d638a064e4af259e195b7b95288b3

-- 
Jan Cholasta