Re: [Freeipa-devel] [PATCH 0473-0476]DNS Locations: Prologue

2016-05-25 Thread Jan Cholasta
On 25.5.2016 18:28, Martin Basti wrote: On 24.05.2016 12:10, Martin Basti wrote: On 23.05.2016 07:44, Jan Cholasta wrote: On 20.5.2016 15:32, Martin Basti wrote: On 20.05.2016 12:30, Petr Spacek wrote: On 18.5.2016 12:43, Martin Basti wrote: On 12.05.2016 16:16, Martin Basti wrote:

Re: [Freeipa-devel] [WIP] Thin client

2016-05-25 Thread Jan Cholasta
On 25.5.2016 18:17, Martin Basti wrote: On 25.05.2016 16:07, Jan Cholasta wrote: On 25.5.2016 15:03, David Kupka wrote: On 18/05/16 08:01, Jan Cholasta wrote: On 16.5.2016 16:35, Martin Basti wrote: On 09.05.2016 14:07, Jan Cholasta wrote: On 6.5.2016 14:32, Martin Basti wrote: On

Re: [Freeipa-devel] Provisioning throughput

2016-05-25 Thread thierry bordaz
On 05/25/2016 08:49 PM, Rob Crittenden wrote: thierry bordaz wrote: Hello, Thanks for all the feedbacks. I updated the design accordingly and with additional tests results (http://www.freeipa.org/page/V4/Performance_Improvements#Proposed_improvements) Several improvements can be done, in

Re: [Freeipa-devel] Provisioning throughput

2016-05-25 Thread Rob Crittenden
thierry bordaz wrote: On 05/25/2016 08:49 PM, Rob Crittenden wrote: thierry bordaz wrote: Hello, Thanks for all the feedbacks. I updated the design accordingly and with additional tests results (http://www.freeipa.org/page/V4/Performance_Improvements#Proposed_improvements) Several

Re: [Freeipa-devel] Provisioning throughput

2016-05-25 Thread Rob Crittenden
thierry bordaz wrote: Hello, Thanks for all the feedbacks. I updated the design accordingly and with additional tests results (http://www.freeipa.org/page/V4/Performance_Improvements#Proposed_improvements) Several improvements can be done, in particular in DS plugins (memberof, retroCL), but

Re: [Freeipa-devel] Provisioning throughput

2016-05-25 Thread thierry bordaz
Hello, Thanks for all the feedbacks. I updated the design accordingly and with additional tests results (http://www.freeipa.org/page/V4/Performance_Improvements#Proposed_improvements) Several improvements can be done, in particular in DS plugins (memberof, retroCL), but for "easy" benefit

Re: [Freeipa-devel] [WIP] Thin client

2016-05-25 Thread Martin Basti
On 25.05.2016 16:07, Jan Cholasta wrote: On 25.5.2016 15:03, David Kupka wrote: On 18/05/16 08:01, Jan Cholasta wrote: On 16.5.2016 16:35, Martin Basti wrote: On 09.05.2016 14:07, Jan Cholasta wrote: On 6.5.2016 14:32, Martin Basti wrote: On 28.04.2016 14:45, Jan Cholasta wrote: Hi,

Re: [Freeipa-devel] [PATCH 0484] remove unused code from automount plugin

2016-05-25 Thread Martin Basti
On 25.05.2016 09:11, Stanislav Laznicka wrote: LGTM, could you please just add the ticket to the commit message? On 05/20/2016 04:28 PM, Martin Basti wrote: On 20.05.2016 15:03, Martin Basti wrote: The removed code is unused for long time. Patch attached.

[Freeipa-devel] [Testplan Review] Server Roles

2016-05-25 Thread Oleg Fayans
Hi guys. Here is a rather schematic (as neither the feature not the design document is not complete) of the server roles testplan. Could you please review it and tell me what is missing? http://www.freeipa.org/page/V4/Server_Roles/Test_Plan -- Oleg Fayans Quality Engineer FreeIPA team RedHat.

Re: [Freeipa-devel] [WIP] Thin client

2016-05-25 Thread David Kupka
On 18/05/16 08:01, Jan Cholasta wrote: On 16.5.2016 16:35, Martin Basti wrote: On 09.05.2016 14:07, Jan Cholasta wrote: On 6.5.2016 14:32, Martin Basti wrote: On 28.04.2016 14:45, Jan Cholasta wrote: Hi, I have pushed my thin client WIP branch to GitHub:

Re: [Freeipa-devel] Questions on git

2016-05-25 Thread Jan Pazdziora
On Wed, May 25, 2016 at 12:17:18PM +0200, Christian Heimes wrote: > > I don't know why github either forces merge commits or a single squashed > commit. If I would have to guess, I would assume it has philosophical > reasons. It took many years to get github to add an alternative to merge The

Re: [Freeipa-devel] [PATCHES 0089-0093] Authentication Indicators

2016-05-25 Thread Sumit Bose
On Tue, May 24, 2016 at 12:21:43PM -0400, Nathaniel McCallum wrote: > New versions again. This time I just removed the stray "TODO: assign > OID" line in the commit as it no longer applies. ACK to patches 1-4. Patch 5 can be committed independently and needs some additional discussion, see

Re: [Freeipa-devel] [PATCH 0104-0109] DNS upgrade: change forwarding policy to "only" if private IPs are used

2016-05-25 Thread Martin Basti
On 20.05.2016 12:19, Petr Spacek wrote: On 11.5.2016 12:08, Martin Basti wrote: On 03.05.2016 14:59, Petr Spacek wrote: Hello, DNS upgrade: change forwarding policy to "only" if private IPs are used. https://fedorahosted.org/freeipa/ticket/5710 This is the upgrade part. I will add one

Re: [Freeipa-devel] [PATCH 0110] DNS: Warn if forwarding policy conflicts with automatic empty zone

2016-05-25 Thread Martin Basti
On 04.05.2016 10:43, Petr Spacek wrote: Hello, DNS: Warn if forwarding policy conflicts with automatic empty zones Forwarding policy "first" or "none" may conflicts with some automatic empty zones. Queries for zones specified by RFC 6303 will ignore forwarding and recursion and always result

Re: [Freeipa-devel] Questions on git

2016-05-25 Thread Martin Kosek
On 05/25/2016 11:55 AM, Christian Heimes wrote: > On 2016-05-25 11:46, Martin Kosek wrote: >> On 05/25/2016 10:03 AM, Jan Pazdziora wrote: >>> On Mon, May 23, 2016 at 04:24:38PM +0200, Florence Blanc-Renaud wrote: - I start working on a specific issue and decide to create a branch on my

Re: [Freeipa-devel] Questions on git

2016-05-25 Thread Christian Heimes
On 2016-05-25 11:46, Martin Kosek wrote: > On 05/25/2016 10:03 AM, Jan Pazdziora wrote: >> On Mon, May 23, 2016 at 04:24:38PM +0200, Florence Blanc-Renaud wrote: >>> >>> - I start working on a specific issue and decide to create a branch on my >>> git repository (on my laptop) >>> git clone

Re: [Freeipa-devel] Questions on git

2016-05-25 Thread Martin Kosek
On 05/25/2016 10:03 AM, Jan Pazdziora wrote: > On Mon, May 23, 2016 at 04:24:38PM +0200, Florence Blanc-Renaud wrote: >> >> - I start working on a specific issue and decide to create a branch on my >> git repository (on my laptop) >> git clone git://git.fedorahosted.org/git/freeipa.git >> git

Re: [Freeipa-devel] [PATCH 0477] upgrade: always start CA

2016-05-25 Thread Stanislav Laznicka
On 05/20/2016 03:00 PM, Martin Basti wrote: On 19.05.2016 13:34, Stanislav Laznicka wrote: Also, I tried to upgrade from 4.2.4 to 4.3.1 and it seems that it might be necessary to start the service even earlier in the upgrade logic. Attached is the trace that occurred during the upgrade. I

Re: [Freeipa-devel] [PATCH 0484] remove unused code from automount plugin

2016-05-25 Thread Stanislav Laznicka
LGTM, could you please just add the ticket to the commit message? On 05/20/2016 04:28 PM, Martin Basti wrote: On 20.05.2016 15:03, Martin Basti wrote: The removed code is unused for long time. Patch attached. https://fedorahosted.org/freeipa/attachment/ticket/5899/ -- Manage your