On 01.03.2016 14:50, Martin Babinsky wrote:
On 02/29/2016 05:37 PM, thierry bordaz wrote:
On 02/26/2016 05:48 PM, Martin Babinsky wrote:
On 02/26/2016 04:24 PM, thierry bordaz wrote:
On 02/25/2016 07:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016
On 02/29/2016 05:37 PM, thierry bordaz wrote:
On 02/26/2016 05:48 PM, Martin Babinsky wrote:
On 02/26/2016 04:24 PM, thierry bordaz wrote:
On 02/25/2016 07:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016
On 02/26/2016 05:48 PM, Martin Babinsky wrote:
On 02/26/2016 04:24 PM, thierry bordaz wrote:
On 02/25/2016 07:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On
On 02/26/2016 04:24 PM, thierry bordaz wrote:
On 02/25/2016 07:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi
On 02/25/2016 07:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
1.)
there is a number of PEP8 errors in the
On 02/25/2016 01:56 PM, Martin Babinsky wrote:
On 02/25/2016 12:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi
On 02/25/2016 12:17 PM, thierry bordaz wrote:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
On 02/25/2016 12:03 PM, Martin Babinsky wrote:
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
1.)
there is a number of PEP8 errors in the
On 02/24/2016 04:30 PM, thierry bordaz wrote:
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
1.)
there is a number of PEP8 errors in the patch
On 01/21/2016 05:04 PM, Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
1.)
there is a number of PEP8 errors in the patch
(http://paste.fedoraproject.org/313246/33893701), please fix them.
See
On 01/21/2016 03:46 PM, Martin Kosek wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Thanks! Couple comments:
I miss ticket number of description.
Thanks Martin for looking at it.
Ouch... the ticket number is https://fedorahosted.org/freeipa/ticket/4026
Does this patch mean that all
On 01/21/2016 04:22 PM, thierry bordaz wrote:
> On 01/21/2016 03:46 PM, Martin Kosek wrote:
>> On 01/21/2016 01:37 PM, thierry bordaz wrote:
>> Thanks! Couple comments:
>>
>> I miss ticket number of description.
>
> Thanks Martin for looking at it.
>
> Ouch... the ticket number is
On 01/21/2016 01:37 PM, thierry bordaz wrote:
>
Thanks! Couple comments:
I miss ticket number of description.
Does this patch mean that all blocker on DS side preventing remote DNA were
fixed? If yes, it may be worth updating Requires in the spec file in that case
and making sure the backport
On 01/21/2016 04:23 PM, Martin Kosek wrote:
On 01/21/2016 04:22 PM, thierry bordaz wrote:
On 01/21/2016 03:46 PM, Martin Kosek wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Thanks! Couple comments:
I miss ticket number of description.
Thanks Martin for looking at it.
Ouch... the
On 01/21/2016 05:38 PM, Martin Babinsky wrote:
On 01/21/2016 05:22 PM, Rob Crittenden wrote:
Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
6.)
+while attempt != MAX_WAIT:
+try:
+entries = conn.get_entries(sharedcfgdn,
On 01/21/2016 05:22 PM, Rob Crittenden wrote:
Martin Babinsky wrote:
On 01/21/2016 01:37 PM, thierry bordaz wrote:
6.)
+while attempt != MAX_WAIT:
+try:
+entries = conn.get_entries(sharedcfgdn,
scope=ldap.SCOPE_ONELEVEL, filter='dnaHostname=%s' %
Martin Babinsky wrote:
> On 01/21/2016 01:37 PM, thierry bordaz wrote:
> 6.)
>
> +while attempt != MAX_WAIT:
> +try:
> +entries = conn.get_entries(sharedcfgdn,
> scope=ldap.SCOPE_ONELEVEL, filter='dnaHostname=%s' % self.fqdn)
> +break
> +
On 01/21/2016 01:37 PM, thierry bordaz wrote:
Hi Thierry,
I have couple of comments to your patch:
1.)
there is a number of PEP8 errors in the patch
(http://paste.fedoraproject.org/313246/33893701), please fix them.
See http://www.freeipa.org/page/Python_Coding_Style for our conventions
19 matches
Mail list logo