[Freeipa-devel] [PATCHES 0175-0176] New forward zone test cases

[Tomas Babej]

On 01/13/2015 01:36 PM, Martin Basti wrote:
> On 13/01/15 10:04, Martin Kosek wrote:
>> On 01/12/2015 06:43 PM, Martin Basti wrote:
>>> On 05/01/15 16:18, Martin Basti wrote:
>>>> On 17/12/14 15:15, Martin Basti wrote:
>>>>> On 16/12/14 17:14, Martin Basti wrote:
>>>>>> On 15/12/14 20:15, Martin Basti wrote:
>>>>>>> On 15/12/14 19:18, Martin Basti wrote:
>>>>>>>> https://fedorahosted.org/freeipa/ticket/4750
>>>>>>>>
>>>>>>>> Patches need rebase and minor pytest modification to apply on
>>>>>>>> master, I
>>>>>>>> will do that after review.
>>>>>>>>
>>>>>>>> Patches for ipa-4-1 attached.
>>>>>>>>
>>>>>>>>
>>>>>>> I forgot to fix copy paste error.
>>>>>>>
>>>>>>> Updated patches attached
>>>>>>>
>>>>>> Removed unneeded  nsrecord validation in 2 tests
>>>>>>
>>>>>> updated patches attached
>>>>>>
>>>>> Added:
>>>>>
>>>>> * find forward zone with --forward-policy=none (no fwzone exists)
>>>>> * find forward zone with --forward-policy=only (no fwzone exists)
>>>>> * find forward zone with --forward-policy=first (no fwzone exists)
>>>>> * enable enabled zone
>>>>> * disable disabled zone
>>>>>
>>>>> Patches attached.
>>>>>
>>>>>
>>>> Hi Scott, could you ack or nack the patchset, other test tickets
>>>> are waiting,
>>>> and we want to avoid huge rebases/merging
>>>>
>>>  From QA point of view, tests are OK. Can somebody do code review
>>> please?
>> Code looks OK, runs cleanly (except that I thought that wait_for_dns
>> is boolean
>> value and not an integer :-/)
>>
>> So ACK from me. After you prepare a version for master branch, we can
>> push them.
>>
>> Martin
> Rebased patches attached
>
>
>
>
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

Pushed to master: 92feba2223575856820c2b95beab88a5229b4f6e

-- 
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | freeipa.org 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150113/285f4732/attachment.htm>