Re: [Freeipa-devel] [PATCH] 067 A new flag to disable creation of UPG

2011-05-25 Thread Martin Kosek
On Tue, 2011-05-24 at 12:09 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Martin Kosek wrote: On Mon, 2011-05-23 at 17:32 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2011-05-20 at 10:58 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Martin Kosek wrote: On Mon,

Re: [Freeipa-devel] [PATCH] 19 Do stricter checking of IP addressed passed to server install

2011-05-25 Thread Martin Kosek
On Tue, 2011-05-24 at 15:42 +0200, Jan Cholasta wrote: On 24.5.2011 14:44, Jan Cholasta wrote: On 24.5.2011 14:43, Martin Kosek wrote: On Fri, 2011-05-20 at 20:34 +0200, Jan Cholasta wrote: On 18.5.2011 10:51, Martin Kosek wrote: On Mon, 2011-05-16 at 19:15 +0200, Jan Cholasta wrote: On

Re: [Freeipa-devel] [PATCH] 762 Let the framework be able to override the hostname

2011-05-25 Thread Martin Kosek
On Fri, 2011-04-01 at 11:47 -0400, Rob Crittenden wrote: The hostname is passed in during the server installation. We should use this hostname for the resulting server as well. It was being discarded and we always used the system hostname value. ticket 1052 rob I have to NACK this

Re: [Freeipa-devel] [PATCH] 768 fix migration between v2 servers

2011-05-25 Thread Martin Kosek
On Fri, 2011-04-08 at 10:24 -0400, Rob Crittenden wrote: Migration from a v2 server would fail because of our fake memberofindirect attribute. This isn't in any objectclass so would cause entries to fail to migrate. We can safely just remove it. Also remove any limits on time/size when

Re: [Freeipa-devel] [PATCH] 790 document problems re-adding a replication

2011-05-25 Thread Rob Crittenden
Dmitri Pal wrote: On 05/24/2011 04:21 PM, Rob Crittenden wrote: If you create a replica, remove it, then re-add it and try to re-initialize the database it will fail because the remote master has the old service principal cached. The remote dirsrv needs to be restarted. This is the issue in

Re: [Freeipa-devel] [PATCH] 0227-2-automount-UI

2011-05-25 Thread Adam Young
On 05/25/2011 12:28 AM, Endi Sukma Dewata wrote: On 5/24/2011 5:32 PM, Adam Young wrote: On 05/24/2011 05:06 PM, Endi Sukma Dewata wrote: On 5/24/2011 1:11 PM, Adam Young wrote: Known issues: the back to list links are broken. Add and delete for keys not working due to API issues discussed

Re: [Freeipa-devel] [PATCH] 0227-2-automount-UI

2011-05-25 Thread Adam Young
On 05/25/2011 12:28 AM, Endi Sukma Dewata wrote: 10. The code assumes that the primary keys will be specified as command arguments, which we know doesn't match the API for automount keys. Suppose we decide not to change the API, there should be a way to invoke the current API

Re: [Freeipa-devel] [PATCH] 790 document problems re-adding a replication

2011-05-25 Thread Simo Sorce
On Wed, 2011-05-25 at 09:09 -0400, Rob Crittenden wrote: Dmitri Pal wrote: On 05/24/2011 04:21 PM, Rob Crittenden wrote: If you create a replica, remove it, then re-add it and try to re-initialize the database it will fail because the remote master has the old service principal cached.

Re: [Freeipa-devel] [PATCH] 762 Let the framework be able to override the hostname

2011-05-25 Thread Rob Crittenden
Martin Kosek wrote: On Fri, 2011-04-01 at 11:47 -0400, Rob Crittenden wrote: The hostname is passed in during the server installation. We should use this hostname for the resulting server as well. It was being discarded and we always used the system hostname value. ticket 1052 rob I have to

Re: [Freeipa-devel] [PATCH] 790 document problems re-adding a replication

2011-05-25 Thread Rob Crittenden
Simo Sorce wrote: On Wed, 2011-05-25 at 09:09 -0400, Rob Crittenden wrote: Dmitri Pal wrote: On 05/24/2011 04:21 PM, Rob Crittenden wrote: If you create a replica, remove it, then re-add it and try to re-initialize the database it will fail because the remote master has the old service

Re: [Freeipa-devel] [PATCH] 790 document problems re-adding a replication

2011-05-25 Thread Simo Sorce
On Wed, 2011-05-25 at 12:39 -0400, Rob Crittenden wrote: Simo Sorce wrote: On Wed, 2011-05-25 at 09:09 -0400, Rob Crittenden wrote: Dmitri Pal wrote: On 05/24/2011 04:21 PM, Rob Crittenden wrote: If you create a replica, remove it, then re-add it and try to re-initialize the database

Re: [Freeipa-devel] [PATCH] 28 Move Managed Entries into their own container

2011-05-25 Thread JR Aquino
On May 24, 2011, at 10:48 AM, Rob Crittenden wrote: JR Aquino wrote: On May 24, 2011, at 8:17 AM, Rob Crittenden wrote: JR Aquino wrote: On May 23, 2011, at 2:42 PM, Rob Crittendenrcrit...@redhat.com wrote: JR Aquino wrote: On May 19, 2011, at 6:16 AM, Rob Crittenden wrote: JR

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread JR Aquino
On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18, 2011, at 2:52 PM, Rob Crittenden wrote: Rob Crittenden wrote: The Managed Entries plugin configurations weren't being created on replica installs. The templates were there but the cn=config portions were not.

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread Martin Kosek
On Wed, 2011-05-25 at 17:22 +, JR Aquino wrote: On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18, 2011, at 2:52 PM, Rob Crittenden wrote: Rob Crittenden wrote: The Managed Entries plugin configurations weren't being created on replica installs. The

Re: [Freeipa-devel] [PATCH] 0227-2-automount-UI

2011-05-25 Thread Endi Sukma Dewata
On 5/25/2011 11:31 AM, Adam Young wrote: 8. The column in the maps search facet doesn't have a title. Done. This actually was an indication of a deeper problem, that the columns were pointing to the wrong entity for param info. This is because the nested_search_facet is defined in the

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread Martin Kosek
On Wed, 2011-05-25 at 19:50 +0200, Martin Kosek wrote: On Wed, 2011-05-25 at 17:22 +, JR Aquino wrote: On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18, 2011, at 2:52 PM, Rob Crittenden wrote: Rob Crittenden wrote: The Managed Entries plugin

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread Dmitri Pal
On 05/25/2011 03:30 PM, Martin Kosek wrote: On Wed, 2011-05-25 at 19:50 +0200, Martin Kosek wrote: On Wed, 2011-05-25 at 17:22 +, JR Aquino wrote: On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18, 2011, at 2:52 PM, Rob Crittenden wrote: Rob Crittenden

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread Martin Kosek
On Wed, 2011-05-25 at 15:51 -0400, Dmitri Pal wrote: On 05/25/2011 03:30 PM, Martin Kosek wrote: On Wed, 2011-05-25 at 19:50 +0200, Martin Kosek wrote: On Wed, 2011-05-25 at 17:22 +, JR Aquino wrote: On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18,

Re: [Freeipa-devel] [PATCH] 786 Configure Managed Entries on replicas.

2011-05-25 Thread Rob Crittenden
Martin Kosek wrote: On Wed, 2011-05-25 at 19:50 +0200, Martin Kosek wrote: On Wed, 2011-05-25 at 17:22 +, JR Aquino wrote: On May 20, 2011, at 7:14 AM, Rob Crittenden wrote: JR Aquino wrote: On May 18, 2011, at 2:52 PM, Rob Crittenden wrote: Rob Crittenden wrote: The Managed Entries

Re: [Freeipa-devel] [PATCH] 0227-2-automount-UI

2011-05-25 Thread Endi Sukma Dewata
A few more minor problems: On 5/25/2011 2:54 PM, Adam Young wrote: 12. The automount map has a description field which can be specified during add modify. The UI doesn't have this. Fixed It's still missing from the map's adder dialog box. 17. The adder dialog boxes for map and key don't