Re: [Freeipa-devel] [PATCHES 297-299] Improvements for idviews xmlrpc tests

2015-01-22 Thread David Kupka
On 01/21/2015 04:51 PM, Tomas Babej wrote: Hi, this couple of patches adds coverage for the scenario in https://fedorahosted.org/freeipa/ticket/4839 , plus fixes issues that caused ipa-run-tests to skip this test file. Tomas ___ Freeipa-devel

[Freeipa-devel] [PATCH] 493 Print PublicError traceback when in debug mode

2015-01-22 Thread Martin Kosek
The framework only shows traceback for the internal/unknown errors, recognized PublicErrors are simply passed back to the FreeIPA clients. However, sometimes it would help to see a traceback of the PublicError to for example see exactly which line returns it.

[Freeipa-devel] [PATCH 300] Fix incorrect python shebang usage

2015-01-22 Thread Tomas Babej
Hi, attached patch fixes few python2 non-explicit shebangs that lurked into the codebase. TomasFrom ce5938cbf3df4b9ff6da08ebb114c342e763cc22 Mon Sep 17 00:00:00 2001 From: Tomas Babej tba...@redhat.com Date: Thu, 22 Jan 2015 12:47:13 +0100 Subject: [PATCH] ipapython: Fix incorrect python

Re: [Freeipa-devel] [PATCH] Use curl instead of wget

2015-01-22 Thread Colin Walters
On Thu, Jan 22, 2015, at 12:23 AM, Alexander Bokovoy wrote: In general, I'm not against trimming this dependency. However, please follow existing pattern by defining paths.BIN_CURL and using it instead of paths.BIN_WGET. FreeIPA client runs on Debian GNU/Linux-based platforms as well and we

Re: [Freeipa-devel] [PATCH] Use curl instead of wget

2015-01-22 Thread Alexander Bokovoy
On Thu, 22 Jan 2015, Colin Walters wrote: On Thu, Jan 22, 2015, at 12:23 AM, Alexander Bokovoy wrote: In general, I'm not against trimming this dependency. However, please follow existing pattern by defining paths.BIN_CURL and using it instead of paths.BIN_WGET. FreeIPA client runs on Debian

Re: [Freeipa-devel] [PATCH 003] Use 'remove-ds.pl' to remove DS instance during server uninstall

2015-01-22 Thread Martin Babinsky
On 01/22/2015 12:19 PM, Martin Kosek wrote: On 01/22/2015 11:58 AM, Martin Babinsky wrote: On 01/22/2015 11:04 AM, Martin Babinsky wrote: The attached patch addresses https://fedorahosted.org/freeipa/ticket/4487. Using 'remove-ds.pl' script from 389-ds during server/replica uninstallation

Re: [Freeipa-devel] [PATCH 003] Use 'remove-ds.pl' to remove DS instance during server uninstall

2015-01-22 Thread Martin Babinsky
On 01/22/2015 12:38 PM, Martin Babinsky wrote: On 01/22/2015 12:19 PM, Martin Kosek wrote: On 01/22/2015 11:58 AM, Martin Babinsky wrote: On 01/22/2015 11:04 AM, Martin Babinsky wrote: The attached patch addresses https://fedorahosted.org/freeipa/ticket/4487. Using 'remove-ds.pl' script from

[Freeipa-devel] [PATCH 003] Use 'remove-ds.pl' to remove DS instance during server uninstall

2015-01-22 Thread Martin Babinsky
The attached patch addresses https://fedorahosted.org/freeipa/ticket/4487. Using 'remove-ds.pl' script from 389-ds during server/replica uninstallation should allow for cleaner removal of DS instance with no leftovers (opened ports etc). Martin^3 From ba7bde3b7eaf80e26fed670fd8d863ad56b7e23a

Re: [Freeipa-devel] [PATCH 003] Use 'remove-ds.pl' to remove DS instance during server uninstall

2015-01-22 Thread Martin Babinsky
On 01/22/2015 11:04 AM, Martin Babinsky wrote: The attached patch addresses https://fedorahosted.org/freeipa/ticket/4487. Using 'remove-ds.pl' script from 389-ds during server/replica uninstallation should allow for cleaner removal of DS instance with no leftovers (opened ports etc). Martin^3

Re: [Freeipa-devel] [PATCH 003] Use 'remove-ds.pl' to remove DS instance during server uninstall

2015-01-22 Thread Martin Kosek
On 01/22/2015 11:58 AM, Martin Babinsky wrote: On 01/22/2015 11:04 AM, Martin Babinsky wrote: The attached patch addresses https://fedorahosted.org/freeipa/ticket/4487. Using 'remove-ds.pl' script from 389-ds during server/replica uninstallation should allow for cleaner removal of DS instance

Re: [Freeipa-devel] [PATCH 300] Fix incorrect python shebang usage

2015-01-22 Thread Martin Basti
On 22/01/15 12:52, Tomas Babej wrote: Hi, attached patch fixes few python2 non-explicit shebangs that lurked into the codebase. Tomas ACK -- Martin Basti ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

[Freeipa-devel] [PATCH 0189] Prevent install scripts to fail silently if timeout exceeded

2015-01-22 Thread Martin Basti
How to reproduce: 1. stop dirsrv 2. ipa-dns-install 3. insert password Patch attached -- Martin Basti From f20cf71a1230c144d7db9045ffb26ed8464a4bf8 Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Thu, 22 Jan 2015 17:09:22 +0100 Subject: [PATCH] Prevent install scripts fail

Re: [Freeipa-devel] [PATCH] Use curl instead of wget

2015-01-22 Thread Alexander Bokovoy
On Thu, 22 Jan 2015, Colin Walters wrote: On Thu, Jan 22, 2015, at 08:45 AM, Alexander Bokovoy wrote: We have abstraction layer to take care of different platforms on a wider scale than just this particular binary. We are gradually moving all code to use platform abstraction to allow other

Re: [Freeipa-devel] [PATCH] 493 Print PublicError traceback when in debug mode

2015-01-22 Thread Martin Basti
On 22/01/15 12:25, Martin Kosek wrote: The framework only shows traceback for the internal/unknown errors, recognized PublicErrors are simply passed back to the FreeIPA clients. However, sometimes it would help to see a traceback of the PublicError to for example see exactly which line returns

Re: [Freeipa-devel] [PATCH] Use curl instead of wget

2015-01-22 Thread Colin Walters
On Thu, Jan 22, 2015, at 08:45 AM, Alexander Bokovoy wrote: We have abstraction layer to take care of different platforms on a wider scale than just this particular binary. We are gradually moving all code to use platform abstraction to allow other platforms to be supported (like FreeBSD or

Re: [Freeipa-devel] [PATCH] 493 Print PublicError traceback when in debug mode

2015-01-22 Thread Martin Kosek
On 01/22/2015 04:09 PM, Martin Basti wrote: On 22/01/15 12:25, Martin Kosek wrote: The framework only shows traceback for the internal/unknown errors, recognized PublicErrors are simply passed back to the FreeIPA clients. However, sometimes it would help to see a traceback of the

Re: [Freeipa-devel] [PATCHES 0187, 0188] DNSSEC ipa-dnskeysyncd fixes

2015-01-22 Thread Jan Cholasta
Hi, Dne 21.1.2015 v 13:39 Martin Basti napsal(a): Patch 188 catch ldap exceptions to prevent false positive abrt reports Patch 187 fixes issues with removing root zone Patches attached. Patch 187: Is there a reason only SERVER_DOWN and CONNECT_ERROR are handled, instead of any LDAPError?

Re: [Freeipa-devel] [PATCH 0184] Always return absolute idnsname in dnszone commands

2015-01-22 Thread Jan Cholasta
Dne 20.1.2015 v 12:49 Martin Basti napsal(a): On 15/01/15 16:07, Jan Cholasta wrote: Dne 15.1.2015 v 15:39 Martin Basti napsal(a): On 15/01/15 15:07, Jan Cholasta wrote: Dne 15.1.2015 v 14:58 Martin Basti napsal(a): On 15/01/15 14:25, Jan Cholasta wrote: Hi, Dne 15.1.2015 v 13:27 Martin