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
> newer on
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
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 Babinsky wro
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 link
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 795c84c8e5be1d77c50328c8d1425af5fec4e8cd Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Tue, 12 Apr 2016 11:18:09 +0200
Subject: [PATCH] Fixed a failure in legacy_client tests
https://fedorahosted.org/f
00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
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 ++
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From df2d57707875d37783c51521e3c5562642652a42 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 7 Apr 2016 11:49:44 +0200
Subject: [PATCH] Add test if replica is working after domain upgrade
Corre
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
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
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
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
C
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From f1821a41498f7829df4d1bdffd338cb70b413667 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 31 Mar 2016 11:58:26 +0200
Subject: [PATCH] Added assertion error messages in topology tests
Added a global
destroy -A`, yes.
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 8e5c99a809a9fff1ec59ec750e4f31d6be837a44 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Wed, 30 Mar 2016 12:34:12 +0200
Subject: [PATCH] Added a kdestroy call to clean ccache
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 8e5c99a809a9fff1ec59ec750e4f31d6be837a44 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Wed, 30 Mar 2016 12:34:12 +0200
Subject: [PATCH] Added a kdestroy call to clean ccache at master/client
uninstallation
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 instal
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 t
ge 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: http://www.freeip
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:
>
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 output).
leg 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 wrote:
>&g
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From a4f0bdbe3a8646070119ccb668ad0bfb8cfb269b Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Mon, 21 Mar 2016 13:27:19 +0100
Subject: [PATCH] Added 5 more tests to Replica Promotion testsuite
https://fedorahost
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:
>>>>
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
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:
>>>
it to
tasks.py
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 2f3d460c0b29d65491a47528802d067ccb9757d8 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Wed, 16 Mar 2016 09:42:58 +0100
Subject: [PATCH] rewrote a misprocessed teardown_method method as a
motion.py --pdb
>>
>>
>> test session starts
>> =
>>
>> platform linux2 -- Python 2.7.10 -- py-1
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:
>>>
>>>
>>>
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:
>>>
>>
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:
>>
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:04, Ol
vel 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
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
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
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
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,
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
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 6842d2c1d068853f16f42df893112cb1e8d716ff Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Mon, 1 Feb 2016 11:34:11 +0100
Subject: [PATCH] Updated connect/disconnect replica to work with both
domain
Hi,
On 01/29/2016 01:44 PM, Martin Basti wrote:
>
>
> On 29.01.2016 13:24, Oleg Fayans wrote:
>> Hi Martin,
>>
>> On 01/29/2016 10:33 AM, Martin Basti wrote:
>>>
>>> On 21.01.2016 12:59, Oleg Fayans wrote:
>>>>
>>> Hello,
>&
Hi Martin,
On 01/29/2016 10:33 AM, Martin Basti wrote:
>
>
> On 21.01.2016 12:59, Oleg Fayans wrote:
>>
>>
> Hello,
>
> 1)
> I'm not sure if it is bug or it is related to this patch, but there is
> missing symetry in test_caless, I see there apply fixes
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From b866ff5336f742e170895b575df7a09419c2d731 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Fri, 29 Jan 2016 16:04:17 +0100
Subject: [PATCH] Moved NM configuration calls to the IntegrationTest base
class
Reconfi
Fellas, can anyone spend a free moment to review this?
On 01/27/2016 09:24 AM, Oleg Fayans wrote:
> Hi guys,
>
> Any chance this can be reviewed any time soon?
>
> On 01/21/2016 12:59 PM, Oleg Fayans wrote:
>>
>>
>>
>
--
Oleg Fayans
Quality Engineer
F
Guys, could you take a look at this one?
On 01/27/2016 11:16 AM, Oleg Fayans wrote:
> Sorry, trailing whitespace detected. This version passes lint
>
> On 01/27/2016 09:23 AM, Oleg Fayans wrote:
>> Hi,
>>
>> On 01/21/2016 04:41 PM, Petr Spacek wrote:
>>> Hel
Hi guys,
Any chance this can be reviewed any time soon?
On 01/21/2016 12:59 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
C
Hi,
On 01/21/2016 04:41 PM, Petr Spacek wrote:
> Hello,
>
> On 21.1.2016 13:42, Oleg Fayans wrote:
>> freeipa-ofayans-0021-Removed-ip-address-option-from-replica-installation.patch
>>
>>
>> From d7ab06a4dcddb919fda351b983d478f1b6968578 Mon Sep 17 00:00:0
Sorry, trailing whitespace detected. This version passes lint
On 01/27/2016 09:23 AM, Oleg Fayans wrote:
> Hi,
>
> On 01/21/2016 04:41 PM, Petr Spacek wrote:
>> Hello,
>>
>> On 21.1.2016 13:42, Oleg Fayans wrote:
>>> freeipa-ofayans-0021-Removed-ip-address-op
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 31d0abefc54f07e20dc7f5a7b09ca989b9575c5e Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 21 Jan 2016 12:54:25 +0100
Subject: [PATCH] Enabled recreation of test directory in apply_common_fixes
function
W
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From d7ab06a4dcddb919fda351b983d478f1b6968578 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 21 Jan 2016 13:30:02 +0100
Subject: [PATCH] Removed --ip-address option from replica installation
Explicitly spec
On 01/18/2016 10:30 AM, Petr Spacek wrote:
> On 14.1.2016 15:05, Oleg Fayans wrote:
>> Date: Thu, 14 Jan 2016 14:59:37 +0100
>> Subject: [PATCH] fixed an issue with master installation not creating reverse
>> zone
>>
>> When resolv.conf is set to point to th
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 249db32dc5a7ce1e6df826c68f2919c07d953c45 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 14 Jan 2016 14:59:37 +0100
Subject: [PATCH] fixed an issue with master installation not creating reverse
zone
Passes lint, fixes an issue with replica installation failures due to
absence of corresponding reverse zone on master.
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 8d905385ae16d5227248f8ebb98abf0a6b9ee932 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date:
; 0030: If the IP addresses in the topology are resolvable, do not add
> them to master.
>
>
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribut
, Milan Kubík wrote:
> Fixes problems in tests uncovered by dns check introduced in ipa 4.3
>
>
>
--
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 F
Hi!
Could you guys take a look at
http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan once again to
see if it lacks something important
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman
Any chance this patch can be merged this week?
On 12/14/2015 02:08 PM, Oleg Fayans wrote:
> Hi Martin,
>
> On 12/11/2015 05:58 PM, Martin Basti wrote:
>>
>>
>> On 11.12.2015 17:28, Oleg Fayans wrote:
>>> +myre = re.compile(".*Backed up to (?P.*?
On 12/21/2015 04:39 PM, Martin Basti wrote:
>
>
> 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:
>>>>
>>>&g
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.
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.
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 02545831dc70ec55fc0ad
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 wrote:
>>> On (17/12/15 13:53), Oleg Fayans wrote:
>>>>
>>>> -
Hi Martin,
I am sorry, in my previous email I attached the old version of patch
0016. The correct on is attached.
On 12/16/2015 05:47 PM, Martin Basti wrote:
>
>
> On 16.12.2015 15:39, Martin Basti wrote:
>>
>>
>> On 15.12.2015 10:29, Oleg Fayans wrote:
>>
ACK
On 12/16/2015 07:14 PM, Milan Kubík wrote:
> Applies to ipa-4-3 and master. Reason in commit message.
>
>
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/f
Commit message was updated. Thanks!
On 12/17/2015 02:05 PM, Lukas Slebodnik wrote:
> On (17/12/15 13:53), Oleg Fayans wrote:
>>
>> --
>> Oleg Fayans
>> Quality Engineer
>> FreeIPA team
>> RedHat.
>
>>From ed4630140386c1043e36733eb42ec402cc276b
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From ed4630140386c1043e36733eb42ec402cc276bee Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 17 Dec 2015 13:50:19 +0100
Subject: [PATCH] Enabled automatic creation of reverse zone during master
instal
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
.freeipa-ofayans-0020-Added-domain-realm-as-a-parameter-to-master-installation.patch.swp
Description: Binary data
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
; 2. Add scenario to validate once domain level of master is raised to 1,
>>> after that it cannot be lowered.
>>> 3. Add scenario for "ipa-restore" using master and replica topology , so
>>> as to identify the effect of restoration on replica by using &q
4e1562b481880b111d12d2020f2
>
> Honza
>
>
>
--
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
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 succ
+stderr part in the thread from patch 0018.
With these two patches applied one of the tests fails due this bug:
https://fedorahosted.org/freeipa/ticket/5550
On 12/09/2015 12:17 PM, Martin Basti wrote:
>
>
> On 09.12.2015 12:10, Martin Basti wrote:
>>
>>
>> On 09.12.20
Hi Martin,
On 12/11/2015 05:58 PM, Martin Basti wrote:
>
>
> On 11.12.2015 17:28, Oleg Fayans wrote:
>> +myre = re.compile(".*Backed up to (?P.*?)\n.*")
>
> IMO this regexp is not good.
>
> 1)
> please name it better than "myre"
Done
&
ACK
On 12/11/2015 02:45 PM, Martin Basti wrote:
> Patch attached.
--
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/p
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 93b8e9fcbbba0db1f21924b46097c557c9cca358 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Fri, 11 Dec 2015 15:58:39 +0100
Subject: [PATCH] Fixed install_ca and install_kra under domain level 0
Also added ipa_
Hi Martin
On 12/09/2015 10:30 AM, Martin Basti wrote:
>
>
> On 08.12.2015 23:48, Oleg Fayans wrote:
>> Substituted a hardcoded suffix name with a constant DOMAIN_SUFFIX_NAME
>>
>> On 12/08/2015 02:33 PM, Oleg Fayans wrote:
>>> Hi all,
>>>
>&g
ACK
On 12/07/2015 10:44 PM, Martin Basti wrote:
>
>
> On 07.12.2015 15:51, Oleg Fayans wrote:
>>
>> On 12/07/2015 03:51 PM, Martin Basti wrote:
>>>
>>> On 07.12.2015 15:49, Oleg Fayans wrote:
>>>> Hi,
>>>>
>>>> On 12/07
On 12/08/2015 10:09 AM, Martin Basti wrote:
>
>
> On 08.12.2015 10:04, Oleg Fayans wrote:
>> ACK
> Pushed to master: a11cddd75b4e887998ad6fd52a05f87e0354ea30
>
> How about patch mbasti-371, this will not work without it.
Looks safe. Could be pushed too :)
>
ACK
On 12/06/2015 09:52 PM, Martin Basti wrote:
> Without this fix, function removes more entries from /etc/host than is
> required, and it causes installation failure in tests without DNS
>
> Patch attached.
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your
Hi all,
The patches are rebased against the current master.
On 12/02/2015 05:10 PM, Martin Basti wrote:
>
>
> On 02.12.2015 16:18, Oleg Fayans wrote:
>> Hi Martin,
>>
>> On 12/01/2015 04:08 PM, Martin Basti wrote:
>>>
>>>
>>> On 27.11.201
ACK. The initial issue is fixed.
On 12/08/2015 03:03 PM, David Kupka wrote:
> https://fedorahosted.org/freeipa/ticket/5531
>
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinf
Substituted a hardcoded suffix name with a constant DOMAIN_SUFFIX_NAME
On 12/08/2015 02:33 PM, Oleg Fayans wrote:
> Hi all,
>
>
> The patches are rebased against the current master.
>
> On 12/02/2015 05:10 PM, Martin Basti wrote:
>>
>>
>> On 02.12.2015 16:
ACK
On 12/09/2015 07:37 AM, Martin Babinsky wrote:
> On 12/07/2015 01:53 PM, Martin Babinsky wrote:
>> On 12/07/2015 12:07 PM, Oleg Fayans wrote:
>>> Hi Martin,
>>>
>>> CONFIGURED_DOMAIN_LEVEL is declared, but not used. The rest looks fine
>>> to me
&g
Hi,
On 12/07/2015 02:37 PM, Martin Basti wrote:
>
>
> On 07.12.2015 14:32, Martin Basti wrote:
>>
>>
>> On 07.12.2015 13:24, Oleg Fayans wrote:
>>> Hi Martin,
>>>
>>> I would prefer both install_kra and install_ca methods to have
>>
Hi,
On 12/07/2015 02:22 PM, Tomas Babej wrote:
>
>
> On 12/07/2015 10:36 AM, Oleg Fayans wrote:
>> This is an error message that I received at the attempt to install
>> replica with the following command:
>>
>> ipa-replica-install --setup-ca -p -w
>
On 12/07/2015 03:51 PM, Martin Basti wrote:
>
>
> On 07.12.2015 15:49, Oleg Fayans wrote:
>> Hi,
>>
>> On 12/07/2015 02:37 PM, Martin Basti wrote:
>>>
>>> On 07.12.2015 14:32, Martin Basti wrote:
>>>>
>>>> On 07.12.2015 13:24
uld not be used with domain level1)
>
> Martin^2
--
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
Anyone to review it guys?
On 11/06/2015 02:04 PM, 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.
gt; this fixes: https://fedorahosted.org/freeipa/ticket/5516
>>
>> Tomas
>>
>
> Pushed under oneliner rule:
> master: 5cb003f0b4b85dce47499f594c410b34b5c961e2
> ipa-4-2: e5189ef6e23e4691f6c74541da5bc1a0b0f2e73f
>
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
. The domain level is 0. The
packages are built last Friday from upstream code.
--
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
Hi Martin,
CONFIGURED_DOMAIN_LEVEL is declared, but not used. The rest looks fine to me
On 12/07/2015 11:05 AM, Martin Babinsky wrote:
> This patch should fix teardown methods in replication-related CI tests
> ran at non-zero domain level.
>
>
>
--
Oleg Fayans
Quality Engine
ached, it needs to have attached all patches I sent today and
> also attached workaround patch (Martin3 will provide proper fix)
>
> I haven't had time/mood/resources to test this patch with domain level
> 0, so testing this patch with domain level 0 is appreciated.
>
>
> Oleg, I
--
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
74 passed in 807.35 seconds =
LS
Here the test affected the negative test setup, which is fixed in the
latest revision.
Thanks,
Tomas
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.c
Hi Martin,
On 12/01/2015 04:08 PM, Martin Basti wrote:
On 27.11.2015 16:26, Oleg Fayans wrote:
And patch N 16 passes lint too:
On 11/27/2015 04:03 PM, Oleg Fayans wrote:
Hi,
On 11/27/2015 03:26 PM, Martin Basti wrote:
On 27.11.2015 15:04, Oleg Fayans wrote:
Hi Martin,
All your
/23/2015 06:51 PM, Oleg Fayans wrote:
Hi all,
Here is a draft of the Replica Promotion test plan
http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan
== Test case: Unprivileged users are not allowed to enroll and
promote clients ==
User credentials are passed there through -p $principal
-4.2.90.201512010847GITfa62480-0.fc23.x86_64
freeipa-debuginfo-4.2.90.201512010847GITfa62480-0.fc23.x86_64
freeipa-tests-4.2.90.201512010847GITfa62480-0.fc23.x86_64
freeipa-client-4.2.90.201512010847GITfa62480-0.fc23.x86_64
Does anybody have any ideas?
--
Oleg Fayans
Quality Engineer
FreeIPA team
Hi Martin,
On 11/27/2015 03:05 PM, Martin Basti wrote:
On 26.11.2015 14:39, Petr Vobornik wrote:
On 11/23/2015 06:51 PM, Oleg Fayans wrote:
Hi all,
Here is a draft of the Replica Promotion test plan
http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan
== Test case: Unprivileged
Hi,
On 11/27/2015 03:26 PM, Martin Basti wrote:
On 27.11.2015 15:04, Oleg Fayans wrote:
Hi Martin,
All your suggestions were taken into account. Both patches are
updated. Thank you for your help!
On 11/26/2015 10:50 AM, Martin Basti wrote:
On 26.11.2015 10:04, Oleg Fayans wrote:
Hi
And patch N 16 passes lint too:
On 11/27/2015 04:03 PM, Oleg Fayans wrote:
Hi,
On 11/27/2015 03:26 PM, Martin Basti wrote:
On 27.11.2015 15:04, Oleg Fayans wrote:
Hi Martin,
All your suggestions were taken into account. Both patches are
updated. Thank you for your help!
On 11/26/2015 10
cls.master.config.domain_level = cls.domain_level
if cls.topology is None:
return
I do not see reason why test should override configuration in config in
this case.
Martin
On 25.11.2015 16:44, Oleg Fayans wrote:
Hi,
Here is the updated version of the patch (more tests +
stall step is enough.
Martin^2
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From e40484bee08091ce1c058712a73040c9b0ce Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Wed, 25 Nov 2015 16:38:37 +0100
Subject: [PATCH] Prepared integration tests for r
plica is already in 'cn', so instead of searching
cn=custodia for entries matching (memberPrincipal=host/$HOSTNAME), we
could get cn={enc,sig}/$HOSTNAME,cn=custodia directly.
Honza
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
h
Fayans
Quality Engineer
FreeIPA team
RedHat.
--
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
Hi all,
Here is a draft of the Replica Promotion test plan
http://www.freeipa.org/page/V4/Replica_Promotion/Test_plan
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
101 - 200 of 359 matches
Mail list logo