Re: [Freeipa-devel] [PATCH] 745 restart dogtag DS instance after install

2011-03-03 Thread Martin Kosek
On Wed, 2011-03-02 at 16:51 -0500, Rob Crittenden wrote: > The dogtag team tells me we should restart their LDAP backend right > after installation. In some configurations not doing this can cause > problems (using the CA as we do isn't one of the known cases but better > safe than sorry). To do

Re: [Freeipa-devel] [PATCH] 745 restart dogtag DS instance after install

2011-03-03 Thread Rob Crittenden
Martin Kosek wrote: On Wed, 2011-03-02 at 16:51 -0500, Rob Crittenden wrote: The dogtag team tells me we should restart their LDAP backend right after installation. In some configurations not doing this can cause problems (using the CA as we do isn't one of the known cases but better safe than s

Re: [Freeipa-devel] [PATCH] 78 Use ldapi: instead of unsecured ldap: in ipa core tools.

2011-03-03 Thread Martin Kosek
On Mon, 2011-02-28 at 18:15 +, JR Aquino wrote: > > On 2/25/11 9:27 AM, "Pavel Zůna" wrote: > > >On 2011-02-25 18:12, JR Aquino wrote: > >> > >> > >> On 2/25/11 5:58 AM, "Pavel Zuna" wrote: > >> > >>> On 02/23/2011 11:53 PM, Simo Sorce wrote: > On Wed, 23 Feb 2011 23:41:33 +0100 >

Re: [Freeipa-devel] [PATCH] 745 restart dogtag DS instance after install

2011-03-03 Thread Martin Kosek
On Thu, 2011-03-03 at 09:30 -0500, Rob Crittenden wrote: > Martin Kosek wrote: > > On Wed, 2011-03-02 at 16:51 -0500, Rob Crittenden wrote: > >> The dogtag team tells me we should restart their LDAP backend right > >> after installation. In some configurations not doing this can cause > >> problems

Re: [Freeipa-devel] [PATCH] 065 Replace only if old and new have nothing in common

2011-03-03 Thread Pavel Zuna
On 03/02/2011 08:50 PM, Jakub Hrozek wrote: On Wed, Feb 23, 2011 at 12:36:06PM -0500, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/23/2011 04:47 PM, Rob Crittenden wrote: Jakub Hrozek wrote: Replace only if old and new have nothing in common

[Freeipa-devel] [PATCH] admiyo-0208-fix-truncated-message

2011-03-03 Thread Adam Young
From b2ee5e6bfe0b5d74ddef8a8a0854ad6606dcb9e0 Mon Sep 17 00:00:00 2001 From: Adam Young Date: Wed, 2 Mar 2011 23:21:07 -0500 Subject: [PATCH] fix truncated message --- install/ui/policy.js |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/install/ui/policy.js b/i

Re: [Freeipa-devel] [PATCH] 035 IPA replica/server install does not check for a client

2011-03-03 Thread Rob Crittenden
Martin Kosek wrote: When IPA replica or server is configured it does not check for possibly installed client. This will cause the installation to fail in the very end. This patch adds a check for already configured client and suggests removing it before server/replica installation. https://fedo

[Freeipa-devel] [PATCH] Fix error in user plugin email normalizer for empty --setattr=mail=.

2011-03-03 Thread Pavel Zuna
An exception was raised when you tried to reset user email addresses and setting new ones using: ipa user-add SOMEUSER --setattr=mail= --addattr=mail=someu...@redhat.com Pavel freeipa-pzuna-85-fixemailnorm.patch Description: application/mbox ___ Fre

Re: [Freeipa-devel] [PATCH] Fix error in user plugin email normalizer for empty --setattr=mail=.

2011-03-03 Thread Pavel Zuna
On 03/03/2011 04:23 PM, Pavel Zuna wrote: An exception was raised when you tried to reset user email addresses and setting new ones using: ipa user-add SOMEUSER --setattr=mail= --addattr=mail=someu...@redhat.com Pavel Just a correction: The example above should read 'ipa user-mod ...' ofc.

Re: [Freeipa-devel] [PATCH] admiyo-0208-fix-truncated-message

2011-03-03 Thread Endi Sukma Dewata
On 3/3/2011 9:23 AM, Adam Young wrote: ACK and pushed to master. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 036 Inconsistent sysrestore file handling by IPA server installer

2011-03-03 Thread Rob Crittenden
Martin Kosek wrote: IPA server/replica uninstallation may fail when it tries to restore a Directory server configuration file in sysrestore directory, which was already restored before. The problem is in Directory Server uninstaller which uses and modifies its own image of sysrestore directory s

Re: [Freeipa-devel] [PATCH] 065 Replace only if old and new have nothing in common

2011-03-03 Thread Rob Crittenden
Pavel Zuna wrote: On 03/02/2011 08:50 PM, Jakub Hrozek wrote: On Wed, Feb 23, 2011 at 12:36:06PM -0500, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/23/2011 04:47 PM, Rob Crittenden wrote: Jakub Hrozek wrote: Replace only if old and new have

Re: [Freeipa-devel] [PATCH] 745 restart dogtag DS instance after install

2011-03-03 Thread Rob Crittenden
Martin Kosek wrote: On Thu, 2011-03-03 at 09:30 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Wed, 2011-03-02 at 16:51 -0500, Rob Crittenden wrote: The dogtag team tells me we should restart their LDAP backend right after installation. In some configurations not doing this can cause prob

Re: [Freeipa-devel] [PATCH] 78 Use ldapi: instead of unsecured ldap: in ipa core tools.

2011-03-03 Thread Martin Kosek
On Thu, 2011-03-03 at 15:29 +0100, Martin Kosek wrote: > On Mon, 2011-02-28 at 18:15 +, JR Aquino wrote: > > > > On 2/25/11 9:27 AM, "Pavel Zůna" wrote: > > > > >On 2011-02-25 18:12, JR Aquino wrote: > > >> > > >> > > >> On 2/25/11 5:58 AM, "Pavel Zuna" wrote: > > >> > > >>> On 02/23/2011 1

[Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Adam Young
From 4253912efaa0a0853ed963bcb749aea388e23770 Mon Sep 17 00:00:00 2001 From: Adam Young Date: Thu, 3 Mar 2011 11:48:53 -0500 Subject: [PATCH] typo in truncation message --- ipalib/plugins/internal.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ipalib/plugins/inter

Re: [Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Endi Sukma Dewata
On 3/3/2011 10:50 AM, Adam Young wrote: ACK and pushed to master. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Dmitri Pal
On 03/03/2011 11:50 AM, Adam Young wrote: > > > ___ > Freeipa-devel mailing list > Freeipa-devel@redhat.com > https://www.redhat.com/mailman/listinfo/freeipa-devel Nack "Query returned more results than the configured size limit *allows to* show." "wil

Re: [Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Dmitri Pal
On 03/03/2011 12:13 PM, Endi Sukma Dewata wrote: > On 3/3/2011 10:50 AM, Adam Young wrote: >> > > ACK and pushed to master. > Akhm :-) -- Thank you, Dmitri Pal Sr. Engineering Manager IPA project, Red Hat Inc. --- Looking to carve out IT costs? www.redhat.com/carveo

Re: [Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Endi Sukma Dewata
On 3/3/2011 11:16 AM, Dmitri Pal wrote: On 03/03/2011 12:13 PM, Endi Sukma Dewata wrote: ACK and pushed to master. Akhm :-) Sorry, we can fix this again in another patch. So what should be the correct wording? While we're at it, is the second sentence grammatically correct? "First ${counte

Re: [Freeipa-devel] [PATCH] admiyo-0209-typo-in-truncation-message

2011-03-03 Thread Dmitri Pal
On 03/03/2011 12:34 PM, Endi Sukma Dewata wrote: > On 3/3/2011 11:16 AM, Dmitri Pal wrote: >> On 03/03/2011 12:13 PM, Endi Sukma Dewata wrote: >>> ACK and pushed to master. >> Akhm :-) > > Sorry, we can fix this again in another patch. So what should be the > correct wording? > > While we're at it,

Re: [Freeipa-devel] [PATCH] Fix error in user plugin email normalizer for empty --setattr=mail=.

2011-03-03 Thread Rob Crittenden
Pavel Zuna wrote: ipa user-add SOMEUSER --setattr=mail= --addattr=mail=someu...@redhat.com Ack, pushed to master I created ticket 1048 for this problem and ammended the git commit message. rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH] 78 Use ldapi: instead of unsecured ldap: in ipa core tools.

2011-03-03 Thread Rob Crittenden
Martin Kosek wrote: On Thu, 2011-03-03 at 15:29 +0100, Martin Kosek wrote: On Mon, 2011-02-28 at 18:15 +, JR Aquino wrote: On 2/25/11 9:27 AM, "Pavel Zůna" wrote: On 2011-02-25 18:12, JR Aquino wrote: On 2/25/11 5:58 AM, "Pavel Zuna" wrote: On 02/23/2011 11:53 PM, Simo Sorce wrot

[Freeipa-devel] [PATCH] two one liners for verbage.

2011-03-03 Thread Adam Young
Since we are close to GA, I'll submit these for review instead of pushing under the one line rule. From 4fbcb5bfa3f93d0ed9d886afef8de60ed9f099ef Mon Sep 17 00:00:00 2001 From: Adam Young Date: Thu, 3 Mar 2011 14:14:59 -0500 Subject: [PATCH 210/211] type in default text --- install/ui/index.html

Re: [Freeipa-devel] [PATCH] two one liners for verbage.

2011-03-03 Thread Endi Sukma Dewata
On 3/3/2011 1:16 PM, Adam Young wrote: Since we are close to GA, I'll submit these for review instead of pushing under the one line rule. ACK and pushed both to master. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https:

[Freeipa-devel] [PATCH] 747 don't check DNS for sanity if we're installing DNS

2011-03-03 Thread Rob Crittenden
Skip the DNS checks during installation if we're configuring IPA as a DNS server. ticket 1036 rob freeipa-rcrit-747-install.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/li

Re: [Freeipa-devel] [PATCH] 747 don't check DNS for sanity if we're installing DNS

2011-03-03 Thread Simo Sorce
On Thu, 03 Mar 2011 16:11:24 -0500 Rob Crittenden wrote: > Skip the DNS checks during installation if we're configuring IPA as a > DNS server. > > ticket 1036 ACK Simo. -- Simo Sorce * Red Hat, Inc * New York ___ Freeipa-devel mailing list Freeip

Re: [Freeipa-devel] [PATCH] 746 style and grammatical issues in help

2011-03-03 Thread David O'Brien
Rob Crittenden wrote: Fix style and grammatical issues in built-in command help. There is a rather large API.txt change but it is only due to changes in the doc string in parameters. ticket 729 rob Couple of picks: --maxusername=INT Max. username length when creating/modifing a user (m