Re: [Freeipa-devel] ds-migrate feature enhancements

2014-11-26 Thread Martin Kosek
On 11/24/2014 11:24 PM, Alan Evans wrote: > I am in the midst of preparing for a migration from OpenLDAP to FreeIPA. > ds-migrate wasn't going to fill all of my needs so I thought I would use it > for most and then make up some LDIF's and massage them to do the last bit > of migration. > > I have

Re: [Freeipa-devel] [PATCH] 0033 Use singular in help metavars + update man pages.

2014-11-26 Thread Martin Basti
On 25/11/14 15:44, David Kupka wrote: On 11/25/2014 03:23 PM, Martin Basti wrote: On 25/11/14 13:16, David Kupka wrote: On 11/25/2014 09:57 AM, David Kupka wrote: On 11/25/2014 09:51 AM, David Kupka wrote: On 11/24/2014 03:59 PM, Martin Basti wrote: On 24/11/14 15:54, David Kupka wrote: htt

Re: [Freeipa-devel] [PATCH][bind-dyndb-ldap] AUTOCONF: Improve detection of bind9 header files

2014-11-26 Thread Tomas Hozza
On 11/12/2014 03:30 PM, Petr Spacek wrote: > On 24.7.2014 11:00, Petr Spacek wrote: > > On 27.2.2014 15:19, Lukas Slebodnik wrote: > >> ehlo, > >> > >> I did some reviews of bind-dyndb-ldap last week and it was little bit > >> annoying > >> to export special CFLAGS for bind9 header files. It can

Re: [Freeipa-devel] [PATCH] 0033 Use singular in help metavars + update man pages.

2014-11-26 Thread David Kupka
On 11/26/2014 12:23 PM, Martin Basti wrote: On 25/11/14 15:44, David Kupka wrote: On 11/25/2014 03:23 PM, Martin Basti wrote: On 25/11/14 13:16, David Kupka wrote: On 11/25/2014 09:57 AM, David Kupka wrote: On 11/25/2014 09:51 AM, David Kupka wrote: On 11/24/2014 03:59 PM, Martin Basti wrote

Re: [Freeipa-devel] [PATCH] 0033 Use singular in help metavars + update man pages.

2014-11-26 Thread Martin Basti
On 26/11/14 12:37, David Kupka wrote: On 11/26/2014 12:23 PM, Martin Basti wrote: On 25/11/14 15:44, David Kupka wrote: On 11/25/2014 03:23 PM, Martin Basti wrote: On 25/11/14 13:16, David Kupka wrote: On 11/25/2014 09:57 AM, David Kupka wrote: On 11/25/2014 09:51 AM, David Kupka wrote: On

Re: [Freeipa-devel] [PATCH 0228] Drop unnecessary #define _BSD_SOURCE

2014-11-26 Thread Tomas Hozza
On 11/25/2014 07:53 PM, Martin Basti wrote: > On 12/11/14 16:34, Petr Spacek wrote: > > On 25.2.2014 15:05, Lukas Slebodnik wrote: > >> On (25/02/14 09:54), Petr Spacek wrote: > >>> On 24.2.2014 18:56, Lukas Slebodnik wrote: > On (24/02/14 16:48), Petr Spacek wrote: > > Hello, > > > >>

Re: [Freeipa-devel] [PATCH 0306] Improve info messages about number of defined/loaded zones

2014-11-26 Thread Tomas Hozza
On 11/07/2014 01:33 PM, Petr Spacek wrote: > Hello, > > Improve info messages about number of defined/loaded zones. > ACK. The new message looks good. Regards, -- Tomas Hozza Software Engineer - EMEA ENG Developer Experience PGP: 1D9F3C2D Red Hat Inc. http://cz.redhat.com _

Re: [Freeipa-devel] [PATCH 0287] Re-initialize NSS database after otptoken plugin tests

2014-11-26 Thread Petr Viktorin
On 11/21/2014 11:47 AM, Tomas Babej wrote: Hi, OTP token tests do not properly reinitialize the NSS db, thus making subsequent xmlrpc tests fail on SSL cert validation. Make sure NSS db is re-initalized in the teardown method. https://fedorahosted.org/freeipa/ticket/4748 Note for reviewers: R

[Freeipa-devel] [PATCH] 0676 Ignore ipap11helper/setup.py in doctests

2014-11-26 Thread Petr Viktorin
A fix in test configuration. Pushed as one-liner to master: 4e99663379b333e2dd851f06a786c705c3e07e4b I've opened https://fedorahosted.org/freeipa/ticket/4769 to fix this a better way; either in upstream pytest (if they deem projects with multiple setup.py files as important enough), or in our

Re: [Freeipa-devel] [PATCH 0309] Fix crash caused by interaction between forward and master zones

2014-11-26 Thread Tomas Hozza
On 11/25/2014 07:07 PM, Martin Basti wrote: > On 25/11/14 18:11, Petr Spacek wrote: > > Hello, > > > > Fix crash caused by interaction between forward and master zones. > > > > LDAP modifications made to idnsName=sub, idnsName=example.com, cn=dns object > > were incorrectly processed using update_z

Re: [Freeipa-devel] [PATCH 0308] Improve detection of BIND 9 isc__errno2result header file

2014-11-26 Thread Tomas Hozza
On 11/25/2014 07:48 PM, Martin Basti wrote: > On 12/11/14 16:11, Petr Spacek wrote: > > Hello, > > > > Improve detection of BIND 9 isc__errno2result header file. > > > > This header file is not in standard distribution so normal isc-config.sh > > detection is not enough. > > > > With this patch, ./

Re: [Freeipa-devel] [PATCH 0307] Send DNS NOTIFY message after any modification to the zone

2014-11-26 Thread Martin Basti
On 07/11/14 15:34, Petr Spacek wrote: Hello, Send DNS NOTIFY message after any modification to the zone. https://fedorahosted.org/bind-dyndb-ldap/ticket/144 Works for me. But don't push it before Tomas check the code please. Martin^2 -- Martin Basti _

Re: [Freeipa-devel] [PATCH 0310] Fix misleading error message about forward zones on reconnect

2014-11-26 Thread Tomas Hozza
On 11/25/2014 07:25 PM, Martin Basti wrote: > On 25/11/14 18:27, Petr Spacek wrote: > > Hello, > > > > Fix misleading error message about forward zones on reconnect. > > > > Previously the plugin could log 'already exist' error after > > successful reconnection to LDAP for each active forward zone.

[Freeipa-devel] Release notes of FreeIPA 4.1.2

2014-11-26 Thread Petr Vobornik
Hi all, FreeIPA 4.1.2 release was created yesterday. The only missing step is sending an announce mail. Here's release notes draft: - http://www.freeipa.org/page/Releases/4.1.2 Feel free to amend. I'll wait 2 hours and then send it. -- Petr Vobornik _

Re: [Freeipa-devel] [PATCH 0037] ipa-managed-entries prompts for password with -p option and incorrect passowrd

2014-11-26 Thread Martin Basti
On 24/11/14 04:43, Gabe Alford wrote: Hello, I have a fix for https://fedorahosted.org/freeipa/ticket/4089 thanks, Gabe Thank you! ACK -- Martin Basti ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinf

Re: [Freeipa-devel] [PATCH] 0033 Use singular in help metavars + update man pages.

2014-11-26 Thread Petr Vobornik
On 11/26/2014 01:04 PM, Martin Basti wrote: On 26/11/14 12:37, David Kupka wrote: Fixed. Thanks! ACK Pushed to ipa-4-1: 2f8c4e7b165609706a4af9f680579c0e47edaeca Pushed to master: 3a6d714bb229f8dd68ae219d94283f05cf57a6d7 -- Petr Vobornik ___ Fr

Re: [Freeipa-devel] [PATCH 0037] ipa-managed-entries prompts for password with -p option and incorrect passowrd

2014-11-26 Thread Petr Vobornik
On 11/26/2014 02:17 PM, Martin Basti wrote: On 24/11/14 04:43, Gabe Alford wrote: Hello, I have a fix for https://fedorahosted.org/freeipa/ticket/4089 thanks, Gabe Thank you! ACK Pushed to master: 45dbd12d8886ca2025bcab5b10ec5e004af3d9ab -- Petr Vobornik ___

Re: [Freeipa-devel] [PATCH][bind-dyndb-ldap] AUTOCONF: Improve detection of bind9 header files

2014-11-26 Thread Lukas Slebodnik
On (12/11/14 15:30), Petr Spacek wrote: >On 24.7.2014 11:00, Petr Spacek wrote: >> On 27.2.2014 15:19, Lukas Slebodnik wrote: >>> ehlo, >>> >>> I did some reviews of bind-dyndb-ldap last week and it was little bit >>> annoying >>> to export special CFLAGS for bind9 header files. It can be automat

[Freeipa-devel] [PATCH] 0677 test_integration: Use python-pytest-multihost

2014-11-26 Thread Petr Viktorin
Hello, I have split FreeIPA's multi-host testing infrastructure into a separate project. It is temoprarily available at: https://github.com/encukou/pytest-multihost and I will move it to fedorahosted as soon as it's approved: https://fedorahosted.org/fedora-infrastructure/ticket/4605 RP

Re: [Freeipa-devel] [PATCH 0228] Drop unnecessary #define _BSD_SOURCE

2014-11-26 Thread Lukas Slebodnik
On (12/11/14 16:34), Petr Spacek wrote: >On 25.2.2014 15:05, Lukas Slebodnik wrote: >> On (25/02/14 09:54), Petr Spacek wrote: >>> On 24.2.2014 18:56, Lukas Slebodnik wrote: On (24/02/14 16:48), Petr Spacek wrote: > Hello, > > Drop unnecessary #define _BSD_SOURCE. > > -- >>

Re: [Freeipa-devel] [PATCH 0228] Drop unnecessary #define _BSD_SOURCE

2014-11-26 Thread Petr Spacek
On 26.11.2014 16:47, Lukas Slebodnik wrote: > On (12/11/14 16:34), Petr Spacek wrote: >> On 25.2.2014 15:05, Lukas Slebodnik wrote: >>> On (25/02/14 09:54), Petr Spacek wrote: On 24.2.2014 18:56, Lukas Slebodnik wrote: > On (24/02/14 16:48), Petr Spacek wrote: >> Hello, >> >> D

Re: [Freeipa-devel] [PATCH][bind-dyndb-ldap] AUTOCONF: Improve detection of bind9 header files

2014-11-26 Thread Petr Spacek
On 26.11.2014 15:57, Lukas Slebodnik wrote: > On (12/11/14 15:30), Petr Spacek wrote: >> On 24.7.2014 11:00, Petr Spacek wrote: >>> On 27.2.2014 15:19, Lukas Slebodnik wrote: ehlo, I did some reviews of bind-dyndb-ldap last week and it was little bit annoying to export sp

Re: [Freeipa-devel] [PATCH 0228] Drop unnecessary #define _BSD_SOURCE

2014-11-26 Thread Lukas Slebodnik
On (26/11/14 16:51), Petr Spacek wrote: >On 26.11.2014 16:47, Lukas Slebodnik wrote: >> On (12/11/14 16:34), Petr Spacek wrote: >>> On 25.2.2014 15:05, Lukas Slebodnik wrote: On (25/02/14 09:54), Petr Spacek wrote: > On 24.2.2014 18:56, Lukas Slebodnik wrote: >> On (24/02/14 16:48), Pe

Re: [Freeipa-devel] [PATCH][bind-dyndb-ldap] AUTOCONF: Improve detection of bind9 header files

2014-11-26 Thread Lukas Slebodnik
On (26/11/14 16:52), Petr Spacek wrote: >On 26.11.2014 15:57, Lukas Slebodnik wrote: >> On (12/11/14 15:30), Petr Spacek wrote: >>> On 24.7.2014 11:00, Petr Spacek wrote: On 27.2.2014 15:19, Lukas Slebodnik wrote: > ehlo, > > I did some reviews of bind-dyndb-ldap last week and it

Re: [Freeipa-devel] [PATCH 0228] Drop unnecessary #define _BSD_SOURCE

2014-11-26 Thread Petr Spacek
On 26.11.2014 13:04, Tomas Hozza wrote: > On 11/25/2014 07:53 PM, Martin Basti wrote: >> > On 12/11/14 16:34, Petr Spacek wrote: >>> > > On 25.2.2014 15:05, Lukas Slebodnik wrote: > >> On (25/02/14 09:54), Petr Spacek wrote: > > >>> On 24.2.2014 18:56, Lukas Slebodnik wrote: >> >

Re: [Freeipa-devel] [PATCH 0309] Fix crash caused by interaction between forward and master zones

2014-11-26 Thread Petr Spacek
On 26.11.2014 13:33, Tomas Hozza wrote: > On 11/25/2014 07:07 PM, Martin Basti wrote: >> > On 25/11/14 18:11, Petr Spacek wrote: >>> > > Hello, >>> > > >>> > > Fix crash caused by interaction between forward and master zones. >>> > > >>> > > LDAP modifications made to idnsName=sub, idnsName=example

Re: [Freeipa-devel] [PATCH 0306] Improve info messages about number of defined/loaded zones

2014-11-26 Thread Petr Spacek
On 26.11.2014 13:07, Tomas Hozza wrote: > On 11/07/2014 01:33 PM, Petr Spacek wrote: >> > Hello, >> > >> > Improve info messages about number of defined/loaded zones. >> > > ACK. > > The new message looks good. Pushed to master: eb600df6af932292e0a15817710cfc674f5c952b -- Petr^2 Spacek ___

Re: [Freeipa-devel] [PATCH][bind-dyndb-ldap] AUTOCONF: Improve detection of bind9 header files

2014-11-26 Thread Petr Spacek
On 26.11.2014 12:33, Tomas Hozza wrote: > On 11/12/2014 03:30 PM, Petr Spacek wrote: >> > On 24.7.2014 11:00, Petr Spacek wrote: >>> > > On 27.2.2014 15:19, Lukas Slebodnik wrote: > >> ehlo, > >> > >> I did some reviews of bind-dyndb-ldap last week and it was little > >> bit a

Re: [Freeipa-devel] [PATCH 0310] Fix misleading error message about forward zones on reconnect

2014-11-26 Thread Petr Spacek
On 26.11.2014 14:13, Tomas Hozza wrote: > On 11/25/2014 07:25 PM, Martin Basti wrote: >> > On 25/11/14 18:27, Petr Spacek wrote: >>> > > Hello, >>> > > >>> > > Fix misleading error message about forward zones on reconnect. >>> > > >>> > > Previously the plugin could log 'already exist' error after

Re: [Freeipa-devel] [PATCH 0308] Improve detection of BIND 9 isc__errno2result header file

2014-11-26 Thread Petr Spacek
On 26.11.2014 13:39, Tomas Hozza wrote: > On 11/25/2014 07:48 PM, Martin Basti wrote: >> > On 12/11/14 16:11, Petr Spacek wrote: >>> > > Hello, >>> > > >>> > > Improve detection of BIND 9 isc__errno2result header file. >>> > > >>> > > This header file is not in standard distribution so normal isc-c

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-26 Thread Gabe Alford
Hello, Wondering if I could get a review. Updated patch attached. Thanks, Gabe On Tue, Nov 11, 2014 at 7:21 AM, Gabe Alford wrote: > Hello, > > Fix for https://fedorahosted.org/freeipa/ticket/4700 > > Thanks, > > Gabe > From 4af61cbaafdadb1b338601699d160bd4a15db551 Mon Sep 17 00:00:00 2