Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-07 Thread Alexander Bokovoy
On Tue, 06 Aug 2013, Ana Krivokapic wrote: On 08/06/2013 12:15 AM, Jakub Hrozek wrote: On Mon, Aug 05, 2013 at 09:55:26PM +0300, Alexander Bokovoy wrote: On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +return dict(result=False) + +attr =

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-07 Thread Martin Kosek
On 08/07/2013 08:48 AM, Alexander Bokovoy wrote: On Tue, 06 Aug 2013, Ana Krivokapic wrote: On 08/06/2013 12:15 AM, Jakub Hrozek wrote: On Mon, Aug 05, 2013 at 09:55:26PM +0300, Alexander Bokovoy wrote: On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-06 Thread Tomas Babej
On 08/05/2013 08:55 PM, Alexander Bokovoy wrote: On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +return dict(result=False) + +attr = groups_entry.get('schema-compat-lookup-sssd') same here. It needs my patch 0112 too -- it changes

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-06 Thread Ana Krivokapic
On 08/06/2013 12:15 AM, Jakub Hrozek wrote: On Mon, Aug 05, 2013 at 09:55:26PM +0300, Alexander Bokovoy wrote: On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +return dict(result=False) + +attr = groups_entry.get('schema-compat-lookup-sssd')

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-05 Thread Alexander Bokovoy
On Fri, 02 Aug 2013, Ana Krivokapic wrote: On 08/01/2013 04:13 PM, Alexander Bokovoy wrote: Hi! On Thu, 01 Aug 2013, Ana Krivokapic wrote: Hello, Thanks Alexander for the quick review! This patch adds ipa-advise plugins to help configure legacy clients for access to trusted domain

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-05 Thread Ana Krivokapic
On 08/05/2013 02:57 PM, Alexander Bokovoy wrote: On Fri, 02 Aug 2013, Ana Krivokapic wrote: On 08/01/2013 04:13 PM, Alexander Bokovoy wrote: Hi! On Thu, 01 Aug 2013, Ana Krivokapic wrote: Hello, Thanks Alexander for the quick review! This patch adds ipa-advise plugins to help configure

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-05 Thread Alexander Bokovoy
On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +return dict(result=False) + +attr = groups_entry.get('schema-compat-lookup-sssd') same here. It needs my patch 0112 too -- it changes ipa-adtrust-install to write proper configuration options to

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-05 Thread Jakub Hrozek
On Mon, Aug 05, 2013 at 09:55:26PM +0300, Alexander Bokovoy wrote: On Mon, 05 Aug 2013, Ana Krivokapic wrote: +except errors.NotFound: +return dict(result=False) + +attr = groups_entry.get('schema-compat-lookup-sssd') same here. It needs my patch 0112 too -- it

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-02 Thread Ana Krivokapic
On 08/01/2013 04:13 PM, Alexander Bokovoy wrote: Hi! On Thu, 01 Aug 2013, Ana Krivokapic wrote: Hello, Thanks Alexander for the quick review! This patch adds ipa-advise plugins to help configure legacy clients for access to trusted domain resources. For more details, please read the

[Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-01 Thread Ana Krivokapic
Hello, This patch adds ipa-advise plugins to help configure legacy clients for access to trusted domain resources. For more details, please read the commit message. Plugins are currently named config-redhat-sssd-before-1-9 and config-generic-sssd-before-1-9; suggestions for better names are

Re: [Freeipa-devel] [PATCH] 0052 Add ipa-advise plugins for legacy clients

2013-08-01 Thread Alexander Bokovoy
Hi! On Thu, 01 Aug 2013, Ana Krivokapic wrote: Hello, This patch adds ipa-advise plugins to help configure legacy clients for access to trusted domain resources. For more details, please read the commit message. Plugins are currently named config-redhat-sssd-before-1-9 and