[Freeipa-devel] [patch 0026] ipatests: replace the test-example.com domain in tests

Applies to ipa-4-3 and master. Reason in commit message. -- Milan Kubik From 912ecd6c48149696eb8ca3c60644e5b24af6e534 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Milan=20Kub=C3=ADk?= Date: Wed, 16 Dec 2015 16:27:34 +0100 Subject: [PATCH] ipatests: replace the

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

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 382: enables various pylint checks, we may reduce the list of check if needed.

Re: [Freeipa-devel] [PATCH] 0749 Package ipapython, ipalib, ipaplatform, ipatests for Python 3

On 12/07/2015 04:22 PM, Jan Cholasta wrote: > On 2.12.2015 13:38, Petr Viktorin wrote: >> On 12/01/2015 02:37 PM, Jan Cholasta wrote: [...] >> >> As for new provides, Fedora's Python packaging guidelines say: >> >> """ >> Using a fictional module named "example", the subpackage

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

https://www.redhat.com/archives/freeipa-users/2015-December/msg00203.html -- David Kupka From 114b4e2c1ffaa5c09dbfed54bb1f90cfa41f4678 Mon Sep 17 00:00:00 2001 From: David Kupka Date: Wed, 16 Dec 2015 12:43:13 + Subject: [PATCH 1/2] installer: Propagate option values from

[Freeipa-devel] [TESTS][PATCH 0008] Fix tests for (stage)user plugin

Hi, this patch fixes few Tracker methods for staged and 'normal' user, which were mistakenly modified by my patch 0006.3. Applies for ipa-4-2 branch only. Lenka From f632687a338bd17f1b143fa72e040f4111510998 Mon Sep 17 00:00:00 2001 From: Lenka Doudova Date: Wed, 16 Dec

[Freeipa-devel] [PATCH 0071] dns: Handle SERVFAIL in check if domain already exists

Hello, dns: Handle SERVFAIL in check if domain already exists. In cases where domain is already delegated to IPA prior installation we might get timeout or SERVFAIL. The answer depends on the recursive server we are using for the check. -- Petr^2 Spacek From

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 16.12.2015 11:15, Martin Kosek wrote: > On 12/16/2015 10:02 AM, Petr Spacek wrote: >> On 16.12.2015 09:53, Jan Cholasta wrote: >>> On 16.12.2015 09:45, Petr Spacek wrote: On 11.12.2015 15:50, Jan Cholasta wrote: > Hi, > > On 10.12.2015 18:04, Petr Spacek wrote: >> On

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 12/16/2015 12:01 PM, Petr Spacek wrote: > On 16.12.2015 11:15, Martin Kosek wrote: >> On 12/16/2015 10:02 AM, Petr Spacek wrote: >>> On 16.12.2015 09:53, Jan Cholasta wrote: On 16.12.2015 09:45, Petr Spacek wrote: > On 11.12.2015 15:50, Jan Cholasta wrote: >> Hi, >> >> On

Re: [Freeipa-devel] Fwd: [PUBLIC] Re: [IPAQE][REVIEW-REQUEST][TEST PLAN] Replica promotion

Hi There, Based on the URL for REPLICA PROMOTION Test plan (http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan), I have following observations / queries. Observations: --- 1. For "ipa-kra-install" and "ipa-ca-install" , expected result for step 1 should be FAILED,

Re: [Freeipa-devel] [PATCH 535] ipautil: remove unused import causing cyclic import in tests

ACK, the issue is gone On 12/15/2015 03:39 PM, Jan Cholasta wrote: > Hi, > > the attached patch fixes . > > Pushed to under the one-liner rule: > master: c265e8736e51d5b4fede94a414d83b3e0ada2853 > ipa-4-3: 2b28704f926d24e1562b481880b111d12d2020f2 >

Re: [Freeipa-devel] [PATCH 0071] dns: Handle SERVFAIL in check if domain already exists

On 16/12/15 14:23, Petr Spacek wrote: Hello, dns: Handle SERVFAIL in check if domain already exists. In cases where domain is already delegated to IPA prior installation we might get timeout or SERVFAIL. The answer depends on the recursive server we are using for the check. Works for me,

Re: [Freeipa-devel] [PATCH] ca-less tests updated - POC

On 06/11/15 14:04, Oleg Fayans wrote: Hi Jan, On 11/06/2015 09:01 AM, Jan Cholasta wrote: Actually it might be better to keep them, but fix them to expect ipa-server-certinstall to success. Done. Updated patch attached. Also in the patch 0013 I removed a trailing whitespace which caused lint

Re: [Freeipa-devel] certmonger everywhere

On 16.12.2015 15:29, Simo Sorce wrote: On Wed, 2015-12-16 at 07:17 +0100, Jan Cholasta wrote: On 15.12.2015 18:22, Simo Sorce wrote: On Tue, 2015-12-15 at 16:23 +0100, Martin Kosek wrote: On 12/15/2015 08:54 AM, Jan Cholasta wrote: Hi, recently I and David discussed the direction of

Re: [Freeipa-devel] certmonger everywhere

On Wed, 2015-12-16 at 07:17 +0100, Jan Cholasta wrote: > On 15.12.2015 18:22, Simo Sorce wrote: > > On Tue, 2015-12-15 at 16:23 +0100, Martin Kosek wrote: > >> On 12/15/2015 08:54 AM, Jan Cholasta wrote: > >>> Hi, > >>> > >>> recently I and David discussed the direction of installers with regard

Re: [Freeipa-devel] Fwd: [PUBLIC] Re: [IPAQE][REVIEW-REQUEST][TEST PLAN] Replica promotion

On 12/16/2015 03:17 PM, Nikhil Dehadrai wrote: Hi There, Based on the URL for REPLICA PROMOTION Test plan (http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan), I have following observations / queries. Observations: --- 1. For "ipa-kra-install" and "ipa-ca-install" ,

Re: [Freeipa-devel] certmonger everywhere

On Wed, 2015-12-16 at 15:41 +0100, Jan Cholasta wrote: > On 16.12.2015 15:29, Simo Sorce wrote: > > On Wed, 2015-12-16 at 07:17 +0100, Jan Cholasta wrote: > >> On 15.12.2015 18:22, Simo Sorce wrote: > >>> On Tue, 2015-12-15 at 16:23 +0100, Martin Kosek wrote: > On 12/15/2015 08:54 AM, Jan

Re: [Freeipa-devel] [TESTS][PATCH 0008] Fix tests for (stage)user plugin

On 12/16/2015 01:51 PM, Lenka Doudova wrote: Hi, this patch fixes few Tracker methods for staged and 'normal' user, which were mistakenly modified by my patch 0006.3. Applies for ipa-4-2 branch only. Lenka ACK. -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH 0071] dns: Handle SERVFAIL in check if domain already exists

On 12/16/2015 03:18 PM, David Kupka wrote: On 16/12/15 14:23, Petr Spacek wrote: Hello, dns: Handle SERVFAIL in check if domain already exists. In cases where domain is already delegated to IPA prior installation we might get timeout or SERVFAIL. The answer depends on the recursive server we

Re: [Freeipa-devel] [PATCH 016 - 017] First part of the replica promotion tests + testplan

On 15.12.2015 10:29, Oleg Fayans wrote: Hi Martin, The updated patches are attached. Patch 0017 includes all changes from patch 0018, so, if you approve this one, there would be no need to continue with the review of 0018. This one contains all changes related to you remarks from 0018 review.

Re: [Freeipa-devel] certmonger everywhere

On Wed, 16 Dec 2015, Simo Sorce wrote: On Wed, 2015-12-16 at 15:41 +0100, Jan Cholasta wrote: On 16.12.2015 15:29, Simo Sorce wrote: > On Wed, 2015-12-16 at 07:17 +0100, Jan Cholasta wrote: >> On 15.12.2015 18:22, Simo Sorce wrote: >>> On Tue, 2015-12-15 at 16:23 +0100, Martin Kosek wrote:

Re: [Freeipa-devel] [TESTS][PATCH 0008] Fix tests for (stage)user plugin

On 16.12.2015 16:03, Martin Babinsky wrote: On 12/16/2015 01:51 PM, Lenka Doudova wrote: Hi, this patch fixes few Tracker methods for staged and 'normal' user, which were mistakenly modified by my patch 0006.3. Applies for ipa-4-2 branch only. Lenka ACK. Pushed to ipa-4-2:

Re: [Freeipa-devel] certmonger everywhere

On Wed, Dec 16, 2015 at 11:11:42AM +0100, Martin Kosek wrote: > On 12/16/2015 09:17 AM, Jan Cholasta wrote: > > On 16.12.2015 08:54, Martin Kosek wrote: > ... > >>> 7. cert-request fetches the configuration for the specified sub-CA, > >>> or the > >>> default sub-CA if none was specified, from

Re: [Freeipa-devel] certmonger everywhere

On Wed, Dec 16, 2015 at 09:17:09AM +0100, Jan Cholasta wrote: > On 16.12.2015 08:54, Martin Kosek wrote: > >On 12/16/2015 08:09 AM, Jan Cholasta wrote: > >>On 16.12.2015 01:40, Fraser Tweedale wrote: > >>>On Tue, Dec 15, 2015 at 04:23:33PM +0100, Martin Kosek wrote: > On 12/15/2015 08:54 AM,

Re: [Freeipa-devel] [PATCH] ca-less tests updated - POC

Hi David, On 12/16/2015 03:35 PM, David Kupka wrote: > On 06/11/15 14:04, Oleg Fayans wrote: >> Hi Jan, >> >> On 11/06/2015 09:01 AM, Jan Cholasta wrote: >>> Actually it might be better to keep them, but fix them to expect >>> ipa-server-certinstall to success. >> >> Done. Updated patch attached.

Re: [Freeipa-devel] [PATCH 0364, 0367] ipa-kra-install: allow first KRA to be installed on replica

On 07.12.2015 14:31, Martin Babinsky wrote: On 12/02/2015 05:24 PM, Martin Basti wrote: On 02.12.2015 14:52, Martin Babinsky wrote: On 11/30/2015 06:29 PM, Martin Basti wrote: On 30.11.2015 14:16, Martin Babinsky wrote: On 11/27/2015 05:02 PM, Martin Basti wrote:

Re: [Freeipa-devel] [PATCH 016 - 017] First part of the replica promotion tests + testplan

On 16.12.2015 15:39, Martin Basti wrote: On 15.12.2015 10:29, Oleg Fayans wrote: Hi Martin, The updated patches are attached. Patch 0017 includes all changes from patch 0018, so, if you approve this one, there would be no need to continue with the review of 0018. This one contains all

Re: [Freeipa-devel] limiting SyncRepl's scope

On Wed, Dec 16, 2015 at 09:26:11AM +0100, Sumit Bose wrote: > On Wed, Dec 16, 2015 at 08:49:04AM +0100, Petr Spacek wrote: > > On 15.12.2015 19:10, Christian Heimes wrote: > > > Hi, > > > > > > in ticket https://fedorahosted.org/freeipa/ticket/5538 Ludwig has > > > suggested to exclude Dogtag's

Re: [Freeipa-devel] Testing FreeIPA 4.3 for GA

On (15/12/15 08:32), Lukas Slebodnik wrote: >On (15/12/15 01:05), Petr Vobornik wrote: >>Blocking patches for FreeIPA 4.3 were pushed, ipa-4-3 branch was created. >>Master branch is ready for 4.4 development. >> >>A build is available for testing in my pvoborni/freeipa-4-3 COPR repo [1] >>until

Re: [Freeipa-devel] limiting SyncRepl's scope

On Wed, Dec 16, 2015 at 08:49:04AM +0100, Petr Spacek wrote: > On 15.12.2015 19:10, Christian Heimes wrote: > > Hi, > > > > in ticket https://fedorahosted.org/freeipa/ticket/5538 Ludwig has > > suggested to exclude Dogtag's o=ipaca tree from the changelog. Sometimes > > vault-archive fails

Re: [Freeipa-devel] limiting SyncRepl's scope

On 12/16/2015 08:49 AM, Petr Spacek wrote: On 15.12.2015 19:10, Christian Heimes wrote: Hi, in ticket https://fedorahosted.org/freeipa/ticket/5538 Ludwig has suggested to exclude Dogtag's o=ipaca tree from the changelog. Sometimes vault-archive fails because of a failed write to the Retro

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 11.12.2015 15:50, Jan Cholasta wrote: > Hi, > > On 10.12.2015 18:04, Petr Spacek wrote: >> On 9.12.2015 15:30, Petr Spacek wrote: >>> Hello, >>> >>> this patch automates some of sanity checks proposed by Petr Vobornik. >>> >>> 'make review' should be used in root of clean Git tree which has

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 16.12.2015 09:45, Petr Spacek wrote: On 11.12.2015 15:50, Jan Cholasta wrote: Hi, On 10.12.2015 18:04, Petr Spacek wrote: On 9.12.2015 15:30, Petr Spacek wrote: Hello, this patch automates some of sanity checks proposed by Petr Vobornik. 'make review' should be used in root of clean Git

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 16.12.2015 09:53, Jan Cholasta wrote: > On 16.12.2015 09:45, Petr Spacek wrote: >> On 11.12.2015 15:50, Jan Cholasta wrote: >>> Hi, >>> >>> On 10.12.2015 18:04, Petr Spacek wrote: On 9.12.2015 15:30, Petr Spacek wrote: > Hello, > > this patch automates some of sanity checks

Re: [Freeipa-devel] certmonger everywhere

On 16.12.2015 08:54, Martin Kosek wrote: On 12/16/2015 08:09 AM, Jan Cholasta wrote: On 16.12.2015 01:40, Fraser Tweedale wrote: On Tue, Dec 15, 2015 at 04:23:33PM +0100, Martin Kosek wrote: On 12/15/2015 08:54 AM, Jan Cholasta wrote: Hi, recently I and David discussed the direction of

Re: [Freeipa-devel] certmonger everywhere

On 12/16/2015 09:17 AM, Jan Cholasta wrote: > On 16.12.2015 08:54, Martin Kosek wrote: ... >>> 7. cert-request fetches the configuration for the specified sub-CA, >>> or the >>> default sub-CA if none was specified, from LDAP >>> >>> 8. cert-request forwards the request to the certmonger CA

Re: [Freeipa-devel] [PATCH 0069] Add 'review' target for make

On 12/16/2015 10:02 AM, Petr Spacek wrote: > On 16.12.2015 09:53, Jan Cholasta wrote: >> On 16.12.2015 09:45, Petr Spacek wrote: >>> On 11.12.2015 15:50, Jan Cholasta wrote: Hi, On 10.12.2015 18:04, Petr Spacek wrote: > On 9.12.2015 15:30, Petr Spacek wrote: >> Hello, >>