Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-02-05 Thread Petr Vobornik
On 01/19/2015 01:20 PM, David Kupka wrote: Hi! This works for me. If all concerns regarding PermissionV2 and ACIs in general are resolved we can push. Both patches were pushed by mkosek on 2015-01-19: master: 6652c4eb2ebece71b6d60001246bd0fee5909099 Allow PassSync user to locate and update

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-19 Thread David Kupka
On 01/14/2015 10:27 PM, Martin Kosek wrote: Adding freeipa-devel back. On 01/14/2015 05:58 PM, Simo Sorce wrote: On Wed, 14 Jan 2015 17:47:51 +0100 Martin Kosek wrote: -add:aci:'(targetfilter="(objectclass=nsContainer)")(version 3.0; acl "Deny read access to replica configuration"; deny(read

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
Adding freeipa-devel back. On 01/14/2015 05:58 PM, Simo Sorce wrote: On Wed, 14 Jan 2015 17:47:51 +0100 Martin Kosek wrote: -add:aci:'(targetfilter="(objectclass=nsContainer)")(version 3.0; acl "Deny read access to replica configuration"; deny(read, search, compare) userdn = "ldap:///anyone";

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
On 01/14/2015 03:58 PM, Martin Kosek wrote: > On 01/14/2015 03:34 PM, Simo Sorce wrote: >> On Wed, 14 Jan 2015 13:41:54 +0100 >> thierry bordaz wrote: >> >>> On 01/14/2015 12:03 PM, Martin Kosek wrote: On 01/14/2015 10:58 AM, thierry bordaz wrote: > On 01/14/2015 10:15 AM, Petr Viktorin w

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
On 01/14/2015 03:42 PM, Alexander Bokovoy wrote: > On Wed, 14 Jan 2015, Simo Sorce wrote: >> On Wed, 14 Jan 2015 13:41:54 +0100 >> thierry bordaz wrote: >> >>> On 01/14/2015 12:03 PM, Martin Kosek wrote: >>> > On 01/14/2015 10:58 AM, thierry bordaz wrote: >>> >> On 01/14/2015 10:15 AM, Petr Viktor

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
On 01/14/2015 03:34 PM, Simo Sorce wrote: > On Wed, 14 Jan 2015 13:41:54 +0100 > thierry bordaz wrote: > >> On 01/14/2015 12:03 PM, Martin Kosek wrote: >>> On 01/14/2015 10:58 AM, thierry bordaz wrote: On 01/14/2015 10:15 AM, Petr Viktorin wrote: > On 01/13/2015 10:52 PM, Martin Kosek wr

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Alexander Bokovoy
On Wed, 14 Jan 2015, Simo Sorce wrote: On Wed, 14 Jan 2015 13:41:54 +0100 thierry bordaz wrote: On 01/14/2015 12:03 PM, Martin Kosek wrote: > On 01/14/2015 10:58 AM, thierry bordaz wrote: >> On 01/14/2015 10:15 AM, Petr Viktorin wrote: >>> On 01/13/2015 10:52 PM, Martin Kosek wrote: On 01

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Simo Sorce
On Wed, 14 Jan 2015 13:41:54 +0100 thierry bordaz wrote: > On 01/14/2015 12:03 PM, Martin Kosek wrote: > > On 01/14/2015 10:58 AM, thierry bordaz wrote: > >> On 01/14/2015 10:15 AM, Petr Viktorin wrote: > >>> On 01/13/2015 10:52 PM, Martin Kosek wrote: > On 01/13/2015 09:55 PM, Simo Sorce wr

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread thierry bordaz
On 01/14/2015 12:03 PM, Martin Kosek wrote: On 01/14/2015 10:58 AM, thierry bordaz wrote: On 01/14/2015 10:15 AM, Petr Viktorin wrote: On 01/13/2015 10:52 PM, Martin Kosek wrote: On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: This is crude

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
On 01/14/2015 10:58 AM, thierry bordaz wrote: > On 01/14/2015 10:15 AM, Petr Viktorin wrote: >> On 01/13/2015 10:52 PM, Martin Kosek wrote: >>> On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: > This is crude first version of the (

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Martin Kosek
On 01/14/2015 10:37 AM, thierry bordaz wrote: > On 01/14/2015 10:15 AM, Petr Viktorin wrote: >> On 01/13/2015 10:52 PM, Martin Kosek wrote: >>> On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: > This is crude first version of the (

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread thierry bordaz
On 01/14/2015 10:15 AM, Petr Viktorin wrote: On 01/13/2015 10:52 PM, Martin Kosek wrote: On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: This is crude first version of the (working) fixes to fix Winsync/Passsync problems caused by the Permissi

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread thierry bordaz
On 01/14/2015 10:15 AM, Petr Viktorin wrote: On 01/13/2015 10:52 PM, Martin Kosek wrote: On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: This is crude first version of the (working) fixes to fix Winsync/Passsync problems caused by the Permissi

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-14 Thread Petr Viktorin
On 01/13/2015 10:52 PM, Martin Kosek wrote: On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: This is crude first version of the (working) fixes to fix Winsync/Passsync problems caused by the PermissionV2 refactoring. Simo/Petr3 or others, any c

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-13 Thread Martin Kosek
On 01/13/2015 09:55 PM, Simo Sorce wrote: On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: This is crude first version of the (working) fixes to fix Winsync/Passsync problems caused by the PermissionV2 refactoring. Simo/Petr3 or others, any concerns? The first patch looks good the se

Re: [Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-13 Thread Simo Sorce
On Tue, 13 Jan 2015 18:16:11 +0100 Martin Kosek wrote: > This is crude first version of the (working) fixes to fix > Winsync/Passsync problems caused by the PermissionV2 refactoring. > > Simo/Petr3 or others, any concerns? > The first patch looks good the second looks .. broad ? Shouldn't you

[Freeipa-devel] [PATCH] 488-489 PermissionsV2 related winsync fixes

2015-01-13 Thread Martin Kosek
This is crude first version of the (working) fixes to fix Winsync/Passsync problems caused by the PermissionV2 refactoring. Simo/Petr3 or others, any concerns? -- Martin Kosek Supervisor, Software Engineering - Identity Management Team Red Hat Inc. From a22df9e17b22298c1409ec4f7830161364fedccd