Re: [SSSD] [PATCH 0/1] RFC: Fix group memberships for subdomain users

2012-11-13 Thread Pavel Březina
On 11/12/2012 04:02 PM, Simo Sorce wrote: On Mon, 2012-11-12 at 13:30 +0100, Sumit Bose wrote: On Sat, Nov 10, 2012 at 10:05:36PM -0500, Simo Sorce wrote: This patch changes the way subdomain users are stored in the database. Thank you for the patch. I run couple of test and have not see an

Re: [SSSD] [PATCH 0/1] RFC: Fix group memberships for subdomain users

2012-11-12 Thread Sumit Bose
On Sat, Nov 10, 2012 at 10:05:36PM -0500, Simo Sorce wrote: This patch changes the way subdomain users are stored in the database. Thank you for the patch. I run couple of test and have not see an issue so far. But I have a couple of comments, see below. The reason for changing the way we

Re: [SSSD] [PATCH 0/1] RFC: Fix group memberships for subdomain users

2012-11-12 Thread Jakub Hrozek
On Mon, Nov 12, 2012 at 01:30:33PM +0100, Sumit Bose wrote: On Sat, Nov 10, 2012 at 10:05:36PM -0500, Simo Sorce wrote: This patch changes the way subdomain users are stored in the database. Thank you for the patch. I run couple of test and have not see an issue so far. But I have a

Re: [SSSD] [PATCH 0/1] RFC: Fix group memberships for subdomain users

2012-11-12 Thread Simo Sorce
On Mon, 2012-11-12 at 13:30 +0100, Sumit Bose wrote: On Sat, Nov 10, 2012 at 10:05:36PM -0500, Simo Sorce wrote: This patch changes the way subdomain users are stored in the database. Thank you for the patch. I run couple of test and have not see an issue so far. But I have a couple of

[SSSD] [PATCH 0/1] RFC: Fix group memberships for subdomain users

2012-11-10 Thread Simo Sorce
This patch changes the way subdomain users are stored in the database. The reason for changing the way we do it is that the sysdb code, before the subdomain patches were added assumed a single domain per cache file. This assumption beled in many other interfaces including the way users are read