Re: [Freeipa-devel] [PATCH 0126-0127] reset openldap client config to point to freshly promote replica

2016-01-14 Thread Martin Babinsky
On 01/13/2016 05:59 PM, Rob Crittenden wrote: Martin Babinsky wrote: fixes https://fedorahosted.org/freeipa/ticket/5584 In order to ensure consistent behavior with ipa-client-install, I opted to reuse the configure_openldap_conf() function and restoring the config from client sysrestore before

Re: [Freeipa-devel] [PATCHES] 0752-0757, 0759 Enable pylint --py3k

2016-01-14 Thread Petr Viktorin
On 01/12/2016 03:44 PM, Petr Viktorin wrote: > >>> Hello I tried --py3k option and it doesn't print any error, can we >>> enable that check by default to prevent python3 regressions? >>> >>> # ./make-lint --py3k >>> No config file found, using default configuration >> >> Squash in the other

Re: [Freeipa-devel] [PATCH 0011-0012][RFE] ipa-replica-manage: automatically clean dangling RUVs

2016-01-14 Thread Stanislav Laznicka
Please see the rebased patches attached. On 01/13/2016 02:01 PM, Martin Basti wrote: On 18.12.2015 12:46, Stanislav Laznicka wrote: Hi, Attached are the patches for auto-find and clean of dangling (cs)ruvs. Currently, the cleaning of an RUV waits for all replicas to be online, even on

Re: [Freeipa-devel] [PATCH 0401] Pylint enable unbalanced tuple unpacking check

2016-01-14 Thread Martin Babinsky
On 01/12/2016 01:09 PM, Martin Basti wrote: Patch attached. ACK. -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] Should we split up ipa-client?

2016-01-14 Thread Jan Cholasta
On 14.1.2016 10:48, Petr Viktorin wrote: On 01/14/2016 07:55 AM, Jan Cholasta wrote: Hi, On 13.1.2016 13:03, Martin Babinsky wrote: On 01/13/2016 11:34 AM, Petr Viktorin wrote: Hello, I'm planning to port the ipa-client to Python 3, and I'm likely to end up shaking out some dusty corners of

Re: [Freeipa-devel] [PATCH 0404] Pylint: enable unpacking-non-sequence check

2016-01-14 Thread Martin Babinsky
On 01/14/2016 11:02 AM, Martin Basti wrote: Patch attached. ACK. -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] Should we split up ipa-client?

2016-01-14 Thread Petr Viktorin
On 01/14/2016 07:55 AM, Jan Cholasta wrote: > Hi, > > On 13.1.2016 13:03, Martin Babinsky wrote: >> On 01/13/2016 11:34 AM, Petr Viktorin wrote: >>> Hello, >>> I'm planning to port the ipa-client to Python 3, and I'm likely to end >>> up shaking out some dusty corners of the codebase, rather than

Re: [Freeipa-devel] [PATCH 0125] IPA upgrade: move replication ACIs to the mapping tree entry

2016-01-14 Thread Martin Babinsky
On 01/13/2016 09:30 AM, Martin Babinsky wrote: On 01/13/2016 07:18 AM, Jan Cholasta wrote: On 12.1.2016 19:13, Martin Babinsky wrote: commit 6ea868e172738bdd6a8fae34e65126cdd134bbbe broke replica install and management on IPA servers upgraded from pre-4.3 version. The attached patch fixes

Re: [Freeipa-devel] [PATCH 562-563] Fix ipa-sam to use the getkeytab control instead of the setkeytab control

2016-01-14 Thread Martin Basti
On 14.01.2016 08:24, Alexander Bokovoy wrote: On Thu, 03 Dec 2015, Simo Sorce wrote: The first patch is preparatory and is needed in general now that we want top allow alias and use krbCanonicalName as the canonical name when multiple values are avilable in krbPrincipalName. The second patch

Re: [Freeipa-devel] [PATCH 562-563] Fix ipa-sam to use the getkeytab control instead of the setkeytab control

2016-01-14 Thread Alexander Bokovoy
On Thu, 14 Jan 2016, Martin Basti wrote: On 14.01.2016 08:24, Alexander Bokovoy wrote: On Thu, 03 Dec 2015, Simo Sorce wrote: The first patch is preparatory and is needed in general now that we want top allow alias and use krbCanonicalName as the canonical name when multiple values are

[Freeipa-devel] [PATCH 0404] Pylint: enable unpacking-non-sequence check

2016-01-14 Thread Martin Basti
Patch attached. From b13ef989eabc361d17fdc5d1d6137a460e650d6b Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Thu, 17 Dec 2015 15:51:32 +0100 Subject: [PATCH] Enable pylint unpacking-non-sequence check Enables check and marks occurences of runtime error in dcerpc.py as

Re: [Freeipa-devel] [PATCH 0402] Warn user about possibility to loss CA, KRA, DNSSEC master during uninstall

2016-01-14 Thread David Kupka
On 13/01/16 18:18, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5544 Patch attached. Thanks for the patch, works for me, ACK. -- David Kupka -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to

Re: [Freeipa-devel] [PATCH 0401] Pylint enable unbalanced tuple unpacking check

2016-01-14 Thread Martin Basti
On 14.01.2016 11:23, Martin Babinsky wrote: On 01/12/2016 01:09 PM, Martin Basti wrote: Patch attached. ACK. Pushed to master: 2320be18a34628fa6d05ffc42e695da5dd6dab6e -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH 0403][test] CI: fix regression in task.install_kra

2016-01-14 Thread Martin Basti
Regression caused by commit c4b9b295d8184694c50c0d56051e0273445c98ec Patch attached. One-liner rule applied. Pushed to: master: 26899c91afd36b39908a3a73363aff1787c9dc07 ipa-4-3: f3c1856d566cdcce0d411bc59efc63ddd42e5d07 From c47dde756f44c2810646bdf345433b1544d5adda Mon Sep 17 00:00:00 2001

Re: [Freeipa-devel] [PATCH 0404] Pylint: enable unpacking-non-sequence check

2016-01-14 Thread Martin Basti
On 14.01.2016 11:22, Martin Babinsky wrote: On 01/14/2016 11:02 AM, Martin Basti wrote: Patch attached. ACK. Pushed to master: 267bad10a81f101db8b645abf01b2cdd62c91775 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH 0126-0127] reset openldap client config to point to freshly promote replica

2016-01-14 Thread Martin Babinsky
On 01/13/2016 05:59 PM, Rob Crittenden wrote: Martin Babinsky wrote: fixes https://fedorahosted.org/freeipa/ticket/5584 In order to ensure consistent behavior with ipa-client-install, I opted to reuse the configure_openldap_conf() function and restoring the config from client sysrestore before

[Freeipa-devel] [TEST][PATCH 0019] A proper fix for reverse-zone creation in integration tests

2016-01-14 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 249db32dc5a7ce1e6df826c68f2919c07d953c45 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Thu, 14 Jan 2016 14:59:37 +0100 Subject: [PATCH] fixed an issue with master installation not creating reverse zone When

Re: [Freeipa-devel] [PATCH 0402] Warn user about possibility to loss CA, KRA, DNSSEC master during uninstall

2016-01-14 Thread Martin Basti
On 14.01.2016 14:18, David Kupka wrote: On 13/01/16 18:18, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5544 Patch attached. Thanks for the patch, works for me, ACK. Pushed to: master: 58c42ddac0964a8cce7c1e1faa7516da53f028ad ipa-4-3:

Re: [Freeipa-devel] [PATCH 0011-0012][RFE] ipa-replica-manage: automatically clean dangling RUVs

2016-01-14 Thread Rob Crittenden
Stanislav Laznicka wrote: > Please see the rebased patches attached. > > On 01/13/2016 02:01 PM, Martin Basti wrote: >> >> >> On 18.12.2015 12:46, Stanislav Laznicka wrote: >>> Hi, >>> >>> Attached are the patches for auto-find and clean of dangling >>> (cs)ruvs. Currently, the cleaning of an RUV

Re: [Freeipa-devel] [PATCH 0011-0012][RFE] ipa-replica-manage: automatically clean dangling RUVs

2016-01-14 Thread Petr Vobornik
On 01/14/2016 04:16 PM, Ludwig Krispenz wrote: On 01/14/2016 03:59 PM, Stanislav Laznicka wrote: On 01/14/2016 03:21 PM, Rob Crittenden wrote: Stanislav Laznicka wrote: Please see the rebased patches attached. On 01/13/2016 02:01 PM, Martin Basti wrote: On 18.12.2015 12:46, Stanislav

Re: [Freeipa-devel] [PATCH 0121] consider IPA master removed from topology when request for host TGT fails

2016-01-14 Thread Simo Sorce
On Wed, 2016-01-13 at 17:31 +0100, Martin Babinsky wrote: > On 01/13/2016 10:31 AM, Martin Babinsky wrote: > > On 01/07/2016 05:38 PM, Martin Babinsky wrote: > >> On 01/07/2016 05:37 PM, Martin Babinsky wrote: > >>> https://fedorahosted.org/freeipa/ticket/5584 > >>> > >> And the patch is here. >

Re: [Freeipa-devel] [PATCH 0011-0012][RFE] ipa-replica-manage: automatically clean dangling RUVs

2016-01-14 Thread Stanislav Laznicka
On 01/14/2016 03:21 PM, Rob Crittenden wrote: Stanislav Laznicka wrote: Please see the rebased patches attached. On 01/13/2016 02:01 PM, Martin Basti wrote: On 18.12.2015 12:46, Stanislav Laznicka wrote: Hi, Attached are the patches for auto-find and clean of dangling (cs)ruvs. Currently,

Re: [Freeipa-devel] [PATCH 0011-0012][RFE] ipa-replica-manage: automatically clean dangling RUVs

2016-01-14 Thread Ludwig Krispenz
On 01/14/2016 03:59 PM, Stanislav Laznicka wrote: On 01/14/2016 03:21 PM, Rob Crittenden wrote: Stanislav Laznicka wrote: Please see the rebased patches attached. On 01/13/2016 02:01 PM, Martin Basti wrote: On 18.12.2015 12:46, Stanislav Laznicka wrote: Hi, Attached are the patches for