Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-02 Thread Martin Basti
On 10/01/2015 02:48 PM, Martin Basti wrote: On 10/01/2015 02:43 PM, Oleg Fayans wrote: Hi Martin, On 10/01/2015 11:18 AM, Martin Basti wrote: On 09/30/2015 01:24 PM, Martin Basti wrote: On 09/30/2015 12:19 PM, Oleg Fayans wrote: On 09/30/2015 11:46 AM, Petr Spacek wrote: On

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-02 Thread Milan Kubík
On 10/02/2015 04:11 PM, Martin Basti wrote: On 10/01/2015 02:48 PM, Martin Basti wrote: On 10/01/2015 02:43 PM, Oleg Fayans wrote: Hi Martin, On 10/01/2015 11:18 AM, Martin Basti wrote: On 09/30/2015 01:24 PM, Martin Basti wrote: On 09/30/2015 12:19 PM, Oleg Fayans wrote: On

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-02 Thread Martin Basti
On 10/02/2015 04:14 PM, Milan Kubík wrote: On 10/02/2015 04:11 PM, Martin Basti wrote: On 10/01/2015 02:48 PM, Martin Basti wrote: On 10/01/2015 02:43 PM, Oleg Fayans wrote: Hi Martin, On 10/01/2015 11:18 AM, Martin Basti wrote: On 09/30/2015 01:24 PM, Martin Basti wrote: On

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-01 Thread Martin Basti
On 10/01/2015 02:43 PM, Oleg Fayans wrote: Hi Martin, On 10/01/2015 11:18 AM, Martin Basti wrote: On 09/30/2015 01:24 PM, Martin Basti wrote: On 09/30/2015 12:19 PM, Oleg Fayans wrote: On 09/30/2015 11:46 AM, Petr Spacek wrote: On 29.9.2015 09:12, Oleg Fayans wrote: +def

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-01 Thread Oleg Fayans
Hi Martin, On 10/01/2015 11:18 AM, Martin Basti wrote: On 09/30/2015 01:24 PM, Martin Basti wrote: On 09/30/2015 12:19 PM, Oleg Fayans wrote: On 09/30/2015 11:46 AM, Petr Spacek wrote: On 29.9.2015 09:12, Oleg Fayans wrote: +def prepare_reverse_zone(host, ip): +zone =

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-10-01 Thread Martin Basti
On 09/30/2015 01:24 PM, Martin Basti wrote: On 09/30/2015 12:19 PM, Oleg Fayans wrote: On 09/30/2015 11:46 AM, Petr Spacek wrote: On 29.9.2015 09:12, Oleg Fayans wrote: +def prepare_reverse_zone(host, ip): +zone = get_reverse_zone_default(ip) +host.run_command(["ipa", +

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-30 Thread Oleg Fayans
Guys, could you please review this patch again? I'd like to have this fix in the new ipa-tests package for downstream team On 09/29/2015 09:12 AM, Oleg Fayans wrote: Hi all, On 09/23/2015 09:13 AM, Petr Spacek wrote: On 22.9.2015 10:42, Oleg Fayans wrote: +++

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-30 Thread Martin Basti
On 09/30/2015 12:19 PM, Oleg Fayans wrote: On 09/30/2015 11:46 AM, Petr Spacek wrote: On 29.9.2015 09:12, Oleg Fayans wrote: +def prepare_reverse_zone(host, ip): +zone = get_reverse_zone_default(ip) +host.run_command(["ipa", + "dnszone-add", +

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-30 Thread Petr Spacek
On 29.9.2015 09:12, Oleg Fayans wrote: > +def prepare_reverse_zone(host, ip): > +zone = get_reverse_zone_default(ip) > +host.run_command(["ipa", > + "dnszone-add", > + zone, > + "--name-from-ip=%s" % ip], raiseonerr=False)

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-30 Thread Oleg Fayans
On 09/30/2015 11:46 AM, Petr Spacek wrote: On 29.9.2015 09:12, Oleg Fayans wrote: +def prepare_reverse_zone(host, ip): +zone = get_reverse_zone_default(ip) +host.run_command(["ipa", + "dnszone-add", + zone, +

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-29 Thread Oleg Fayans
Hi all, On 09/23/2015 09:13 AM, Petr Spacek wrote: On 22.9.2015 10:42, Oleg Fayans wrote: +++ b/ipatests/test_integration/tasks.py @@ -58,6 +58,14 @@ def check_arguments_are(slice, instanceof): return wrapped return wrapper +def prepare_reverse_zone(host, ip): +nums =

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-23 Thread Petr Spacek
On 22.9.2015 10:42, Oleg Fayans wrote: > +++ b/ipatests/test_integration/tasks.py > @@ -58,6 +58,14 @@ def check_arguments_are(slice, instanceof): > return wrapped > return wrapper > > +def prepare_reverse_zone(host, ip): > +nums = ip.split('.')[:-1] > +zone =