Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-20 Thread Niranjan
Petr Viktorin wrote: > On 04/20/2016 06:11 AM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/18/2016 12:39 PM, Niranjan wrote: > >>> Niranjan wrote: > Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/06/2016 10:55 AM, Niranjan wrote: > >>> Greetings, > >>> > >>>

[Freeipa-devel] [PATCH] 0055 Authorise CA Agent to manage lightweight CAs

2016-04-20 Thread Fraser Tweedale
The attached patch is part of lightweight CA support. It just adds some ACL rules to Dogtag database, and does not depend the version of Dogtag (so it's ok to merge immediately, when ACKed). Thanks, Fraser From 362f7f9ec385cc2625d852ccf514508e231e78db Mon Sep 17 00:00:00 2001 From: Fraser

Re: [Freeipa-devel] [PATCH] 0053..0054 Configure lightweight CA key replication

2016-04-20 Thread Fraser Tweedale
On Thu, Apr 14, 2016 at 04:39:37PM +1000, Fraser Tweedale wrote: > Hi all, > > The attached patches configure lightweight CA key replication on IPA > CAs, on upgrade and installation. > > Patches 0051..0052 from my other mail are also needed for the system > to work, but this patchset does not

Re: [Freeipa-devel] [PATCH] 0013, 0017 webui: Add ability to convert users from preserved to staged state

2016-04-20 Thread Pavel Vomacka
On 04/15/2016 06:44 PM, Petr Vobornik wrote: On 04/13/2016 02:56 PM, Pavel Vomacka wrote: Hello, This patch adds ability to convert users from preserved to staged state. Fixes this ticket: https://fedorahosted.org/freeipa/ticket/5371 -- Pavel^3 Vomacka The patch requires rebase.

Re: [Freeipa-devel] [PATCH] 0050 caacl: correctly handle full user principal name

2016-04-20 Thread Martin Basti
On 14.03.2016 06:18, Alexander Bokovoy wrote: On Mon, 14 Mar 2016, Fraser Tweedale wrote: The attached patch fixes https://fedorahosted.org/freeipa/ticket/5733. Thanks to Alexander for finding and reporting. Cheers, Fraser From 9bd7b74d9c928f386bd7dae59588580881ed1a9d Mon Sep 17 00:00:00

Re: [Freeipa-devel] [TEST][patch-0033] Added assertion errors to topology tests, track N 5772

2016-04-20 Thread Martin Basti
On 12.04.2016 16:15, Martin Babinsky wrote: On 04/06/2016 02:40 PM, Oleg Fayans wrote: Hi Martin, The updated patches are attached On 04/04/2016 06:46 PM, Martin Babinsky wrote: On 03/31/2016 05:15 PM, Oleg Fayans wrote: Hi Martin, Thanks for the review. The updated patch(es) are

Re: [Freeipa-devel] [PATCH] 0015, 16 webui: Add 'Skip overlap check' checkbox to the dns adder dialogs

2016-04-20 Thread Pavel Vomacka
On 04/14/2016 02:03 PM, Martin Basti wrote: On 14.04.2016 13:03, Pavel Vomacka wrote: Hello, The first patch (0015) adds the checkbox to the dns zone adder dialog. The second patch (0016) adds the 'skip overlap check' checkbox to the dns forward zone adder dialog. This patch requires the

[Freeipa-devel] [PATCHES 0464-0468] always set hostname during installation

2016-04-20 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5794 It requires my patch 441.2 Patches attached. From e398581d1a681b8ef126c86951e35a5e0cb0e4d7 Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Tue, 19 Apr 2016 18:36:32 +0200 Subject: [PATCH 1/5] Always set hostname This prevents

Re: [Freeipa-devel] [PATCH] Added description related to 'status' in ipactl man page

2016-04-20 Thread Martin Basti
On 18.04.2016 10:55, Petr Spacek wrote: On 15.4.2016 13:43, Abhijeet Kasurde wrote: Hi All, Please review these patches. Fixes: https://fedorahosted.org/freeipa/ticket/5768 LGTM Works for me, ACK pushed to * master: 5b81435ae2422ae5c8afe26ec40eaf8b36d18359 Added description related to

Re: [Freeipa-devel] [PATCH 0441] Configure httpd service from installer

2016-04-20 Thread Martin Basti
On 24.03.2016 14:27, Martin Basti wrote: On 24.03.2016 13:55, Jan Cholasta wrote: On 18.3.2016 23:27, Timo Aaltonen wrote: On 17.03.2016 18:36, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5681 would be nicer if ipa-httpd.conf was a template with the current hardcoded

[Freeipa-devel] [PATCH 0463] Performance: do not download password attributes in host/find-user command

2016-04-20 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5281 Patch attached. From 96dcfec0399ccefbfcaebf2b7d19ea680aedeffa Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 8 Apr 2016 16:18:08 +0200 Subject: [PATCH] Performace: don't download password attributes in host/user-find For

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-20 Thread Petr Viktorin
On 04/20/2016 06:11 AM, Niranjan wrote: > Petr Viktorin wrote: >> On 04/18/2016 12:39 PM, Niranjan wrote: >>> Niranjan wrote: Niranjan wrote: > Petr Viktorin wrote: >> On 04/06/2016 10:55 AM, Niranjan wrote: >>> Greetings, >>> >>> For

Re: [Freeipa-devel] [PATCH] 0009 webui: Show certificates in useful format

2016-04-20 Thread Pavel Vomacka
On 04/15/2016 03:42 PM, Petr Vobornik wrote: On 03/30/2016 12:46 PM, Pavel Vomacka wrote: On 03/29/2016 11:42 AM, Pavel Vomacka wrote: On 03/16/2016 06:56 PM, Petr Vobornik wrote: On 03/15/2016 01:23 PM, Pavel Vomacka wrote: Hello, patch for https://fedorahosted.org/freeipa/ticket/5311

Re: [Freeipa-devel] [PATCH 0461] Limit max username lenght to 255 in config-mod

2016-04-20 Thread Martin Basti
On 20.04.2016 13:27, David Kupka wrote: On 15/04/16 15:42, Martin Basti wrote: On 15.04.2016 15:03, Petr Vobornik wrote: On 04/15/2016 01:25 PM, Martin Basti wrote: On 15.04.2016 13:18, David Kupka wrote: On 14/04/16 17:47, Martin Basti wrote:

Re: [Freeipa-devel] [PATCH 0461] Limit max username lenght to 255 in config-mod

2016-04-20 Thread David Kupka
On 15/04/16 15:42, Martin Basti wrote: On 15.04.2016 15:03, Petr Vobornik wrote: On 04/15/2016 01:25 PM, Martin Basti wrote: On 15.04.2016 13:18, David Kupka wrote: On 14/04/16 17:47, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5774 Patch attached. Hi, works for me,

Re: [Freeipa-devel] [PATCH] ca-less tests updated

2016-04-20 Thread Oleg Fayans
Hi David, On 04/20/2016 11:55 AM, David Kupka wrote: > On 20/04/16 10:35, David Kupka wrote: >> On 19/04/16 11:13, Oleg Fayans wrote: >>> OK, that one, though passing lint, did not actually work. I gave up my >>> attempts to define method decorators inside the class. Now it passes >>> lint AND

Re: [Freeipa-devel] [PATCH] ca-less tests updated

2016-04-20 Thread David Kupka
On 20/04/16 10:35, David Kupka wrote: On 19/04/16 11:13, Oleg Fayans wrote: OK, that one, though passing lint, did not actually work. I gave up my attempts to define method decorators inside the class. Now it passes lint AND works:) Hi Oleg! 1) Current commit message is useless. Please use

Re: [Freeipa-devel] [PATCH] ca-less tests updated

2016-04-20 Thread David Kupka
On 19/04/16 11:13, Oleg Fayans wrote: OK, that one, though passing lint, did not actually work. I gave up my attempts to define method decorators inside the class. Now it passes lint AND works:) Hi Oleg! 1) Current commit message is useless. Please use it to describe what is the point of

Re: [Freeipa-devel] [PATCH] 0051 Allow CustodiaClient to be used by arbitrary principals

2016-04-20 Thread Fraser Tweedale
On Mon, Apr 18, 2016 at 03:44:08PM -0400, Simo Sorce wrote: > On Thu, 2016-04-14 at 16:33 +1000, Fraser Tweedale wrote: > > On Wed, Apr 13, 2016 at 11:15:50AM +1000, Fraser Tweedale wrote: > > > On Tue, Apr 12, 2016 at 09:31:30AM -0400, Simo Sorce wrote: > > > > On Sat, 2016-04-09 at 10:11 +1000,