[Freeipa-devel] [TEST][Patch 0022] small refactoring in integration tests due to BZ 1303095

2016-01-29 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From b866ff5336f742e170895b575df7a09419c2d731 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Fri, 29 Jan 2016 16:04:17 +0100 Subject: [PATCH] Moved NM configuration calls to the IntegrationTest base class Reconfiguring and restarting of

Re: [Freeipa-devel] [TEST][Patch 0021] Fixed recent replica installation issues in the lab

2016-02-01 Thread Oleg Fayans
Hi Petr, Please find the new version of the patch attached. Comments are inline On 01/29/2016 11:58 AM, Petr Spacek wrote: > On 27.1.2016 11:16, Oleg Fayans wrote: >> Sorry, trailing whitespace detected. This version passes lint >> >> On 01/27/2016 09:23 AM, Oleg

[Freeipa-devel] [TEST][Patch 0023] Updated connect/disconnect replica to work on both domain levels

2016-02-01 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 6842d2c1d068853f16f42df893112cb1e8d716ff Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Mon, 1 Feb 2016 11:34:11 +0100 Subject: [PATCH] Updated connect/disconnect replica to work with both domainlevels --- ipatests/test_integration

Re: [Freeipa-devel] [TEST][Patch 0022] small refactoring in integration tests due to BZ 1303095

2016-02-03 Thread Oleg Fayans
Guys, can anyone take a look at this? On 01/29/2016 04:09 PM, Oleg Fayans wrote: > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to Free

Re: [Freeipa-devel] [TEST][Patch 0022] small refactoring in integration tests due to BZ 1303095

2016-02-03 Thread Oleg Fayans
Hi Petr, On 02/03/2016 02:19 PM, Petr Spacek wrote: > On 3.2.2016 10:22, Oleg Fayans wrote: >> Guys, can anyone take a look at this? > > The commit message does not explain why you are setting search path. Fixed. > > I have to say that I do not like touching resolv.conf,

Re: [Freeipa-devel] [TEST][Patch 0021] Fixed recent replica installation issues in the lab

2016-02-04 Thread Oleg Fayans
Hi Petr, An updated patch is attached. Please see my comment inline. On 02/01/2016 12:47 PM, Petr Spacek wrote: > On 1.2.2016 11:52, Oleg Fayans wrote: >> Hi Petr, >> >> Please find the new version of the patch attached. Comments are inline >> >> On 01/29/2016 11

Re: [Freeipa-devel] [TEST][Patch 0021] Fixed recent replica installation issues in the lab

2016-02-04 Thread Oleg Fayans
Hi, On 02/04/2016 02:07 PM, Martin Basti wrote: > > > On 04.02.2016 13:49, Oleg Fayans wrote: >> Hi Petr, >> >> An updated patch is attached. Please see my comment inline. >> >> On 02/01/2016 12:47 PM, Petr Spacek wrote: >>> On 1.2.2016 11:52, O

Re: [Freeipa-devel] [TEST][Patch 0022] small refactoring in integration tests due to BZ 1303095

2016-02-19 Thread Oleg Fayans
Hi Milan, On 02/12/2016 04:03 PM, Milan Kubík wrote: > On 02/04/2016 08:49 AM, Oleg Fayans wrote: >> Hi Petr, >> >> On 02/03/2016 02:19 PM, Petr Spacek wrote: >>> On 3.2.2016 10:22, Oleg Fayans wrote: >>>> Guys, can anyone take a look at this? >>&

Re: [Freeipa-devel] [PATCH 0421] Make PTR records check optional for IPA installation

2016-02-26 Thread Oleg Fayans
0 and with replica-install on domlevel1. ACK -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- 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] [TEST][Patch 0022] small refactoring in integration tests due to BZ 1303095

2016-02-26 Thread Oleg Fayans
Hey guys, could anyone review this please? On 02/19/2016 02:11 PM, Oleg Fayans wrote: > Hi Milan, > > On 02/12/2016 04:03 PM, Milan Kubík wrote: >> On 02/04/2016 08:49 AM, Oleg Fayans wrote: >>> Hi Petr, >>> >>> On 02/03/2016 02:19 PM, Petr Spacek wrote:

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

2016-02-29 Thread Oleg Fayans
Hi all, Finally the tests pass. The patch 0024 applies on top of patch 0022 (please, consider reviewing it also). Besides, the whole functionality depends on Martin's patch N 0421 All patches pass pylint. On 12/19/2015 11:56 PM, Martin Basti wrote: > > > On 17.12.2015 10:0

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

2016-03-01 Thread Oleg Fayans
Hi Martin, On 03/01/2016 07:04 PM, Martin Basti wrote: > > > On 01.03.2016 14:56, Martin Basti wrote: >> >> >> >> On 01.03.2016 12:37, Martin Basti wrote: >>> >>> >>> On 01.03.2016 12:32, Martin Basti wrote: >>

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

2016-03-02 Thread Oleg Fayans
Hi Martin, I've made the requested changes. The full set of necessary patches is attached. On 03/02/2016 10:05 AM, Martin Basti wrote: > > > On 02.03.2016 00:12, Oleg Fayans wrote: >> Hi Martin, >> >> On 03/01/2016 07:04 PM, Martin Basti wrote: >>> &g

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

2016-03-04 Thread Oleg Fayans
Hi, Martin. An updated version of the patch is attached. Please see my comments below On 03/04/2016 08:39 AM, Martin Basti wrote: > > > On 04.03.2016 08:37, Martin Basti wrote: >> >> >> On 03.03.2016 18:38, Martin Basti wrote: >>> >>> >>>

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

2016-03-08 Thread Oleg Fayans
s applied >> correctly? >> >> bash-4.3$ ipa-run-tests test_integration/test_replica_promotion.py --pdb >> ======== >> >> test session starts >> ===

[Freeipa-devel] [TEST][Patch-0027] Fixed test failure during in-tree session, ticket N 5736

2016-03-16 Thread Oleg Fayans
e put it to tasks.py -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 2f3d460c0b29d65491a47528802d067ccb9757d8 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Wed, 16 Mar 2016 09:42:58 +0100 Subject: [PATCH] rewrote a misprocessed teardown_method method as a custom decorator

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

2016-03-19 Thread Oleg Fayans
Hi Lukas, On 03/17/2016 11:28 AM, Lukas Slebodnik wrote: > On (10/03/16 23:09), Oleg Fayans wrote: >> Hi Martin, >> >> >> >> On 03/08/2016 08:18 PM, Martin Basti wrote: >>> >>> >>> On 08.03.2016 18:24, Martin Basti wrote: >>>

Re: [Freeipa-devel] [TEST][Patch-0027] Fixed test failure during in-tree session, ticket N 5736

2016-03-19 Thread Oleg Fayans
Hi Martin On 03/16/2016 02:39 PM, Martin Basti wrote: > > > On 16.03.2016 10:59, Oleg Fayans wrote: >> With this patch applied integration tests pass and in-tree tests are >> gracefully skipped. >> >> @mkubik, It is not possible to put the decorator to util.py

Re: [Freeipa-devel] [TEST][Patch-0027] Fixed test failure during in-tree session, ticket N 5736

2016-03-21 Thread Oleg Fayans
Hi Martin, On 03/16/2016 03:35 PM, Martin Basti wrote: > > > On 16.03.2016 15:13, Martin Basti wrote: >> >> >> On 16.03.2016 14:59, Oleg Fayans wrote: >>> Hi Martin >>> >>> On 03/16/2016 02:39 PM, Martin Basti wrote: >>>>

[Freeipa-devel] [TEST][Patch-0030]Next part of replica promotion tests

2016-03-21 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From a4f0bdbe3a8646070119ccb668ad0bfb8cfb269b Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Mon, 21 Mar 2016 13:27:19 +0100 Subject: [PATCH] Added 5 more tests to Replica Promotion testsuite https://fedorahosted.org/freeipa/ticket/5723

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

2016-03-21 Thread Oleg Fayans
16:00), Oleg Fayans wrote: >> Hi Lukas, >> >> On 03/17/2016 11:28 AM, Lukas Slebodnik wrote: >>> On (10/03/16 23:09), Oleg Fayans wrote: >>>> Hi Martin, >>>> >>>> >>>> >>>> On 03/08/2016 08:18 PM, Martin Basti wro

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

2016-03-22 Thread Oleg Fayans
On 03/22/2016 02:06 PM, Martin Basti wrote: > > > On 21.03.2016 15:54, Oleg Fayans wrote: >> Hi Lukas, Martin, >> >> Looks I've implemented the approach proposed by Martin. The issue seems >> to have gone (see the external_ca_out for external_ca test out

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

2016-03-22 Thread Oleg Fayans
Hi Martin, As per discussion, reverted 0025 patch and added try-catch in prepare_host method to make replica promotion tests pass On 03/22/2016 03:03 PM, Martin Basti wrote: > > > On 22.03.2016 14:30, Oleg Fayans wrote: >> >> On 03/22/2016 02:06 PM, Martin Basti wrote: >

[Freeipa-devel] Instructions to build ipa under RHEL

2016-03-29 Thread Oleg Fayans
lable. No package python3-devel available. This leads to quite expected failure to build srpms. -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: ht

Re: [Freeipa-devel] Instructions to build ipa under RHEL

2016-03-29 Thread Oleg Fayans
OK, I enabled the following repo: http://cosmos.lab.eng.pnq.redhat.com/idmqe-extras/rhel/7Server/x86_64/ and that gave me: pylint python-polib python-pytest-multihost python-pytest-sourceorder But the rest is still unavailable On 03/29/2016 10:16 AM, Oleg Fayans wrote: > Hi team, > >

Re: [Freeipa-devel] Instructions to build ipa under RHEL

2016-03-29 Thread Oleg Fayans
Petr, Martin, thanks guys! On 03/29/2016 10:37 AM, Petr Vobornik wrote: > On 03/29/2016 10:16 AM, Oleg Fayans wrote: >> Hi team, >> >> Is there any kind of $subj available? Like, which repos to enable, etc. >> I'm raising the topic because I was unable to i

[Freeipa-devel] [TEST][patch-0032] Added a kdestroy call to clean ccache

2016-03-30 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 8e5c99a809a9fff1ec59ec750e4f31d6be837a44 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Wed, 30 Mar 2016 12:34:12 +0200 Subject: [PATCH] Added a kdestroy call to clean ccache at master/client uninstallation https://fedorahosted.org

Re: [Freeipa-devel] [TEST][patch-0032] Added a kdestroy call to clean ccache

2016-03-30 Thread Oleg Fayans
Hi, The updated patch is included On 03/30/2016 08:50 PM, Robbie Harwood wrote: > Rob Crittenden writes: > >> Would it be more robust to call kdestroy -A or is that just overkill in >> this case? > > I believe it would be superior to call `kdestroy -A`, yes. >

[Freeipa-devel] [TEST][patch-0033] Added assertion errors to topology tests, track N 5772

2016-03-31 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From f1821a41498f7829df4d1bdffd338cb70b413667 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Thu, 31 Mar 2016 11:58:26 +0200 Subject: [PATCH] Added assertion error messages in topology tests Added a global sleep interval used to wait for

Re: [Freeipa-devel] [TEST][patch-0033] Added assertion errors to topology tests, track N 5772

2016-03-31 Thread Oleg Fayans
Please, disregard it for a while, it does not pass lint. On 03/31/2016 12:05 PM, Oleg Fayans wrote: > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Cont

Re: [Freeipa-devel] [TEST][patch-0033] Added assertion errors to topology tests, track N 5772

2016-03-31 Thread Oleg Fayans
Hi Martin, Thanks for the review. The updated patch(es) are included Testrun output can be found here: http://fpaste.org/347800/59421745/ On 03/31/2016 01:10 PM, Martin Basti wrote: > > > On 31.03.2016 12:07, Oleg Fayans wrote: >> Please, disregard it for a while, it doe

Re: [Freeipa-devel] [TEST][Patch-0030]Next part of replica promotion tests

2016-04-01 Thread Oleg Fayans
Hi Martin, Thanks for the review! The new version is attached On 03/24/2016 06:08 PM, Martin Babinsky wrote: > On 03/21/2016 01:51 PM, Oleg Fayans wrote: >> >> >> > Hi Oleg, > > I have a few comments: > > 1.) > please make the commit message more clea

Re: [Freeipa-devel] [TEST][patch-0033] Added assertion errors to topology tests, track N 5772

2016-04-06 Thread Oleg Fayans
Hi Martin, The updated patches are attached On 04/04/2016 06:46 PM, Martin Babinsky wrote: > On 03/31/2016 05:15 PM, Oleg Fayans wrote: >> Hi Martin, >> >> Thanks for the review. The updated patch(es) are included >> >> Testrun output can be found here: >&g

[Freeipa-devel] [TEST][patch-0035] Test replica installed under domain level 0 is functional after domain upgrade

2016-04-07 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From df2d57707875d37783c51521e3c5562642652a42 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Thu, 7 Apr 2016 11:49:44 +0200 Subject: [PATCH] Add test if replica is working after domain upgrade Corresponds to the testcase described in http

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

2016-04-08 Thread Oleg Fayans
ep 17 00:00:00 2001 From: Oleg Fayans Date: Tue, 8 Dec 2015 10:49:18 +0100 Subject: [PATCH] Updated the script creating test certificate chains https://fedorahosted.org/freeipa/ticket/4589 --- .../test_integration/scripts/caless-create-pki | 29 ++ 1 file changed, 18 inser

[Freeipa-devel] [TEST][patch-0035] Fixed failing legacy client tests

2016-04-12 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 795c84c8e5be1d77c50328c8d1425af5fec4e8cd Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Tue, 12 Apr 2016 11:18:09 +0200 Subject: [PATCH] Fixed a failure in legacy_client tests https://fedorahosted.org/freeipa/ticket/5800 --- ipatests

Re: [Freeipa-devel] [TEST][patch-0035] Test replica installed under domain level 0 is functional after domain upgrade

2016-04-13 Thread Oleg Fayans
Hi Martin, I've updated the patch with regard to your review. Thank you! On 04/12/2016 03:35 PM, Martin Babinsky wrote: > On 04/07/2016 12:35 PM, Oleg Fayans wrote: >> >> >> > Hi Oleg, > > since this is a part of replica promotion test suite please add the

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

2016-04-13 Thread Oleg Fayans
Please. disregard my previous patch-0014: it doesn't pass pylint. The newer one does. On 04/08/2016 04:46 PM, Oleg Fayans wrote: > Hi all, > > It's been a while, and now the patch seems to be stable. It does hit one > known issue with replica installation occationally [1],

Re: [Freeipa-devel] [TEST][patch-0035] Test replica installed under domain level 0 is functional after domain upgrade

2016-04-14 Thread Oleg Fayans
The new version disables a falsely failing pylint check for ldap_connect method in a master host instance. On 04/13/2016 10:30 AM, Oleg Fayans wrote: > Hi Martin, > > I've updated the patch with regard to your review. Thank you! > > On 04/12/2016 03:35 PM, Martin Babinsk

Re: [Freeipa-devel] [TEST][patch-0035] Test replica installed under domain level 0 is functional after domain upgrade

2016-04-15 Thread Oleg Fayans
Hi Martin & Martin, The updated patches are included. both apply cleanly and pass pylint On 04/14/2016 02:17 PM, Martin Basti wrote: > > > On 14.04.2016 14:04, Oleg Fayans wrote: >> The new version disables a falsely failing pylint check for ldap_connect >> method

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

2016-04-19 Thread Oleg Fayans
OK, that one, though passing lint, did not actually work. I gave up my attempts to define method decorators inside the class. Now it passes lint AND works :) On 04/13/2016 12:24 PM, Oleg Fayans wrote: > Please. disregard my previous patch-0014: it doesn't pass pylint. The > new

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

2016-04-20 Thread Oleg Fayans
Hi David, On 04/20/2016 11:55 AM, David Kupka wrote: > On 20/04/16 10:35, David Kupka wrote: >> On 19/04/16 11:13, Oleg Fayans wrote: >>> OK, that one, though passing lint, did not actually work. I gave up my >>> attempts to define method decorators inside the class.

[Freeipa-devel] [DESIGN REVIEW] V4/Certs_in_ID_overrides

2016-05-05 Thread Oleg Fayans
Hi, The document looks fine. Would be nice if it had some link on a HOWTO page about generation of a user certificate to use for AD-originated users. Apart from that - everything is pretty clear -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa

[Freeipa-devel] [DESIGN REVIEW] V4/Server_Roles

2016-05-05 Thread Oleg Fayans
The document is perfect. No remarks from QE side: ready for testplan design -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org

[Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-06 Thread Oleg Fayans
test_integration/test_dnssec.py = 8 passed in 5561.48 seconds == -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From c716ef162166758795f30f9ee79124ad7cd0f752 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Fri, 6 May 2016 08:56:46

Re: [Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-06 Thread Oleg Fayans
On 05/06/2016 09:48 AM, Martin Basti wrote: > > > On 06.05.2016 09:36, Oleg Fayans wrote: >> Tests are finally stable: >> >> = test session starts >> == >> platform linux2 -- Python 2.7.11 -- py-1.4.

Re: [Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-06 Thread Oleg Fayans
On 05/06/2016 11:42 AM, Martin Basti wrote: > > > On 06.05.2016 11:14, Oleg Fayans wrote: >> >> On 05/06/2016 09:48 AM, Martin Basti wrote: >>> >>> On 06.05.2016 09:36, Oleg Fayans wrote: >>>> Tests are finally stable: >>

Re: [Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-06 Thread Oleg Fayans
On 05/06/2016 12:08 PM, Martin Babinsky wrote: > On 05/06/2016 11:14 AM, Oleg Fayans wrote: >> >> >> On 05/06/2016 09:48 AM, Martin Basti wrote: >>> >>> >>> On 06.05.2016 09:36, Oleg Fayans wrote: >>>> Tests are finally stable:

Re: [Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-06 Thread Oleg Fayans
On 05/06/2016 03:25 PM, Petr Spacek wrote: > On 6.5.2016 15:03, Oleg Fayans wrote: >> >> >> On 05/06/2016 12:08 PM, Martin Babinsky wrote: >>> On 05/06/2016 11:14 AM, Oleg Fayans wrote: >>>> >>>> >>>> On 05/06/2016 09:48 AM, Mart

[Freeipa-devel] [TBD] Automated tests, regressions and workarounds

2016-05-09 Thread Oleg Fayans
eems that most of our team puts the principle "No workarounds, we need to fix the bug no matter what" above any common sense. [1] https://fedorahosted.org/freeipa/ticket/5348 -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel mailing

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

2016-05-10 Thread Oleg Fayans
The full set of necessary patches is attached On 04/20/2016 10:35 AM, David Kupka wrote: > On 19/04/16 11:13, Oleg Fayans wrote: >> OK, that one, though passing lint, did not actually work. I gave up my >> attempts to define method decorators inside the class. Now it passes &g

Re: [Freeipa-devel] [TEST][patch-0037]Fixes of dnssec tests

2016-05-11 Thread Oleg Fayans
PM, Martin Basti wrote: > > > On 06.05.2016 14:58, Oleg Fayans wrote: >> >> On 05/06/2016 11:42 AM, Martin Basti wrote: >>> >>> On 06.05.2016 11:14, Oleg Fayans wrote: >>>> On 05/06/2016 09:48 AM, Martin Basti wrote: >>>>> On 06

[Freeipa-devel] [DESIGN-REVIEW] V4/Manage_replication_topology_4_4

2016-05-16 Thread Oleg Fayans
Hi, The design is OK, the onlz thing that is missing is a HowToTest section in track tickets [1] and [2] about clean-dangling-ruvs and abort-clean-ruv respectively. It would really help if these tickets had detailed steps to test (in case of dangling RUV's - steps to generate them) --

Re: [Freeipa-devel] [DESIGN-REVIEW] V4/Manage_replication_topology_4_4

2016-05-17 Thread Oleg Fayans
Sorry, I forgot to list the tickets themselves On 05/16/2016 03:48 PM, Oleg Fayans wrote: > Hi, > > The design is OK, the onlz thing that is missing is a HowToTest section > in track tickets [1] and [2] about clean-dangling-ruvs and > abort-clean-ruv respectively. It would reall

Re: [Freeipa-devel] [DESIGN-REVIEW] V4/Manage_replication_topology_4_4

2016-05-18 Thread Oleg Fayans
x27;ipa-replica-manage del' will be deprecated? On 05/16/2016 03:48 PM, Oleg Fayans wrote: > Hi, > > The design is OK, the onlz thing that is missing is a HowToTest section > in track tickets [1] and [2] about clean-dangling-ruvs and > abort-clean-ruv respectively. It would re

Re: [Freeipa-devel] [DESIGN-REVIEW] V4/Manage_replication_topology_4_4

2016-05-18 Thread Oleg Fayans
Hi Martin, I should probably rephrase my question: will the server_del API call be added to 'ipa-server-install --uninstall' within 4.4 or is it a more distant plan? On 05/18/2016 05:18 PM, Martin Babinsky wrote: > On 05/18/2016 05:01 PM, Oleg Fayans wrote: >> Hi guys, >

[Freeipa-devel] [Testplan Review]

2016-05-19 Thread Oleg Fayans
Hi all, I've created the first versio of the testplan for Topology Management feature in 4.4 release: http://www.freeipa.org/page/V4/Manage_replication_topology_4_4/Test_Plan Could someone please review it? -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscriptio

Re: [Freeipa-devel] [Testplan Review]

2016-05-23 Thread Oleg Fayans
Hi Petr, The test plan is updated. On 05/19/2016 12:54 PM, Petr Vobornik wrote: > On 05/19/2016 12:38 PM, Oleg Fayans wrote: >> Hi all, >> >> I've created the first versio of the testplan for Topology Management >> feature in 4.4 release: &

[Freeipa-devel] [Testplan Review] Server Roles

2016-05-25 Thread Oleg Fayans
Hi guys. Here is a rather schematic (as neither the feature not the design document is not complete) of the server roles testplan. Could you please review it and tell me what is missing? http://www.freeipa.org/page/V4/Server_Roles/Test_Plan -- Oleg Fayans Quality Engineer FreeIPA team RedHat

[Freeipa-devel] [Test][patch-0042] Automated 2 testcases from Managed Topology testplan

2016-06-03 Thread Oleg Fayans
The patch applies and passes pylint -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 39fd39118a6d6e84a4c8791c17ad54da5cbffd0d Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Fri, 3 Jun 2016 13:22:48 +0200 Subject: [PATCH] Automated 2 managed topology 4.4 testcases --- ipatests

Re: [Freeipa-devel] [Testplan Review]

2016-06-06 Thread Oleg Fayans
? On 05/31/2016 06:36 PM, Petr Vobornik wrote: > On 05/23/2016 09:23 AM, Oleg Fayans wrote: >> Hi Petr, >> >> The test plan is updated. > > Thanks, > > is it possible to number test cases? It is hard to refer to them without > copying the full name. > >

[Freeipa-devel] [Testplan Review] Certs in ID overrides

2016-06-09 Thread Oleg Fayans
Hi guys, Here is the first somewhat skeletal and pretty short version of the testplan. Could you please review it anyone? http://www.freeipa.org/page/V4/Certs_in_ID_overrides/Test_Plan -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [Test][Patch-0043-0045] DNSSec key rotation test

2016-06-13 Thread Oleg Fayans
tried to decrease default key TTL values with no success. Could anyone take a look into this (after 4.4 is released, of course)? -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From f7f3d8c256fc9de3f8f0d82056be5a6d10f6c9a7 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Mon, 13 Jun 2016 08

[Freeipa-devel] [Patch-0046] Increased certmonger timeout to address ticket N 5758

2016-06-16 Thread Oleg Fayans
With this change the certmonger timeout issue is no longer observed in abcd lab. -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 2063d59f3d8303abf056d38a68ac75f9f2d9cd24 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Thu, 16 Jun 2016 10:25:59 +0200 Subject: [PATCH] Increased

[Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-06-16 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From debac0cf5cb24e1c2072d10373f4d9f72cb875a7 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Thu, 16 Jun 2016 16:45:03 +0200 Subject: [PATCH] Automated clean-ruv subcommand test https://fedorahosted.org/freeipa/ticket/5964 --- ipatests

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-06-17 Thread Oleg Fayans
Fixed a bug in the previous patch, automated 2 more testcases from http://www.freeipa.org/page/V4/Manage_replication_topology_4_4/Test_Plan On 06/16/2016 04:46 PM, Oleg Fayans wrote: > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 2b087b1dc64500d58e72296f287aabd82cb

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-06-17 Thread Oleg Fayans
One more test was added to the patch-0048 On 06/17/2016 09:43 AM, Oleg Fayans wrote: > Fixed a bug in the previous patch, automated 2 more testcases from > http://www.freeipa.org/page/V4/Manage_replication_topology_4_4/Test_Plan > > On 06/16/2016 04:46 PM, Oleg

Re: [Freeipa-devel] [PATCH 0021][Tests] Fix failing ipatests/test_ipaserver/test_rpcserver.py

2016-06-24 Thread Oleg Fayans
ACK On 06/24/2016 10:29 AM, Lenka Doudova wrote: > Hi, > > attaching patch for one of the failing tests. Failure caused by an > assertion that was no longer valid. > > Lenka > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- Manage your subscri

Re: [Freeipa-devel] [Testplan Review] Certs in ID overrides

2016-06-27 Thread Oleg Fayans
; On Thu, Jun 09, 2016 at 04:48:57PM +0200, Oleg Fayans wrote: >> Hi guys, >> >> Here is the first somewhat skeletal and pretty short version of the >> testplan. Could you please review it anyone? >> >> http://www.freeipa.org/page/V4/Certs_in_ID_overrides/Test_

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-06-27 Thread Oleg Fayans
Hi guys, Is there a chance the patches NN 0047.1 and 0048.1 get reviewed before 4.4 release? They cover a good part of the Managed Topology 4.4 feature. On 06/17/2016 11:18 AM, Oleg Fayans wrote: > One more test was added to the patch-0048 > > On 06/17/2016 09:43 AM, Oleg Fayans wrote:

Re: [Freeipa-devel] [Testplan Review] Certs in ID overrides

2016-06-28 Thread Oleg Fayans
Hi Sumit, The testplan is updated according to your second note. The WebUI part I'll test once Pavel's patch is merged. On 06/27/2016 10:28 AM, Sumit Bose wrote: > On Mon, Jun 27, 2016 at 10:06:23AM +0200, Oleg Fayans wrote: >> Hi Sumit, >> >> I've updated t

[Freeipa-devel] [Test][Patch-0049, 0050] Certs in ID overrides test

2016-06-28 Thread Oleg Fayans
Passing test output: https://paste.fedoraproject.org/385774/71035231/ -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 7bc97eb762c951a8bc3762d8bd23da4ee06a6edb Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Tue, 28 Jun 2016 10:33:13 +0200 Subject: [PATCH] Added methods to manipulate

Re: [Freeipa-devel] [Test][Patch-0049, 0050] Certs in ID overrides test

2016-06-28 Thread Oleg Fayans
Patch-0050 rebased against latest upstream branch On 06/28/2016 10:45 AM, Oleg Fayans wrote: > Passing test output: > > https://paste.fedoraproject.org/385774/71035231/ > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From f032df3a1d58e200d0f8bf8dbc121e5f03eb041

[Freeipa-devel] [Test][Patch-0051] Fixed import error in replica promotion test

2016-06-28 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 0268535dcc8426667f5742a05f4554f8ff9bd031 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Tue, 28 Jun 2016 16:00:08 +0200 Subject: [PATCH] Fixed import error --- ipatests/test_integration/test_replica_promotion.py | 2 +- 1 file

[Freeipa-devel] [Test][patch-0052] Test for incorrect client domain

2016-06-29 Thread Oleg Fayans
-- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 6fe2f67807a2cd3d9519c1c919c884dd18867f74 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Wed, 29 Jun 2016 10:53:44 +0200 Subject: [PATCH] Test for incorrect client domain https://fedorahosted.org/freeipa/ticket/5976

Re: [Freeipa-devel] [PATCH 0024][Tests] Fix integration tests not to produce incorrect /etc/hosts file

2016-06-29 Thread Oleg Fayans
ot just specific IP/hostname/shortname > strings. > > > Lenka > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. -- 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] [PATCH 0022][Tests] Prevent trust test failures cause by adding duplicate DNS forward zone

2016-06-30 Thread Oleg Fayans
t;>>>>> dnsforwardzone-add. >>>>>>>>>> >>>>>>>>> Grr, I meant this: >>>>>>>>> Even more importantly, the forward zone is completely >>>>>>>>> unnecessary when >>>

Re: [Freeipa-devel] [Test][patch-0052] Test for incorrect client domain

2016-06-30 Thread Oleg Fayans
Hi Martin, Attached is a new version of the patch with two test cases separated. On 06/29/2016 12:23 PM, Martin Basti wrote: > > > On 29.06.2016 10:56, Oleg Fayans wrote: >> >> > > Hello, > > +assert_error(result, > + "

Re: [Freeipa-devel] [PATCH 0025][Tests] RFE: External trust

2016-06-30 Thread Oleg Fayans
ly for > forest root domains even for subdomains, I made some changes in > ipatests/test_integration/tasks.py file, so that it would enable me to > reuse existing function without copy-pasting them for one variable change. > > > Lenka > > > -- Oleg Fayans

Re: [Freeipa-devel] [Test][patch-0052] Test for incorrect client domain

2016-07-01 Thread Oleg Fayans
Hi Martin. Now I have this client installation thing sorted out. The test works as expected On 06/30/2016 02:57 PM, Martin Basti wrote: > > > On 30.06.2016 14:40, Oleg Fayans wrote: >> Hi Martin, >> >> Attached is a new version of the patch with two test cases separa

Re: [Freeipa-devel] [Test][patch-0052] Test for incorrect client domain

2016-07-01 Thread Oleg Fayans
Hi Martin, Thanks for the review. The updated patch is attached On 07/01/2016 04:09 PM, Martin Basti wrote: > > > On 01.07.2016 14:38, Oleg Fayans wrote: >> Hi Martin. Now I have this client installation thing sorted out. The >> test works as expected >> >> On

[Freeipa-devel] [Test][patch-0053] Forced-client-reenrollment test fixed.

2016-07-04 Thread Oleg Fayans
2 out of 7 tests currently fail due to a known issue [1], others pass. [1] https://fedorahosted.org/freeipa/ticket/6029 -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From b19ef400462c722976aea5d2eb853315af1e1099 Mon Sep 17 00:00:00 2001 From: Oleg Fayans Date: Mon, 4 Jul 2016 22:47:05

Re: [Freeipa-devel] [Test][patch-0053] Forced-client-reenrollment test fixed.

2016-07-06 Thread Oleg Fayans
Updated version of the patch is attached with the failing tests marked as xfailed (let's make the jenkins green). On 07/04/2016 10:50 PM, Oleg Fayans wrote: 2 out of 7 tests currently fail due to a known issue [1], others pass. [1] https://fedorahosted.org/freeipa/ticket/6029 --

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-07-08 Thread Oleg Fayans
Hi Martin, Thanks for the review! On 07/08/2016 02:18 PM, Martin Basti wrote: On 27.06.2016 13:53, Oleg Fayans wrote: Hi guys, Is there a chance the patches NN 0047.1 and 0048.1 get reviewed before 4.4 release? They cover a good part of the Managed Topology 4.4 feature. On 06/17/2016 11

<    1   2   3   4