Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-13 Thread Anthony Messina
On Wednesday, May 13, 2015 01:28:44 PM Martin Babinsky wrote: On 05/12/2015 06:47 PM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Petr Vobornik wrote: On 05/12/2015 11:22 AM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Martin Babinsky wrote: %attr(644,root,root)

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Martin Basti
On 13/05/15 10:06, Jan Cholasta wrote: Hi, Dne 5.5.2015 v 10:38 Martin Basti napsal(a): On 05/05/15 08:29, Fraser Tweedale wrote: On Mon, May 04, 2015 at 06:35:45PM +0200, Martin Basti wrote: On 04/05/15 15:36, Fraser Tweedale wrote: Hello, Please review the first cut of the 'certprofile'

Re: [Freeipa-devel] [PATCH] Password vault

2015-05-13 Thread Jan Cholasta
Dne 12.5.2015 v 12:52 Endi Sukma Dewata napsal(a): Please take a look at the attached patch (#353-9). It obsoletes all previous patches. See comments below. On 4/20/2015 1:12 AM, Jan Cholasta wrote: I'm planning to merge the vault and vault container object and use the vault type attribute to

Re: [Freeipa-devel] [PATCH] 0048 Remove unused enable() method from DogtagInstance.

2015-05-13 Thread Martin Basti
On 12/05/15 16:54, David Kupka wrote: On 05/12/2015 02:16 PM, Martin Basti wrote: On 12/05/15 13:53, David Kupka wrote: DogtagInstance.enable() overrides Service.enable() and does nothing usefulll. Also removing it solves bug discovered recently in uninstall procedure. NACK 1) this is used

Re: [Freeipa-devel] [PATCH 0245] Fix uniqueness plugins vol. 2

2015-05-13 Thread Martin Babinsky
On 05/12/2015 09:03 PM, Martin Basti wrote: On 12/05/15 18:23, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/4921 To test this, the mkosek/freeipa-master copr repo with 389-ds-base 1.3.4.0 is needed. All previous changes to uniqueness plugins were made just in master branch so

Re: [Freeipa-devel] [PATCH] 829 webui: option to not create user private group

2015-05-13 Thread Martin Basti
On 12/05/15 17:58, Petr Vobornik wrote: Web UI was not able to create a user without a private group. New field added to user adder dialog to allow that. https://fedorahosted.org/freeipa/ticket/4986 ACK -- Martin Basti -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-13 Thread Martin Babinsky
On 05/12/2015 06:47 PM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Petr Vobornik wrote: On 05/12/2015 11:22 AM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Martin Babinsky wrote: %attr(644,root,root) %{_unitdir}/ipa-ods-exporter.service +%attr(644,root,root)

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Fraser Tweedale
Hi Jan, thanks for review. Comments inline. On Wed, May 13, 2015 at 10:06:04AM +0200, Jan Cholasta wrote: Hi, Dne 5.5.2015 v 10:38 Martin Basti napsal(a): On 05/05/15 08:29, Fraser Tweedale wrote: On Mon, May 04, 2015 at 06:35:45PM +0200, Martin Basti wrote: On 04/05/15 15:36, Fraser

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Jan Cholasta
Dne 13.5.2015 v 11:41 Fraser Tweedale napsal(a): Hi Jan, thanks for review. Comments inline. On Wed, May 13, 2015 at 10:06:04AM +0200, Jan Cholasta wrote: 12) IMO the profile backend should be merged in to the ra backend. I don't see a need to have these two separate. I wasn't sure, so I

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Jan Cholasta
Dne 13.5.2015 v 10:36 Martin Basti napsal(a): On 13/05/15 10:06, Jan Cholasta wrote: Hi, Dne 5.5.2015 v 10:38 Martin Basti napsal(a): On 05/05/15 08:29, Fraser Tweedale wrote: On Mon, May 04, 2015 at 06:35:45PM +0200, Martin Basti wrote: On 04/05/15 15:36, Fraser Tweedale wrote: Hello,

Re: [Freeipa-devel] [PATCHES 0239-0243] Server Upgrade: minor fixes

2015-05-13 Thread David Kupka
On 05/12/2015 02:44 PM, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/4904 Patches attached. Works for me, ACK. -- David Kupka -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [PATCH 0246] Don't use proxy to check CA status during install/upgrade

2015-05-13 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/4994 Patch attached. -- Martin Basti From 47572f8771a5ba63b58fd14962acb192de120eea Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Wed, 13 May 2015 10:53:59 +0200 Subject: [PATCH] Dont use the proxy to check CA status Checking status

[Freeipa-devel] [TEST PLAN] User lifecycle plugin

2015-05-13 Thread Lenka Ryznarova
Hi, I've prepared test plan design for User Lifecycle Plugin - [1]. Please review and let me know if you have any comments on that. Thanks, Lenka [1] http://www.freeipa.org/page/V4/User_Life-Cycle_Management/Test_Plan -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-13 Thread Anthony Messina
On Wednesday, May 13, 2015 02:58:40 PM Alexander Bokovoy wrote: On Wed, 13 May 2015, Anthony Messina wrote: On Wednesday, May 13, 2015 01:28:44 PM Martin Babinsky wrote: On 05/12/2015 06:47 PM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Petr Vobornik wrote: On 05/12/2015 11:22 AM,

Re: [Freeipa-devel] [TEST PLAN] User lifecycle plugin

2015-05-13 Thread David Kupka
On 05/13/2015 02:57 PM, Lenka Ryznarova wrote: Hi, I've prepared test plan design for User Lifecycle Plugin - [1]. Please review and let me know if you have any comments on that. Thanks, Lenka [1] http://www.freeipa.org/page/V4/User_Life-Cycle_Management/Test_Plan Hi, thanks for sharing

Re: [Freeipa-devel] [TEST PLAN] User lifecycle plugin

2015-05-13 Thread Martin Basti
On 13/05/15 17:44, David Kupka wrote: On 05/13/2015 02:57 PM, Lenka Ryznarova wrote: Hi, I've prepared test plan design for User Lifecycle Plugin - [1]. Please review and let me know if you have any comments on that. Thanks, Lenka [1]

Re: [Freeipa-devel] [PATCH] Password vault

2015-05-13 Thread Endi Sukma Dewata
On 5/13/2015 4:09 AM, Jan Cholasta wrote: Dne 12.5.2015 v 12:52 Endi Sukma Dewata napsal(a): Please take a look at the attached patch (#353-9). It obsoletes all previous patches. See comments below. On 4/20/2015 1:12 AM, Jan Cholasta wrote: I'm planning to merge the vault and vault container

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Fraser Tweedale
On Wed, May 13, 2015 at 01:19:49PM +0200, Jan Cholasta wrote: Dne 13.5.2015 v 11:41 Fraser Tweedale napsal(a): Hi Jan, thanks for review. Comments inline. On Wed, May 13, 2015 at 10:06:04AM +0200, Jan Cholasta wrote: 12) IMO the profile backend should be merged in to the ra backend. I don't

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-13 Thread Alexander Bokovoy
On Wed, 13 May 2015, Anthony Messina wrote: On Wednesday, May 13, 2015 02:58:40 PM Alexander Bokovoy wrote: On Wed, 13 May 2015, Anthony Messina wrote: On Wednesday, May 13, 2015 01:28:44 PM Martin Babinsky wrote: On 05/12/2015 06:47 PM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Petr

Re: [Freeipa-devel] [PATCHES 0001-0005] Profile management commands

2015-05-13 Thread Jan Cholasta
Hi, Dne 5.5.2015 v 10:38 Martin Basti napsal(a): On 05/05/15 08:29, Fraser Tweedale wrote: On Mon, May 04, 2015 at 06:35:45PM +0200, Martin Basti wrote: On 04/05/15 15:36, Fraser Tweedale wrote: Hello, Please review the first cut of the 'certprofile' command and other changes associated