Re: [Freeipa-devel] [PATCH 0170] Detect and warn about invalid forwardzone configuration

2014-12-11 Thread Petr Spacek
On 11.12.2014 16:50, Martin Basti wrote: > Updated aptch attached: Nice work, ACK! -- Petr^2 Spacek ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] topology management question

2014-12-11 Thread Simo Sorce
On Thu, 11 Dec 2014 17:03:55 +0100 Petr Spacek wrote: > On 11.12.2014 15:20, Simo Sorce wrote: > > On Thu, 11 Dec 2014 14:18:36 +0100 > > Ludwig Krispenz wrote: > > > >> > >> On 12/05/2014 04:50 PM, Simo Sorce wrote: > >>> On Thu, 04 Dec 2014 14:33:09 +0100 > >>> Ludwig Krispenz wrote: > >>> >

Re: [Freeipa-devel] topology management question

2014-12-11 Thread Petr Spacek
On 11.12.2014 15:20, Simo Sorce wrote: > On Thu, 11 Dec 2014 14:18:36 +0100 > Ludwig Krispenz wrote: > >> >> On 12/05/2014 04:50 PM, Simo Sorce wrote: >>> On Thu, 04 Dec 2014 14:33:09 +0100 >>> Ludwig Krispenz wrote: >>> hi, I just have another (hopefully this will end soon) issue

Re: [Freeipa-devel] [PATCH 0170] Detect and warn about invalid forwardzone configuration

2014-12-11 Thread Martin Basti
Updated aptch attached: diff with previous: diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py index f9d8321..7a80036 100644 --- a/ipalib/plugins/dns.py +++ b/ipalib/plugins/dns.py @@ -1735,7 +1735,7 @@ def _normalize_zone(zone): def _get_auth_zone_ldap(name): """ -Find author

Re: [Freeipa-devel] [PATCH 0170] Detect and warn about invalid forwardzone configuration

2014-12-11 Thread Petr Spacek
Hello, I have only few nitpicks and one minor non-nitpick. Rest is in-line. On 10.12.2014 18:20, Martin Basti wrote: > freeipa-mbasti-0170.4-Detect-and-warn-about-invalid-DNS-forward-zone-confi.patch > > > From a1b70e7a12ffdb08941d43587a05d7e36b57ab2b Mon Sep 17 00:00:00 2001 > From: Martin Bas

Re: [Freeipa-devel] [PATCH 0291] idviews: Complain if host is already assigned the ID View in idview-apply

2014-12-11 Thread Tomas Babej
On 12/11/2014 11:07 AM, Jan Cholasta wrote: > Hi, > > Dne 11.12.2014 v 09:23 Tomas Babej napsal(a): >> Hi, >> >> When running a idview-apply command, the hosts that were already >> assigned >> the desired view were silently ignored. Make sure such hosts show up in >> the list of failed hosts. >> >

Re: [Freeipa-devel] FreeIPA integration with external DNS services

2014-12-11 Thread Martin Kosek
On 12/11/2014 03:05 PM, Simo Sorce wrote: > On Thu, 11 Dec 2014 10:43:02 +0100 > Petr Spacek wrote: > >> On 10.12.2014 18:50, Simo Sorce wrote: >>> On Wed, 10 Dec 2014 15:13:30 +0100 >>> Petr Spacek wrote: >>> I think that external DNS could depend on Vault (assuming that external DNS

Re: [Freeipa-devel] topology management question

2014-12-11 Thread Simo Sorce
On Thu, 11 Dec 2014 14:18:36 +0100 Ludwig Krispenz wrote: > > On 12/05/2014 04:50 PM, Simo Sorce wrote: > > On Thu, 04 Dec 2014 14:33:09 +0100 > > Ludwig Krispenz wrote: > > > >> hi, > >> > >> I just have another (hopefully this will end soon) issue I want to > >> get your input. (please read t

Re: [Freeipa-devel] FreeIPA integration with external DNS services

2014-12-11 Thread Simo Sorce
On Thu, 11 Dec 2014 10:43:02 +0100 Petr Spacek wrote: > On 10.12.2014 18:50, Simo Sorce wrote: > > On Wed, 10 Dec 2014 15:13:30 +0100 > > Petr Spacek wrote: > > > >> I think that external DNS could depend on Vault (assuming that > >> external DNS support will be purely optional). > > > > TBH,

Re: [Freeipa-devel] topology management question

2014-12-11 Thread Ludwig Krispenz
On 12/05/2014 04:50 PM, Simo Sorce wrote: On Thu, 04 Dec 2014 14:33:09 +0100 Ludwig Krispenz wrote: hi, I just have another (hopefully this will end soon) issue I want to get your input. (please read to teh end first) To recapture the conditions: - the topology plugin manages the connectio

Re: [Freeipa-devel] [PATCH 0040] Remove dependency on subscription-manager

2014-12-11 Thread Martin Kosek
On 12/11/2014 12:05 PM, Martin Basti wrote: > On 11/12/14 05:01, Gabe Alford wrote: >> Updated patch attached. >> >> Thanks, >> >> Gabe >> >> On Wed, Dec 10, 2014 at 8:02 AM, Martin Basti > > wrote: >> >> On 10/12/14 15:45, Gabe Alford wrote: >> >> Hello, >> >>

Re: [Freeipa-devel] [PATCH 0177] Fix adding (warning) messages on client side

2014-12-11 Thread Martin Basti
On 11/12/14 11:19, Jan Cholasta wrote: Hi, Dne 9.12.2014 v 16:07 Martin Basti napsal(a): Ticket: https://fedorahosted.org/freeipa/ticket/4793 I'm able to reproduce it only in one nose test. Which test? If you apply my patch 170 and add a random forwardzone, then DNS root zone tests failed.

Re: [Freeipa-devel] [PATCH 0040] Remove dependency on subscription-manager

2014-12-11 Thread Martin Basti
On 11/12/14 05:01, Gabe Alford wrote: Updated patch attached. Thanks, Gabe On Wed, Dec 10, 2014 at 8:02 AM, Martin Basti > wrote: On 10/12/14 15:45, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4783 Thanks,

Re: [Freeipa-devel] [PATCH 0168] Better workaround to get status of CA during upgrade

2014-12-11 Thread Jan Cholasta
Dne 11.12.2014 v 10:01 Martin Basti napsal(a): On 10/12/14 19:21, Jan Cholasta wrote: Dne 10.12.2014 v 18:01 Jan Cholasta napsal(a): Dne 1.12.2014 v 16:48 Martin Basti napsal(a): On 01/12/14 08:46, Jan Cholasta wrote: Hi, Dne 27.11.2014 v 14:24 Martin Basti napsal(a): Ticket: https://fedora

Re: [Freeipa-devel] [PATCH 0177] Fix adding (warning) messages on client side

2014-12-11 Thread Jan Cholasta
Hi, Dne 9.12.2014 v 16:07 Martin Basti napsal(a): Ticket: https://fedorahosted.org/freeipa/ticket/4793 I'm able to reproduce it only in one nose test. Which test? Patch attached. What about: result['messages'] = result.get('messages', ()) + (message.to_dict(),) (My point is, don't

Re: [Freeipa-devel] [PATCH 0291] idviews: Complain if host is already assigned the ID View in idview-apply

2014-12-11 Thread Jan Cholasta
Hi, Dne 11.12.2014 v 09:23 Tomas Babej napsal(a): Hi, When running a idview-apply command, the hosts that were already assigned the desired view were silently ignored. Make sure such hosts show up in the list of failed hosts. https://fedorahosted.org/freeipa/ticket/4743 Shouldn't the error m

Re: [Freeipa-devel] FreeIPA integration with external DNS services

2014-12-11 Thread Petr Spacek
On 10.12.2014 18:50, Simo Sorce wrote: > On Wed, 10 Dec 2014 15:13:30 +0100 > Petr Spacek wrote: > >> I think that external DNS could depend on Vault (assuming that >> external DNS support will be purely optional). > > TBH, I do not think this is a sensible option, the Vault will drag huge > dep

Re: [Freeipa-devel] [PATCH 0168] Better workaround to get status of CA during upgrade

2014-12-11 Thread Martin Basti
On 10/12/14 19:21, Jan Cholasta wrote: Dne 10.12.2014 v 18:01 Jan Cholasta napsal(a): Dne 1.12.2014 v 16:48 Martin Basti napsal(a): On 01/12/14 08:46, Jan Cholasta wrote: Hi, Dne 27.11.2014 v 14:24 Martin Basti napsal(a): Ticket: https://fedorahosted.org/freeipa/ticket/4676 Replaces current

[Freeipa-devel] [PATCH 0291] idviews: Complain if host is already assigned the ID View in idview-apply

2014-12-11 Thread Tomas Babej
Hi, When running a idview-apply command, the hosts that were already assigned the desired view were silently ignored. Make sure such hosts show up in the list of failed hosts. https://fedorahosted.org/freeipa/ticket/4743 -- Tomas Babej Associate Software Engineer | Red Hat | Identity Management