Re: [Freeipa-devel] [PATCH 0153] fix regression: DNS zonemgr validation raises assertion error

2014-10-27 Thread Petr Viktorin 

On 10/24/2014 02:12 PM, David Kupka wrote:

Works for me, ACK.

On 10/24/2014 01:27 PM, Martin Basti wrote:

https://fedorahosted.org/freeipa/ticket/4663

Patch attached.





Pushed to:
master: e971fad5c1235bb755e58d131f8183f1646770e6
ipa-4-1: 75cdc50ba91a94682eac11f5746be283f668

--
Petr³

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH 0153] fix regression: DNS zonemgr validation raises assertion error

2014-10-24 Thread Martin Basti 

https://fedorahosted.org/freeipa/ticket/4663

Patch attached.

--
Martin Basti

From aac897f2c269de06675a6a5cca74b0d30ad3d39e Mon Sep 17 00:00:00 2001
From: Martin Basti mba...@redhat.com
Date: Fri, 24 Oct 2014 12:15:17 +0200
Subject: [PATCH] Fix dns zonemgr validation regression

https://fedorahosted.org/freeipa/ticket/4663
---
 ipalib/util.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ipalib/util.py b/ipalib/util.py
index fcb2bab96bcf5669de444846d8dea572eefce793..7a283106d70ba6a3e25cc7129d57b44b80876882 100644
--- a/ipalib/util.py
+++ b/ipalib/util.py
@@ -277,6 +277,7 @@ def validate_zonemgr(zonemgr):
 
 def validate_zonemgr_str(zonemgr):
 zonemgr = normalize_zonemgr(zonemgr)
+zonemgr = DNSName(zonemgr)
 return validate_zonemgr(zonemgr)
 
 def validate_hostname(hostname, check_fqdn=True, allow_underscore=False, allow_slash=False):
-- 
1.8.3.1

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH 0153] fix regression: DNS zonemgr validation raises assertion error

2014-10-24 Thread David Kupka 

Works for me, ACK.

On 10/24/2014 01:27 PM, Martin Basti wrote:

https://fedorahosted.org/freeipa/ticket/4663

Patch attached.



--
David Kupka

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel