Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Gabe Alford
I believe that Martin is right about the server installer no longer putting 127.0.0.1 in the resolv.conf. Here is a mod patch to address Martin's concern if the note needs to be changed to show a real IP address. Gabe On Thu, Mar 27, 2014 at 4:14 AM, Martin Basti wrote: > On Thu, 2014-03-27 at

[Freeipa-devel] Fwd: Re: FreeIPA 3.3.5

2014-03-27 Thread Matthew Harmsen
Re-sending since I am now subscribed to this list. Original Message Subject:Re: FreeIPA 3.3.5 Date: Thu, 27 Mar 2014 14:27:32 -0700 From: Matthew Harmsen To: Martin Kosek , freeipa-devel@redhat.com On 03/27/14 04:45, Martin Kosek wrote: Hello all! I think

Re: [Freeipa-devel] [PATCH 0015] Add wait_for_dns option to default.conf

2014-03-27 Thread Petr Spacek
On 27.3.2014 13:15, Martin Kosek wrote: On 02/20/2014 03:56 PM, Martin Basti wrote: On Thu, 2014-02-20 at 14:36 +0100, Petr Spacek wrote: On 19.2.2014 17:55, Martin Basti wrote: On Wed, 2014-02-19 at 17:10 +0100, Petr Spacek wrote: On 19.2.2014 15:11, Petr Spacek wrote: On 18.2.2014 17:34, N

Re: [Freeipa-devel] [PATCH]Extending user plugin with employeenumber field

2014-03-27 Thread Misnyovszki Adam
On Wed, 26 Mar 2014 14:07:50 +0100 Misnyovszki Adam wrote: > On Tue, 25 Mar 2014 18:26:53 +0100 > Misnyovszki Adam wrote: > > > On Tue, 25 Mar 2014 14:31:15 +0100 > > Petr Vobornik wrote: > > > > > On 21.3.2014 11:00, Misnyovszki Adam wrote: > > > > On Fri, 21 Mar 2014 10:13:55 +0100 > > > >

Re: [Freeipa-devel] [PATCH][RFC] 7 automember rebuild nowait feature added

2014-03-27 Thread Misnyovszki Adam
On Wed, 26 Mar 2014 13:15:55 +0100 Petr Viktorin wrote: > On 03/25/2014 03:36 PM, Misnyovszki Adam wrote: > > On Mon, 24 Mar 2014 17:06:41 +0100 > > Martin Kosek wrote: > > > >> On 03/24/2014 11:42 AM, Misnyovszki Adam wrote: > >>> On Fri, 21 Mar 2014 13:06:21 +0100 > >>> Petr Viktorin wrote: >

Re: [Freeipa-devel] [PATCH 0034] Deny LDAP binds for user accounts with expired principal

2014-03-27 Thread Simo Sorce
this need rebasing due to OTP patches, however comments inline. On Tue, 2014-01-07 at 13:47 +0100, Tomas Babej wrote: > diff --git a/daemons/ipa-slapi-plugins/ipa-pwd-extop/prepost.c > b/daemons/ipa-slapi-plugins/ipa-pwd-extop/prepost.c > index > ef37b5e173359f9404b241c12d8a6dc6e77da128..df74a671

Re: [Freeipa-devel] [PATCH] 562-563 webui: move RPC code from IPA module to its own module

2014-03-27 Thread Petr Vobornik
On 27.3.2014 14:38, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:29 +0100 Petr Vobornik wrote: depends on path #561(make navigation module independent on app module) [PATCH] 562 webui: move RPC code from IPA module to its own module - moves RPC code from ipa.js to it's own module - part

Re: [Freeipa-devel] [PATCH 0034] Deny LDAP binds for user accounts with expired principal

2014-03-27 Thread Martin Kosek
On 01/07/2014 01:47 PM, Tomas Babej wrote: > > On 01/07/2014 07:23 AM, Alexander Bokovoy wrote: >> On Mon, 06 Jan 2014, Tomas Babej wrote: >>> >>> On 01/06/2014 12:16 PM, Tomas Babej wrote: On 04/15/2013 12:43 PM, Tomas Babej wrote: > On 04/08/2013 03:55 PM, Martin Kosek wrote: >> On

Re: [Freeipa-devel] [PATCH] 562-563 webui: move RPC code from IPA module to its own module

2014-03-27 Thread Misnyovszki Adam
On Wed, 19 Mar 2014 16:02:29 +0100 Petr Vobornik wrote: > depends on path #561(make navigation module independent on app module) > > [PATCH] 562 webui: move RPC code from IPA module to its own module > > - moves RPC code from ipa.js to it's own module > - part of ongoing effort where the ultima

Re: [Freeipa-devel] [PATCH 0015] Add wait_for_dns option to default.conf

2014-03-27 Thread Martin Kosek
On 02/20/2014 03:56 PM, Martin Basti wrote: > On Thu, 2014-02-20 at 14:36 +0100, Petr Spacek wrote: >> On 19.2.2014 17:55, Martin Basti wrote: >>> On Wed, 2014-02-19 at 17:10 +0100, Petr Spacek wrote: On 19.2.2014 15:11, Petr Spacek wrote: > On 18.2.2014 17:34, Nathaniel McCallum wrote: >>

[Freeipa-devel] [PATCH 0161] ipa-range-check: Fix memory leaks when freeing range object

2014-03-27 Thread Tomas Babej
Hi, When cleaning the range_info struct, simple free of the struct is not enough, we have to free contents of char pointers in the struct as well. https://fedorahosted.org/freeipa/ticket/4276 -- Tomas Babej Associate Software Engineer | Red Hat | Identity Management RHCE | Brno Site | IRC: tbab

Re: [Freeipa-devel] [PATCH 0158] Extend ipa-range-check DS plugin to handle range types

2014-03-27 Thread Tomas Babej
The updated version handles the ret variable properly. It also makes sure the memory is properly freed. On 03/18/2014 04:45 PM, Alexander Bokovoy wrote: > On Tue, 18 Mar 2014, Tomas Babej wrote: >> >> On 03/18/2014 09:19 AM, Alexander Bokovoy wrote: >>> On Mon, 17 Mar 2014, Tomas Babej wrote:

Re: [Freeipa-devel] [PATCH] 561 webui: make navigation module independent on app module

2014-03-27 Thread Petr Vobornik
On 26.3.2014 15:36, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:19 +0100 Petr Vobornik wrote: When some module used 'freeipa/navigation' it pulled the entire Web UI because navigation depended on app. This patch splits the app into two modules: app and app_container. App specifies the e

[Freeipa-devel] FreeIPA 3.3.5

2014-03-27 Thread Martin Kosek
Hello all! I think we are close to releasing next bugfixing version on FreeIPA 3.3.5. We have a lot of bug fixes to offer, especially in the AD Trust area. I wrote the changelog to our wiki, fixes or suggestions welcome: http://www.freeipa.org/page/Releases/3.3.5 I have just one last concern tho

Re: [Freeipa-devel] [PATCH] [DOC] Review section on NetworkManager

2014-03-27 Thread Jan Pazdziora
On Wed, Mar 26, 2014 at 05:30:13PM -0600, Gabe Alford wrote: > All, > > Please review this patch for https://fedorahosted.org/freeipa/ticket/4156 > Added links to documentation on configuring NetworkManager. Thank you for the patch. ACK. -- Jan Pazdziora Principal Software Engineer, Identity M

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Martin Basti
On Thu, 2014-03-27 at 10:33 +0100, Petr Spacek wrote: > On 27.3.2014 10:23, Martin Basti wrote: > > On Wed, 2014-03-26 at 17:40 -0600, Gabe Alford wrote: > >> All, > >> > >> Please review patch for https://fedorahosted.org/freeipa/ticket/3085 > >> > >> Added note that 'nameserver 127.0.0.1' is adde

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Petr Spacek
On 27.3.2014 10:23, Martin Basti wrote: On Wed, 2014-03-26 at 17:40 -0600, Gabe Alford wrote: All, Please review patch for https://fedorahosted.org/freeipa/ticket/3085 Added note that 'nameserver 127.0.0.1' is added to resolv.conf, that it is recommended to add more replicas to resolv.conf, an

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Martin Basti
On Wed, 2014-03-26 at 17:40 -0600, Gabe Alford wrote: > All, > > Please review patch for https://fedorahosted.org/freeipa/ticket/3085 > > Added note that 'nameserver 127.0.0.1' is added to resolv.conf, that > it is recommended to add more replicas to resolv.conf, and the max > nameservers allowed

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Petr Viktorin
On 03/27/2014 09:42 AM, Petr Spacek wrote: On 27.3.2014 00:40, Gabe Alford wrote: All, Please review patch for https://fedorahosted.org/freeipa/ticket/3085 Added note that 'nameserver 127.0.0.1' is added to resolv.conf, that it is recommended to add more replicas to resolv.conf, and the max nam

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Petr Spacek
On 27.3.2014 00:40, Gabe Alford wrote: All, Please review patch for https://fedorahosted.org/freeipa/ticket/3085 Added note that 'nameserver 127.0.0.1' is added to resolv.conf, that it is recommended to add more replicas to resolv.conf, and the max nameservers allowed in resolv.conf. Thank you