Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Ludwig Krispenz
On 10/13/2014 08:19 AM, Martin Kosek wrote: On 10/10/2014 06:44 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 18:38:36 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/10/2014 06:30 PM, James wrote: On 10 October 2014 12:21, Simo Sorce s...@redhat.com wrote: First thing, I do not think

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Ludwig Krispenz
On 10/10/2014 06:21 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 17:52:15 +0200 Ludwig Krispenz lkris...@redhat.com wrote: Hello, this is the current status of my work on #4302, and there are a few pieces still missing, eg the management command needs more input checking and error handling, but

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Petr Viktorin
On 10/14/2014 10:21 AM, Ludwig Krispenz wrote: On 10/10/2014 06:21 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 17:52:15 +0200 Ludwig Krispenz lkris...@redhat.com wrote: Hello, this is the current status of my work on #4302, and there are a few pieces still missing, eg the management command

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Simo Sorce
On Tue, 14 Oct 2014 10:12:24 +0200 Ludwig Krispenz lkris...@redhat.com wrote: ok for me, I was just straightforward reading cn=config to get cn=config info, but I like the idea to do it via rootdse. we have to expose the suffix(es) controlled by the topology plugin and the entry point for the

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Simo Sorce
On Tue, 14 Oct 2014 10:21:57 +0200 Ludwig Krispenz lkris...@redhat.com wrote: we already have ipa-replica-manage and ipa-csreplica-manage, and - I did'n want to integrate the topology management into both and duplicate code - there is much change on the way to refactor the ipa commands, to

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Ludwig Krispenz
On 10/14/2014 02:39 PM, Simo Sorce wrote: On Tue, 14 Oct 2014 10:12:24 +0200 Ludwig Krispenz lkris...@redhat.com wrote: ok for me, I was just straightforward reading cn=config to get cn=config info, but I like the idea to do it via rootdse. we have to expose the suffix(es) controlled by the

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Rob Crittenden
Simo Sorce wrote: On Tue, 14 Oct 2014 10:21:57 +0200 Ludwig Krispenz lkris...@redhat.com wrote: we already have ipa-replica-manage and ipa-csreplica-manage, and - I did'n want to integrate the topology management into both and duplicate code - there is much change on the way to refactor

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Simo Sorce
On Tue, 14 Oct 2014 15:10:21 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/14/2014 02:39 PM, Simo Sorce wrote: On Tue, 14 Oct 2014 10:12:24 +0200 Ludwig Krispenz lkris...@redhat.com wrote: ok for me, I was just straightforward reading cn=config to get cn=config info, but I

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Simo Sorce
On Tue, 14 Oct 2014 11:46:47 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/10/2014 06:21 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 17:52:15 +0200 Ludwig Krispenz lkris...@redhat.com wrote: Hello, this is the current status of my work on #4302, and there are a few pieces

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Simo Sorce
On Tue, 14 Oct 2014 09:17:46 -0400 Rob Crittenden rcrit...@redhat.com wrote: Simo Sorce wrote: On Tue, 14 Oct 2014 10:21:57 +0200 Ludwig Krispenz lkris...@redhat.com wrote: we already have ipa-replica-manage and ipa-csreplica-manage, and - I did'n want to integrate the topology

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-14 Thread Ludwig Krispenz
On 10/14/2014 04:38 PM, Simo Sorce wrote: On Tue, 14 Oct 2014 11:46:47 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/10/2014 06:21 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 17:52:15 +0200 Ludwig Krispenz lkris...@redhat.com wrote: Hello, this is the current status of my work on

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-13 Thread Martin Kosek
On 10/10/2014 06:44 PM, Simo Sorce wrote: On Fri, 10 Oct 2014 18:38:36 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/10/2014 06:30 PM, James wrote: On 10 October 2014 12:21, Simo Sorce s...@redhat.com wrote: First thing, I do not think we want a new command here. If we need

[Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-10 Thread Ludwig Krispenz
Hello, this is the current status of my work on #4302, and there are a few pieces still missing, eg the management command needs more input checking and error handling, but - I wanted to give people interested a chance to have a look again and get feedback - there came up the following

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-10 Thread Simo Sorce
On Fri, 10 Oct 2014 17:52:15 +0200 Ludwig Krispenz lkris...@redhat.com wrote: Hello, this is the current status of my work on #4302, and there are a few pieces still missing, eg the management command needs more input checking and error handling, but - I wanted to give people interested

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-10 Thread James
On 10 October 2014 12:21, Simo Sorce s...@redhat.com wrote: First thing, I do not think we want a new command here. If we need commands outside of the ipa framework they should be integrated in the ipa-replica-manage tool. But really one of the reasons to move data in the shared tree was

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-10 Thread Ludwig Krispenz
On 10/10/2014 06:30 PM, James wrote: On 10 October 2014 12:21, Simo Sorce s...@redhat.com wrote: First thing, I do not think we want a new command here. If we need commands outside of the ipa framework they should be integrated in the ipa-replica-manage tool. But really one of the reasons to

Re: [Freeipa-devel] [PATCH] move replication topology to shared tree

2014-10-10 Thread Simo Sorce
On Fri, 10 Oct 2014 18:38:36 +0200 Ludwig Krispenz lkris...@redhat.com wrote: On 10/10/2014 06:30 PM, James wrote: On 10 October 2014 12:21, Simo Sorce s...@redhat.com wrote: First thing, I do not think we want a new command here. If we need commands outside of the ipa framework they