Re: [Freeipa-devel] [PATCH 0074] Make token window sizes configurable

2014-10-29 Thread Martin Kosek
On 10/28/2014 09:59 PM, Nathaniel McCallum wrote: On Thu, 2014-10-23 at 18:07 -0400, Nathaniel McCallum wrote: This patch gives the administrator variables to control the size of the authentication and synchronization windows for OTP tokens. https://fedorahosted.org/freeipa/ticket/4511

Re: [Freeipa-devel] [PATCH 0074] Make token window sizes configurable

2014-10-29 Thread Petr Viktorin
On 10/29/2014 10:37 AM, Martin Kosek wrote: On 10/28/2014 09:59 PM, Nathaniel McCallum wrote: On Thu, 2014-10-23 at 18:07 -0400, Nathaniel McCallum wrote: This patch gives the administrator variables to control the size of the authentication and synchronization windows for OTP tokens.

Re: [Freeipa-devel] [PATCH] 333 Handle profile changes in dogtag-ipa-ca-renew-agent

2014-10-29 Thread David Kupka
On 10/14/2014 10:52 AM, Jan Cholasta wrote: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4627. (The original patch was posted at http://www.redhat.com/archives/freeipa-devel/2014-September/msg00454.html.) How to test: 1. install server 2. run ipa-certupdate

Re: [Freeipa-devel] [PATCH, slapi-nis] ID view-related patches to slapi-nis

2014-10-29 Thread thierry bordaz
On 10/28/2014 10:11 PM, Alexander Bokovoy wrote: Hi, two patches to slapi-nis are attached: - make sure only DNs from the schema-compat trees are targeted for ID view replacement. This solves issue of https://bugzilla.redhat.com/show_bug.cgi?id=1157989 found by Sumit. - support ID

Re: [Freeipa-devel] [PATCH] 352 Fixed KRA backend.

2014-10-29 Thread Petr Viktorin
On 10/28/2014 10:51 PM, Endi Sukma Dewata wrote: Thanks for the review. New patch attached. On 10/23/2014 3:59 AM, Petr Viktorin wrote: In IPA we usually include the full ticket URL, not just the number. Fixed. The build fails with a lint message: * Module

Re: [Freeipa-devel] [PATCH 0020] Fix zone name to directory name conversion in BINDMgr

2014-10-29 Thread Martin Basti
On 23/10/14 15:44, Martin Basti wrote: On 23/10/14 14:15, Petr Spacek wrote: Hello, Fix zone name to directory name conversion in BINDMgr. https://fedorahosted.org/freeipa/ticket/4657 ACK, signing the root zone works fine now The patch is ready to be pushed. -- Martin Basti

Re: [Freeipa-devel] [PATCH] 0025 Respect UID and GID soft static allocation.

2014-10-29 Thread David Kupka
On 10/24/2014 03:05 PM, David Kupka wrote: On 10/24/2014 01:06 PM, David Kupka wrote: On 10/24/2014 10:43 AM, Martin Basti wrote: On 24/10/14 09:51, David Kupka wrote: https://fedorahosted.org/freeipa/ticket/4585 NACK 1) Why is there line with 'DS System User?' The comment should depend on

Re: [Freeipa-devel] [PATCH 0074] Make token window sizes configurable

2014-10-29 Thread Nathaniel McCallum
On Wed, 2014-10-29 at 12:21 +0100, Petr Viktorin wrote: On 10/29/2014 10:37 AM, Martin Kosek wrote: On 10/28/2014 09:59 PM, Nathaniel McCallum wrote: On Thu, 2014-10-23 at 18:07 -0400, Nathaniel McCallum wrote: This patch gives the administrator variables to control the size of the

Re: [Freeipa-devel] [PATCH 0020] Fix zone name to directory name conversion in BINDMgr

2014-10-29 Thread Petr Viktorin
On 10/29/2014 02:01 PM, Martin Basti wrote: On 23/10/14 15:44, Martin Basti wrote: On 23/10/14 14:15, Petr Spacek wrote: Hello, Fix zone name to directory name conversion in BINDMgr. https://fedorahosted.org/freeipa/ticket/4657 ACK, signing the root zone works fine now The patch is ready

Re: [Freeipa-devel] [PATCH] 333 Handle profile changes in dogtag-ipa-ca-renew-agent

2014-10-29 Thread Petr Viktorin
On 10/29/2014 12:37 PM, David Kupka wrote: On 10/14/2014 10:52 AM, Jan Cholasta wrote: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4627. (The original patch was posted at http://www.redhat.com/archives/freeipa-devel/2014-September/msg00454.html.) How to test: 1.

Re: [Freeipa-devel] [PATCH, slapi-nis] ID view-related patches to slapi-nis

2014-10-29 Thread thierry bordaz
On 10/29/2014 02:23 PM, Alexander Bokovoy wrote: On Wed, 29 Oct 2014, thierry bordaz wrote: The patches fixed the test case in https://bugzilla.redhat.com/show_bug.cgi?id=1157989. Few comments regarding the patch: * in backend_search_cb, it checks if the search is in one of the container.

Re: [Freeipa-devel] [PATCHES] 0661-0672 Switch the test suite to pytest

2014-10-29 Thread Petr Viktorin
On 10/29/2014 01:22 PM, Tomas Babej wrote: On 10/27/2014 04:38 PM, Petr Viktorin wrote: On 10/15/2014 02:58 PM, Petr Viktorin wrote: This almost completes the switch to pytest. There are two missing things: - the details of test results (--with-xunit) are not read correctly by Jenkins. I have

Re: [Freeipa-devel] [PATCHES] 0661-0672 Switch the test suite to pytest

2014-10-29 Thread Petr Viktorin
Please ignore the previous mail, I hit Send my mistake. -- Petr³ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 0025 Respect UID and GID soft static allocation.

2014-10-29 Thread David Kupka
On 10/29/2014 02:34 PM, David Kupka wrote: On 10/24/2014 03:05 PM, David Kupka wrote: On 10/24/2014 01:06 PM, David Kupka wrote: On 10/24/2014 10:43 AM, Martin Basti wrote: On 24/10/14 09:51, David Kupka wrote: https://fedorahosted.org/freeipa/ticket/4585 NACK 1) Why is there line with 'DS

Re: [Freeipa-devel] [PATCH] 352 Fixed KRA backend.

2014-10-29 Thread Endi Sukma Dewata
New patch attached. On 10/29/2014 7:58 AM, Petr Viktorin wrote: Dependency is bumped to 10.2.1-0.1 which is available from my COPR repo: dnf copr enable edewata/pki OK. We should get that to an IPA COPR before merging this. How do we do that? Here is the SRPM: