[Freeipa-devel] [PATCH] 167 Fix service-disable in CA-less install

2013-08-29 Thread Jan Cholasta
Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/3886. Honza -- Jan Cholasta From 90f08d568ecc085bc559c7565bb106e6e7415839 Mon Sep 17 00:00:00 2001 From: Jan Cholasta jchol...@redhat.com Date: Thu, 29 Aug 2013 08:44:43 +0200 Subject: [PATCH] Fix service-disable in CA-less

Re: [Freeipa-devel] [PATCH] 167 Fix service-disable in CA-less install

2013-08-29 Thread Petr Viktorin
On 08/29/2013 08:48 AM, Jan Cholasta wrote: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/3886. Honza Thanks, ACK, pushed to: master: 7c66912824fbc2a4a2d1daf603b204fd7321bd8f ipa-3-3: fe4a96de52c15eb54d828a4ea0144b392d4a101b -- PetrĀ³

Re: [Freeipa-devel] FreeIPA server package group

2013-08-29 Thread Petr Viktorin
On 08/28/2013 12:20 PM, Tomas Babej wrote: On 08/28/2013 12:03 PM, Petr Viktorin wrote: On 08/28/2013 11:46 AM, Tomas Babej wrote: On 08/26/2013 10:14 AM, Tomas Babej wrote: On Mon 26 Aug 2013 10:12:09 AM CEST, Petr Vobornik wrote: On 08/26/2013 09:54 AM, Tomas Babej wrote: Hi, I cooked up

Re: [Freeipa-devel] FreeIPA server package group

2013-08-29 Thread Tomas Babej
On 08/29/2013 11:55 AM, Petr Viktorin wrote: On 08/28/2013 12:20 PM, Tomas Babej wrote: On 08/28/2013 12:03 PM, Petr Viktorin wrote: On 08/28/2013 11:46 AM, Tomas Babej wrote: On 08/26/2013 10:14 AM, Tomas Babej wrote: On Mon 26 Aug 2013 10:12:09 AM CEST, Petr Vobornik wrote: On 08/26/2013

Re: [Freeipa-devel] [PATCH 0087] Log proper error message when defaultNamingContext not found

2013-08-29 Thread Petr Viktorin
On 08/28/2013 02:11 PM, Tomas Babej wrote: On 08/28/2013 01:22 PM, Rob Crittenden wrote: Tomas Babej wrote: Hi, When adding a trust using trust-add with misconfigured DNS, an improper LDAP entry might be returned. Log a proper error message. https://fedorahosted.org/freeipa/ticket/3690 I

[Freeipa-devel] [PATCH] 0059 Create DS user and group during ipa-restore

2013-08-29 Thread Ana Krivokapic
Hello, This patch addresses ticket https://fedorahosted.org/freeipa/ticket/3856. -- Regards, Ana Krivokapic Associate Software Engineer FreeIPA team Red Hat Inc. From f1a2a00f72961c88530be8aa1a62fb15758d90b5 Mon Sep 17 00:00:00 2001 From: Ana Krivokapic akriv...@redhat.com Date: Thu, 29 Aug

Re: [Freeipa-devel] [PATCH] 1104 move NULL check

2013-08-29 Thread Tomas Babej
On 08/28/2013 04:40 PM, Rob Crittenden wrote: Move a NULL check in the lockout plugin to address something Coverity found. There is no risk of a crash here but it there is also no point calling something if we know the values are NULL. rob ___

Re: [Freeipa-devel] [PATCH] 0269 Add man pages for testing tools

2013-08-29 Thread Rob Crittenden
Petr Viktorin wrote: On 08/28/2013 12:02 AM, Rob Crittenden wrote: Petr Viktorin wrote: Hello, This patch adds man pages for testing tools. As far as I can see, we use autotools for installing man pages. I added the autotools machinery to ipatests/man only. I'd appreciate if an autotools

Re: [Freeipa-devel] [PATCH] 119 CLDAP: make sure an empty reply is returned on any error

2013-08-29 Thread Tomas Babej
On 08/28/2013 10:25 AM, Sumit Bose wrote: Hi, this patch fixes an issue in the CLDAP plugin found by Coverity. bye, Sumit ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel ACK -- Tomas

Re: [Freeipa-devel] [PATCH] 0269 Add man pages for testing tools

2013-08-29 Thread Petr Viktorin
On 08/29/2013 03:03 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 08/28/2013 12:02 AM, Rob Crittenden wrote: Petr Viktorin wrote: Hello, This patch adds man pages for testing tools. As far as I can see, we use autotools for installing man pages. I added the autotools machinery to

Re: [Freeipa-devel] [PATCH] 1104 move NULL check

2013-08-29 Thread Petr Viktorin
On 08/29/2013 02:57 PM, Tomas Babej wrote: On 08/28/2013 04:40 PM, Rob Crittenden wrote: Move a NULL check in the lockout plugin to address something Coverity found. There is no risk of a crash here but it there is also no point calling something if we know the values are NULL. rob

Re: [Freeipa-devel] [PATCH] 119 CLDAP: make sure an empty reply is returned on any error

2013-08-29 Thread Petr Viktorin
On 08/29/2013 03:15 PM, Tomas Babej wrote: On 08/28/2013 10:25 AM, Sumit Bose wrote: Hi, this patch fixes an issue in the CLDAP plugin found by Coverity. bye, Sumit ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH] EXTDOM: Do not overwrite domain_name for INP_SID

2013-08-29 Thread Sumit Bose
On Mon, Aug 26, 2013 at 10:14:27AM +0200, Jakub Hrozek wrote: Hi, I found the problem when testing Sumit's PAC responder SSSD patches. It seems that the domain name is always overwritten with input SID. I think using the domain we parse out from output of getnamebysid can be safely used,

Re: [Freeipa-devel] [PATCH] EXTDOM: Do not overwrite domain_name for INP_SID

2013-08-29 Thread Petr Viktorin
On 08/29/2013 03:30 PM, Sumit Bose wrote: On Mon, Aug 26, 2013 at 10:14:27AM +0200, Jakub Hrozek wrote: Hi, I found the problem when testing Sumit's PAC responder SSSD patches. It seems that the domain name is always overwritten with input SID. I think using the domain we parse out from output

Re: [Freeipa-devel] [PATCH] 161 Use configured dogtag LDAP port instead of default one when renewing certs

2013-08-29 Thread Jan Cholasta
On 23.7.2013 18:26, Jan Cholasta wrote: On 22.7.2013 17:40, Simo Sorce wrote: On Mon, 2013-07-22 at 17:36 +0200, Jan Cholasta wrote: if nickname == 'subsystemCert cert-pki-ca': -update_people_entry('pkidbuser', cert) +update_people_entry(dogtag_uri, 'pkidbuser', cert) This is

Re: [Freeipa-devel] [PATCH] 161 Use configured dogtag LDAP port instead of default one when renewing certs

2013-08-29 Thread Jan Cholasta
On 29.8.2013 16:32, Jan Cholasta wrote: On 23.7.2013 18:26, Jan Cholasta wrote: On 22.7.2013 17:40, Simo Sorce wrote: On Mon, 2013-07-22 at 17:36 +0200, Jan Cholasta wrote: if nickname == 'subsystemCert cert-pki-ca': -update_people_entry('pkidbuser', cert) +

[Freeipa-devel] [PATCH] 0060 Add warning when uninstalling active replica

2013-08-29 Thread Ana Krivokapic
Hello, This patch addresses ticket https://fedorahosted.org/freeipa/ticket/3867. -- Regards, Ana Krivokapic Associate Software Engineer FreeIPA team Red Hat Inc. From 29732542431dfa3d2a41607f6537690655a7a027 Mon Sep 17 00:00:00 2001 From: Ana Krivokapic akriv...@redhat.com Date: Thu, 29 Aug

[Freeipa-devel] Announcing FreeIPA 3.3.1

2013-08-29 Thread Petr Viktorin
The FreeIPA team is proud to announce FreeIPA v3.3.1! This is a bugfix release. It can be downloaded from http://www.freeipa.org/page/Downloads. Fedora 19 builds will be ready soon. == Highlights in 3.3.1 == === Bug fixes === * ipa-server-certinstall now works correctly both with a CA