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

[Martin Basti]

On 21.04.2016 15:30, Martin Basti wrote:
>
>
> On 21.04.2016 15:25, 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)
>>
> PushError: Push rejected by bot (Reason: test should not be marked as 
> xfail)
>
... because issue was already fixed