Re: [Freeipa-devel] [PATCHES] changes in preparation of replica promotion work

2015-08-04 Thread Jan Cholasta
Hi, Dne 31.7.2015 v 12:46 Simo Sorce napsal(a): I've been carrying these patches in my tree for a while, I think it is time to put them in master as they stand on their own. Simo. Patch 530: ACK Patch 531: ACK Patch 532: The methods should be static methods: @staticmethod def setO

Re: [Freeipa-devel] [PATCH] Port from python-kerberos library to python-gssapi

2015-08-04 Thread Jan Cholasta
Dne 4.8.2015 v 17:02 Robbie Harwood napsal(a): Michael Šimáček writes: Attaching new revision of the patch that performs the full negotiation cycle. Looks good to me, thanks! IPA compiles and installs fine with the patch applied, so ACK. Pushed to master: f0b4c4487ed77a3037cbbc46206d598c5

[Freeipa-devel] [PATCH] 0032 Fix otptoken-remove-managedby command summary

2015-08-04 Thread Fraser Tweedale
Small doc fix. Cheers, Fraser From 2879f147cacef06f0b3373ac21a78b9d08f8afbb Mon Sep 17 00:00:00 2001 From: Fraser Tweedale Date: Wed, 5 Aug 2015 15:50:07 +1000 Subject: [PATCH] Fix otptoken-remove-managedby command summary --- ipalib/plugins/otptoken.py | 2 +- 1 file changed, 1 insertion(+), 1

[Freeipa-devel] [PATCH] 0031 certprofile: add profile format explanation

2015-08-04 Thread Fraser Tweedale
The attached patch addresses https://fedorahosted.org/freeipa/ticket/5089 Thanks, Fraser From 6002c60a4794c0e6ecc315e21575ef618cff6f06 Mon Sep 17 00:00:00 2001 From: Fraser Tweedale Date: Thu, 23 Jul 2015 23:07:10 -0400 Subject: [PATCH] certprofile: add profile format explanation Part of: https:

Re: [Freeipa-devel] Update of User Certificates design page

2015-08-04 Thread Jan Pazdziora
On Tue, Aug 04, 2015 at 05:39:01PM +0200, Martin Babinsky wrote: > > I am working on the update of > http://www.freeipa.org/page/V4/User_Certificates design page. > > For now I have edited the "Feature Management" and "Howto Test" sections. > > Since this is my first time writing/editing designs

[Freeipa-devel] Update of User Certificates design page

2015-08-04 Thread Martin Babinsky
Hello list, I am working on the update of http://www.freeipa.org/page/V4/User_Certificates design page. For now I have edited the "Feature Management" and "Howto Test" sections. Since this is my first time writing/editing designs, there surely is a room for improvement. Feel free to send me

Re: [Freeipa-devel] [PATCH] 369 Added CLI param and ACL for vault service operations.

2015-08-04 Thread Endi Sukma Dewata
On 8/4/2015 8:51 AM, Martin Kosek wrote: Please also note that my next patch that adds the ability to change vault type, password, and keys will also require a client upgrade because the functionality is mainly implemented on the client side. In this case API URL versioning will be necessary. A

Re: [Freeipa-devel] [PATCH] Port from python-kerberos library to python-gssapi

2015-08-04 Thread Robbie Harwood
Michael Šimáček writes: > Attaching new revision of the patch that performs the full negotiation > cycle. Looks good to me, thanks! signature.asc Description: PGP signature -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Co

Re: [Freeipa-devel] 4.1 branch is broken

2015-08-04 Thread Rob Crittenden
Tomas Babej wrote: On 08/04/2015 03:44 PM, Rob Crittenden wrote: Commit 4d2ef43f broke the 4.1 branch. There is no ipaplatform/paths.py. rob I don't think so, that particular commit is from May 29 2014, which predates the release of FreeIPA 4.0. This file (along with couple of other platf

Re: [Freeipa-devel] [PATCH] 369 Added CLI param and ACL for vault service operations.

2015-08-04 Thread Martin Kosek
On 08/04/2015 02:20 PM, Simo Sorce wrote: > On Tue, 2015-08-04 at 08:05 +0200, Martin Kosek wrote: >> On 08/03/2015 10:37 PM, Endi Sukma Dewata wrote: >>> On 8/3/2015 2:47 PM, Martin Kosek wrote: On 08/03/2015 05:36 PM, Endi Sukma Dewata wrote: > On 8/3/2015 2:31 AM, Martin Kosek wrote: >>

Re: [Freeipa-devel] 4.1 branch is broken

2015-08-04 Thread Tomas Babej
On 08/04/2015 03:44 PM, Rob Crittenden wrote: > Commit 4d2ef43f broke the 4.1 branch. There is no ipaplatform/paths.py. > > rob > I don't think so, that particular commit is from May 29 2014, which predates the release of FreeIPA 4.0. This file (along with couple of other platform dependant f

[Freeipa-devel] 4.1 branch is broken

2015-08-04 Thread Rob Crittenden
Commit 4d2ef43f broke the 4.1 branch. There is no ipaplatform/paths.py. rob -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] Exporting users "access formulars"

2015-08-04 Thread Florian Crouzat
Hey, For security reason (mostly PCI-DSS) I have to print and sign-off access formular for every users, and also to maintain these formulars in time which means that every time I add a host to a hostgroup for example, I should reprint all access formulars for users with access to this hostgroup...

Re: [Freeipa-devel] [PATCH] 369 Added CLI param and ACL for vault service operations.

2015-08-04 Thread Simo Sorce
On Tue, 2015-08-04 at 08:05 +0200, Martin Kosek wrote: > On 08/03/2015 10:37 PM, Endi Sukma Dewata wrote: > > On 8/3/2015 2:47 PM, Martin Kosek wrote: > >> On 08/03/2015 05:36 PM, Endi Sukma Dewata wrote: > >>> On 8/3/2015 2:31 AM, Martin Kosek wrote: > On 07/31/2015 05:07 PM, Endi Sukma Dewat

Re: [Freeipa-devel] [PATCH 0052] store user certificates in 'userCertificate; binary' attributes

2015-08-04 Thread Jan Cholasta
Dne 4.8.2015 v 13:50 Martin Babinsky napsal(a): On 08/04/2015 10:27 AM, Martin Babinsky wrote: On 08/03/2015 06:41 PM, Martin Babinsky wrote: On 08/03/2015 03:39 PM, Jan Cholasta wrote: Dne 3.8.2015 v 14:58 Martin Babinsky napsal(a): On 08/03/2015 02:46 PM, Jan Cholasta wrote: Dne 3.8.2015 v

Re: [Freeipa-devel] [PATCH 0052] store user certificates in 'userCertificate; binary' attributes

2015-08-04 Thread Martin Babinsky
On 08/04/2015 10:27 AM, Martin Babinsky wrote: On 08/03/2015 06:41 PM, Martin Babinsky wrote: On 08/03/2015 03:39 PM, Jan Cholasta wrote: Dne 3.8.2015 v 14:58 Martin Babinsky napsal(a): On 08/03/2015 02:46 PM, Jan Cholasta wrote: Dne 3.8.2015 v 14:14 Jan Cholasta napsal(a): Hi, Dne 3.8.2015

Re: [Freeipa-devel] [PATCH 0002] TEST: Stageuser plugin

2015-08-04 Thread Lenka Doudova
Dne 30.7.2015 v 16:10 Martin Basti napsal(a): On 30/07/15 16:09, Martin Basti wrote: On 29/07/15 16:10, Martin Basti wrote: On 29/07/15 15:29, Lenka Doudova wrote: Hi, thanks a lot for the comments, will work on it tomorrow. Lenka Dne 29.7.2015 v 15:27 Martin Basti napsal(a): On 27/07/15

Re: [Freeipa-devel] [PATH 0053] Inconsistency between ipasearchrecordslimit and --sizelimit

2015-08-04 Thread Jan Cholasta
Dne 31.7.2015 v 17:08 Gabe Alford napsal(a): Updated patch attached. Thanks, Gabe On Thu, Jul 30, 2015 at 7:15 AM, Gabe Alford mailto:redhatri...@gmail.com>> wrote: On Thu, Jul 30, 2015 at 1:32 AM, Jan Cholasta mailto:jchol...@redhat.com>> wrote: Dne 30.7.2015 v 09:23 Jan Cholast

Re: [Freeipa-devel] C coding style guide update

2015-08-04 Thread Lukas Slebodnik
On (27/07/15 15:54), Michal Židek wrote: >On 07/26/2015 10:09 PM, Jakub Hrozek wrote: >> - Variable Length arrays are very helpful, but explicitly mention >> they should be used with caution, especially if array size might >> come from the user > >+1 >We overuse talloc for very smal

Re: [Freeipa-devel] [PATCH 0052] store user certificates in 'userCertificate; binary' attributes

2015-08-04 Thread Martin Babinsky
On 08/03/2015 06:41 PM, Martin Babinsky wrote: On 08/03/2015 03:39 PM, Jan Cholasta wrote: Dne 3.8.2015 v 14:58 Martin Babinsky napsal(a): On 08/03/2015 02:46 PM, Jan Cholasta wrote: Dne 3.8.2015 v 14:14 Jan Cholasta napsal(a): Hi, Dne 3.8.2015 v 14:00 Martin Babinsky napsal(a): This patch