[Freeipa-devel] [patch 0035] ipatests: Add test case for requesting a certificate with full principal.

[Martin Babinsky]

On 04/21/2016 03:25 PM, Martin Babinsky wrote:
> On 04/21/2016 11:24 AM, Milan Kubík wrote:
>> On 04/05/2016 12:07 PM, Martin Babinsky wrote:
>>> On 04/05/2016 10:24 AM, Milan Kubík wrote:
>>>> On 04/05/2016 10:17 AM, Milan Kubík wrote:
>>>>> On 04/05/2016 09:31 AM, Martin Babinsky wrote:
>>>>>> On 04/01/2016 12:02 PM, Milan Kubík wrote:
>>>>>>
>>>>>>> Patches attached.
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> https://fedorahosted.org/freeipa/ticket/5733
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>> Hi Milan,
>>>>>>
>>>>>>
>>>>>>
>>>>>> I would be more happy if you could send a separate patch for the
>>>>>> context
>>>>>> manager fix, since the issue is orthogonal to the added test case
>>>>>> (even
>>>>>> if the test suite explodes without it).
>>>>>>
>>>>>>
>>>>>>
>>>>>> Otherwise LGTM.
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>> Done. Patch 0035 now applies to all branches, context manager fix
>>>>> needs separate patch for ipa-4-2.
>>>>>
>>>> Updated commit message in patches 0036 to include the ticket.
>>>>
>>> Thanks, ACK.
>>>
>> Add freeipa-devel back to the loop & push request :)
>>
>> --
>> Milan Kubik
>>
> Ah sorry I forgot how to mailing list.
>
> ACK again for our push-bot (aka mbasti)
>

I see that the fix for the failing test was already pushed so you can 
remove the xfail mark from the test and it should be all green now.

Sorry for the confusion.

-- 
Martin^3 Babinsky