Re: [Freeipa-devel] [PATCH-0019][Tests]Enabled --auto-reverse by default for master installation

2015-12-21 Thread Oleg Fayans
Hi Martin, On 12/19/2015 08:02 PM, Martin Basti wrote: > > > On 18.12.2015 09:35, Oleg Fayans wrote: >> Hi Petr, >> >> On 12/17/2015 08:19 PM, Petr Spacek wrote: >>> On 17.12.2015 14:27, Oleg Fayans wrote: Commit message was updated. Thanks! On 12/17/2015 02:05 PM, Lukas Slebodnik

Re: [Freeipa-devel] [PATCH 0101] raise more descriptive Backend connection-related exceptions

2015-12-21 Thread Martin Basti
On 20.11.2015 14:51, Martin Babinsky wrote: Fixes https://fedorahosted.org/freeipa/ticket/5473 Patch is for master only, I will rebase it for 4-2 if we decide that it can land there as well. ACK Pushed to master: cd5fa389450d15d5bc696131f73d062000d3558f Pushed to ipa-4-3: f1244a1058aba5

Re: [Freeipa-devel] [PATCH 0114] harden domain level 1 topology connectivity checks

2015-12-21 Thread Martin Basti
On 14.12.2015 09:59, Martin Babinsky wrote: On 12/08/2015 05:35 PM, Martin Babinsky wrote: A sort of auxilliary patch which makes topology checks more resistant to https://fedorahosted.org/freeipa/ticket/5526 If required I will open a separate ticket for it though. Bump for review. ACK

Re: [Freeipa-devel] [PATCH 0368] FIX DNS tests

2015-12-21 Thread Petr Spacek
On 2.12.2015 17:34, Martin Basti wrote: > Test needs to be updated due deprecation of dns-resolve command. > > Patch attached ACK -- Petr^2 Spacek -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http:/

Re: [Freeipa-devel] [PATCH 0368] FIX DNS tests

2015-12-21 Thread Martin Basti
On 21.12.2015 14:01, Petr Spacek wrote: On 2.12.2015 17:34, Martin Basti wrote: Test needs to be updated due deprecation of dns-resolve command. Patch attached ACK Pushed to: master: ee51ad0f25d4de2c20c3c666156781e4ff678863 ipa-4-3: 81e40af95de3b4975c006d0936dbd6e65200c5c0 -- Manage your

Re: [Freeipa-devel] [patch 0027] ipatests: Roll back the forwarder config after a test case

2015-12-21 Thread Martin Basti
On 17.12.2015 20:16, Petr Spacek wrote: On 17.12.2015 14:35, Milan KubĂ­k wrote: Patch attached. -- Milan Kubik freeipa-mkubik-0027-ipatests-Roll-back-the-forwarder-config-after-a-test.patch From 66db35cf2a315f0a58e002b06f0fcc5bd915a5c5 Mon Sep 17 00:00:00 2001 From: Milan Kubik Date: Th

Re: [Freeipa-devel] [PATCH-0019][Tests]Enabled --auto-reverse by default for master installation

2015-12-21 Thread Martin Basti
On 21.12.2015 11:25, Oleg Fayans wrote: Hi Martin, On 12/19/2015 08:02 PM, Martin Basti wrote: On 18.12.2015 09:35, Oleg Fayans wrote: Hi Petr, On 12/17/2015 08:19 PM, Petr Spacek wrote: On 17.12.2015 14:27, Oleg Fayans wrote: Commit message was updated. Thanks! On 12/17/2015 02:05 PM,

Re: [Freeipa-devel] [PATCH 0020][Tests] Added --realm parameter to master installation procedure in integration tests

2015-12-21 Thread Martin Basti
On 18.12.2015 09:37, Oleg Fayans wrote: Sorry, my bad :) On 12/17/2015 04:44 PM, Martin Babinsky wrote: On 12/17/2015 04:28 PM, Oleg Fayans wrote: Oleg, you sent a vim swap file instead of the patch. ACK Pushed to: master: b12ba14e3d07b5f2d8dd50245407da029ae7dc54 ipa-4-3: c47a5df72

[Freeipa-devel] [PATCH 0072-0081] DNSSEC: ipa-dnskeysyncd: call ods-signer ldap-cleanup on zone removaDNSSEC: ipa-ods-exporter: add ldap-cleanup commanDNSSEC: ipa-dnskeysyncd: Skip zones with old DNSS

2015-12-21 Thread Petr Spacek
Hello, this patch set fixes key rotation in DNSSEC. You can use attached template files for OpenDNSSEC config to shorten time intervals between key rotations. Please let me know if you have any questions, I'm all ears! -- Petr^2 Spacek From ef4c0e6ef2a5238ef2afe54e2f912480385abadd Mon Sep 17 0

Re: [Freeipa-devel] [PATCH 0072-0081] DNSSEC: ipa-dnskeysyncd: call ods-signer ldap-cleanup on zone removaDNSSEC: ipa-ods-exporter: add ldap-cleanup commanDNSSEC: ipa-dnskeysyncd: Skip zones with old

2015-12-21 Thread Martin Basti
On 21.12.2015 15:33, Petr Spacek wrote: Hello, this patch set fixes key rotation in DNSSEC. You can use attached template files for OpenDNSSEC config to shorten time intervals between key rotations. Please let me know if you have any questions, I'm all ears! Please fix whitespace error: A

Re: [Freeipa-devel] [PATCH-0019][Tests]Enabled --auto-reverse by default for master installation

2015-12-21 Thread Martin Basti
On 21.12.2015 15:02, Martin Basti wrote: On 21.12.2015 11:25, Oleg Fayans wrote: Hi Martin, On 12/19/2015 08:02 PM, Martin Basti wrote: On 18.12.2015 09:35, Oleg Fayans wrote: Hi Petr, On 12/17/2015 08:19 PM, Petr Spacek wrote: On 17.12.2015 14:27, Oleg Fayans wrote: Commit message wa

Re: [Freeipa-devel] [PATCH 0075-0076] Fix installer regression

2015-12-21 Thread Martin Basti
On 18.12.2015 15:13, Jan Cholasta wrote: On 18.12.2015 14:11, David Kupka wrote: On 18/12/15 13:57, David Kupka wrote: On 17/12/15 13:44, Jan Cholasta wrote: On 17.12.2015 13:26, David Kupka wrote: On 17/12/15 12:14, Petr Vobornik wrote: On 12/16/2015 02:31 PM, David Kupka wrote: https://w

Re: [Freeipa-devel] [PATCH 0072-0081] DNSSEC: ipa-dnskeysyncd: call ods-signer ldap-cleanup on zone removaDNSSEC: ipa-ods-exporter: add ldap-cleanup commanDNSSEC: ipa-dnskeysyncd: Skip zones with old

2015-12-21 Thread Martin Basti
On 21.12.2015 15:45, Martin Basti wrote: On 21.12.2015 15:33, Petr Spacek wrote: Hello, this patch set fixes key rotation in DNSSEC. You can use attached template files for OpenDNSSEC config to shorten time intervals between key rotations. Please let me know if you have any questions, I

Re: [Freeipa-devel] [PATCH 0379 - 0393] Winter code cleanup

2015-12-21 Thread Jan Cholasta
On 17.12.2015 14:18, Martin Basti wrote: On 17.12.2015 07:54, Jan Cholasta wrote: Hi, On 17.12.2015 02:11, Martin Basti wrote: Hello, following patches cleanup the code and add checks to pylint to avoid the mess again Patches: 379-381: the most important patches, they cleanup imports Patch