Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-29 Thread Oleg Fayans
Hi all, On 09/23/2015 09:13 AM, Petr Spacek wrote: On 22.9.2015 10:42, Oleg Fayans wrote: +++ b/ipatests/test_integration/tasks.py @@ -58,6 +58,14 @@ def check_arguments_are(slice, instanceof): return wrapped return wrapper +def prepare_reverse_zone(host, ip): +nums = ip.sp

Re: [Freeipa-devel] [PATCH 0086] Migrate OTP import script to python-cryptography

2015-09-29 Thread Martin Babinsky
On 09/25/2015 07:05 PM, Nathaniel McCallum wrote: On Fri, 2015-09-25 at 18:29 +0200, Martin Babinsky wrote: On 09/25/2015 04:53 PM, Nathaniel McCallum wrote: On Mon, 2015-08-31 at 11:08 -0400, Nathaniel McCallum wrote: https://fedorahosted.org/freeipa/ticket/5192 -- Manage your subscription fo

Re: [Freeipa-devel] [PATCH] 0087 Fix an integer underflow bug in libotp

2015-09-29 Thread Tomas Babej
On 09/25/2015 06:38 PM, Nathaniel McCallum wrote: > On Fri, 2015-09-25 at 12:18 -0400, Nathaniel McCallum wrote: >> Temporarily storing the offset time in an unsigned integer causes the >> value of the offset to underflow when a (valid) negative offset value >> is generated. Using a signed variabl

Re: [Freeipa-devel] [PATCH] 0087 Fix an integer underflow bug in libotp

2015-09-29 Thread Tomas Babej
On 09/29/2015 01:57 PM, Tomas Babej wrote: > > On 09/25/2015 06:38 PM, Nathaniel McCallum wrote: >> On Fri, 2015-09-25 at 12:18 -0400, Nathaniel McCallum wrote: >>> Temporarily storing the offset time in an unsigned integer causes the >>> value of the offset to underflow when a (valid) negative

Re: [Freeipa-devel] [PATCHES 466-468, 0316] install: Add common base class for server and replica install

2015-09-29 Thread Milan Kubík
On 09/23/2015 05:01 PM, Martin Basti wrote: On 09/22/2015 12:10 PM, Jan Cholasta wrote: On 22.9.2015 10:29, Martin Babinsky wrote: On 09/16/2015 10:44 AM, Jan Cholasta wrote: On 16.9.2015 08:11, Jan Cholasta wrote: On 15.9.2015 07:22, Jan Cholasta wrote: On 10.8.2015 16:58, Martin Babinsky

Re: [Freeipa-devel] [PATCHES 466-468, 0316] install: Add common base class for server and replica install

2015-09-29 Thread Martin Basti
On 09/29/2015 03:11 PM, Milan Kubík wrote: On 09/23/2015 05:01 PM, Martin Basti wrote: On 09/22/2015 12:10 PM, Jan Cholasta wrote: On 22.9.2015 10:29, Martin Babinsky wrote: On 09/16/2015 10:44 AM, Jan Cholasta wrote: On 16.9.2015 08:11, Jan Cholasta wrote: On 15.9.2015 07:22, Jan Cholas

Re: [Freeipa-devel] [PATCH] 0087 Fix an integer underflow bug in libotp

2015-09-29 Thread Tomas Babej
On 09/29/2015 03:10 PM, Tomas Babej wrote: > > > On 09/29/2015 01:57 PM, Tomas Babej wrote: >> >> On 09/25/2015 06:38 PM, Nathaniel McCallum wrote: >>> On Fri, 2015-09-25 at 12:18 -0400, Nathaniel McCallum wrote: Temporarily storing the offset time in an unsigned integer causes the va

Re: [Freeipa-devel] [PATCH 0066] fix for regression in ipa-restore

2015-09-29 Thread David Kupka
On 25/09/15 18:13, Martin Babinsky wrote: fixes https://fedorahosted.org/freeipa/ticket/5328 Fixes the issue for me, ACK. -- David Kupka -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.fr

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Jan Pazdziora
On Thu, Sep 24, 2015 at 09:10:30AM -0400, Simo Sorce wrote: > > I think the problem is that the patch was pushed prematurely. > The option should become unused once the other patches in this patchset are > applied, that is why that patch was not on top of the list but rather down > close to the bo

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Simo Sorce
On 29/09/15 09:28, Jan Pazdziora wrote: On Thu, Sep 24, 2015 at 09:10:30AM -0400, Simo Sorce wrote: I think the problem is that the patch was pushed prematurely. The option should become unused once the other patches in this patchset are applied, that is why that patch was not on top of the lis

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Oleg Fayans
Hi Simo, Is this [1] the correct link to the repo containing all latest replica-promotion patches? I tried to build the packages from this code and the build failed due to libpdb not having make_pdb_method [2] I was able to successfully build from the clean upstream tree on the same machine.

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Simo Sorce
On 29/09/15 10:39, Oleg Fayans wrote: Hi Simo, Is this [1] the correct link to the repo containing all latest replica-promotion patches? I tried to build the packages from this code and the build failed due to libpdb not having make_pdb_method [2] I was able to successfully build from the clean

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Oleg Fayans
Hi Simo, It seems to have resolved the initial issue, but now the build fails due to lint complaints: https://paste.fedoraproject.org/272714/54174014/ On 09/29/2015 04:54 PM, Simo Sorce wrote: On 29/09/15 10:39, Oleg Fayans wrote: Hi Simo, Is this [1] the correct link to the repo containing

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Simo Sorce
On 29/09/15 11:50, Oleg Fayans wrote: Hi Simo, It seems to have resolved the initial issue, but now the build fails due to lint complaints: https://paste.fedoraproject.org/272714/54174014/ These happens if you do not have custodia installed. I guess I should make it also a BuildRequires ? Sim

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Oleg Fayans
On 09/29/2015 06:47 PM, Simo Sorce wrote: On 29/09/15 11:50, Oleg Fayans wrote: Hi Simo, It seems to have resolved the initial issue, but now the build fails due to lint complaints: https://paste.fedoraproject.org/272714/54174014/ These happens if you do not have custodia installed. I guess

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-29 Thread Simo Sorce
On 29/09/15 14:56, Oleg Fayans wrote: On 09/29/2015 06:47 PM, Simo Sorce wrote: On 29/09/15 11:50, Oleg Fayans wrote: Hi Simo, It seems to have resolved the initial issue, but now the build fails due to lint complaints: https://paste.fedoraproject.org/272714/54174014/ These happens if you

[Freeipa-devel] The Community Auth.NEXT Working Group Inagural Meeting

2015-09-29 Thread Brian Stinson
Hi FreeIPA! We are starting a working group of member projects looking to solve problems related to Community Authentication. The FreeIPA Community Portal feature added this summer is one of the important bits we are evaluating. We are hoping to collaborate on centos-de...@centos.org, and have I

Re: [Freeipa-devel] The Community Auth.NEXT Working Group Inagural Meeting

2015-09-29 Thread Alexander Bokovoy
On Tue, 29 Sep 2015, Brian Stinson wrote: Hi FreeIPA! We are starting a working group of member projects looking to solve problems related to Community Authentication. The FreeIPA Community Portal feature added this summer is one of the important bits we are evaluating. We are hoping to collabo