Re: [Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Tomas Babej
On 02/22/2016 06:14 PM, Alexander Bokovoy wrote: > On Mon, 22 Feb 2016, Tomas Babej wrote: >> >> >> On 02/22/2016 11:48 AM, Alexander Bokovoy wrote: >>> Hi, >>> >>> attached patch should update compat tree configuration if it exist to >>> follow slapi-nis 0.55 which has support for external

Re: [Freeipa-devel] [PATCH 0416-0417] fix broken configuration of sidgen and extdom plugins

2016-02-22 Thread Tomas Babej
On 02/22/2016 07:15 PM, Martin Basti wrote: > > > On 22.02.2016 17:05, Martin Basti wrote: >> >> >> On 19.02.2016 15:02, Alexander Bokovoy wrote: >>> On Fri, 19 Feb 2016, Petr Vobornik wrote: On 02/19/2016 11:12 AM, Alexander Bokovoy wrote: > On Fri, 19 Feb 2016, Martin Basti wrote:

Re: [Freeipa-devel] [PATCH 0416-0418] fix broken configuration of sidgen and extdom plugins

2016-02-22 Thread Martin Basti
On 22.02.2016 19:15, Martin Basti wrote: On 22.02.2016 17:05, Martin Basti wrote: On 19.02.2016 15:02, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Petr Vobornik wrote: On 02/19/2016 11:12 AM, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Martin Basti wrote: WIP patch attached

[Freeipa-devel] [PATCHES] 0772-0774 Python3 fixes in for client installation

2016-02-22 Thread Petr Viktorin
Hello, These fixes are needed for the "happy path" of ipa-client-install --server on Python 3. -- Petr Viktorin From d51ff822165f434ed7c87cde44f64b8f05e33797 Mon Sep 17 00:00:00 2001 From: Petr Viktorin Date: Fri, 19 Feb 2016 17:26:57 +0100 Subject: [PATCH] ipadiscovery:

[Freeipa-devel] [PATCH] 0771 Package python3-ipaclient

2016-02-22 Thread Petr Viktorin
Hello, This will make a python3-ipaclient RPM. -- Petr Viktorin From 7fed33caa22d8bc26345bbaef7879fde04d85a31 Mon Sep 17 00:00:00 2001 From: Petr Viktorin Date: Fri, 19 Feb 2016 14:54:18 +0100 Subject: [PATCH] Package python3-ipaclient Part of the work for

[Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Alexander Bokovoy
Hi, attached patch should update compat tree configuration if it exist to follow slapi-nis 0.55 which has support for external members of IPA groups. However, the real work is done in SSSD. These patches are not upstreamed yet. We'll need to bump SSSD dependency in future once they come to

[Freeipa-devel] [PATCH 0135] upgrade: unconditional import of certificate profiles into LDAP

2016-02-22 Thread Martin Babinsky
https://fedorahosted.org/freeipa/ticket/5682 -- Martin^3 Babinsky From 57af96fe59c0f0e75edc93e3cfd3504fc558d5f0 Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Mon, 22 Feb 2016 13:35:41 +0100 Subject: [PATCH] upgrade: unconditional import of certificate profiles into

Re: [Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Alexander Bokovoy
On Mon, 22 Feb 2016, Lukas Slebodnik wrote: On (22/02/16 12:48), Alexander Bokovoy wrote: Hi, attached patch should update compat tree configuration if it exist to follow slapi-nis 0.55 which has support for external members of IPA groups. However, the real work is done in SSSD. These patches

Re: [Freeipa-devel] [PATCH] 0008 Refactor test_sudocmdgroup_plugin, create SudoCmdGroupTracker

2016-02-22 Thread Filip Skola
Hi, the test has been updated so it now uses the SudoCmdTracker (from the previous patch). Filip - Original Message - > NACK. > > "create_sudocmd" and "delete_sudocmd" should be imported from Tracker, not > from the previous test (sudocmd_plugin). > > - alich - > > - Original

Re: [Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Lukas Slebodnik
On (22/02/16 12:48), Alexander Bokovoy wrote: >Hi, > >attached patch should update compat tree configuration if it exist to >follow slapi-nis 0.55 which has support for external members of IPA >groups. > >However, the real work is done in SSSD. These patches are not upstreamed >yet. We'll need to

Re: [Freeipa-devel] [PATCH] 0007 Refactor test_sudocmd_plugin

2016-02-22 Thread Filip Skola
Hi, sudocmd tracker has been created. Filip - Original Message - > NACK. > > "create_sudocmd" and "delete_sudocmd" should be placed in Tracker. So this > patch should create the tracker as well. > > - Original Message - > > From: "Filip Skola" > > To:

Re: [Freeipa-devel] [PATCH 0011] Move freeipa certmonger helpers to libexecdir.

2016-02-22 Thread David Kupka
On 22/02/16 07:28, Jan Cholasta wrote: On 18.2.2016 10:10, David Kupka wrote: On 19/01/16 16:10, David Kupka wrote: On 19/01/16 14:38, Jan Cholasta wrote: On 19.1.2016 14:26, Martin Kosek wrote: On 01/19/2016 01:47 PM, David Kupka wrote: I've polished the patch attached to #5586 by Timo

[Freeipa-devel] [PATCH 546] client: stop using /etc/pki/nssdb

2016-02-22 Thread Jan Cholasta
Hi, the attached patch fixes . Honza -- Jan Cholasta From 30e528a6254c72b52f89dbfa164c2b972939c58f Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Mon, 22 Feb 2016 15:05:35 +0100 Subject: [PATCH] client: stop using

Re: [Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Tomas Babej
On 02/22/2016 11:48 AM, Alexander Bokovoy wrote: > Hi, > > attached patch should update compat tree configuration if it exist to > follow slapi-nis 0.55 which has support for external members of IPA > groups. > > However, the real work is done in SSSD. These patches are not upstreamed > yet.

Re: [Freeipa-devel] [PATCH] 0004 webui: topology graph: Add pan and zoom functionality

2016-02-22 Thread Pavel Vomacka
On 02/18/2016 06:21 PM, Petr Vobornik wrote: On 02/03/2016 03:37 PM, Pavel Vomacka wrote: Hello, I'm sending a patch for review. This patch adds pan and zoom functionality to the topology graph. The page remembers old position and size of the graph. So, it keeps these settings after

Re: [Freeipa-devel] [PATCH 0011] Move freeipa certmonger helpers to libexecdir.

2016-02-22 Thread Martin Kosek
On 02/22/2016 04:04 PM, Jan Cholasta wrote: > On 22.2.2016 15:56, David Kupka wrote: >> On 22/02/16 07:28, Jan Cholasta wrote: >>> On 18.2.2016 10:10, David Kupka wrote: On 19/01/16 16:10, David Kupka wrote: > On 19/01/16 14:38, Jan Cholasta wrote: >> On 19.1.2016 14:26, Martin Kosek

Re: [Freeipa-devel] [PATCH 0011] Move freeipa certmonger helpers to libexecdir.

2016-02-22 Thread Jan Cholasta
On 22.2.2016 15:56, David Kupka wrote: On 22/02/16 07:28, Jan Cholasta wrote: On 18.2.2016 10:10, David Kupka wrote: On 19/01/16 16:10, David Kupka wrote: On 19/01/16 14:38, Jan Cholasta wrote: On 19.1.2016 14:26, Martin Kosek wrote: On 01/19/2016 01:47 PM, David Kupka wrote: I've polished

Re: [Freeipa-devel] [PATCH 0416] fix broken configuration of sidgen and extdom plugins

2016-02-22 Thread Martin Basti
On 19.02.2016 15:02, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Petr Vobornik wrote: On 02/19/2016 11:12 AM, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Martin Basti wrote: WIP patch attached https://fedorahosted.org/freeipa/ticket/5665 Comments inline. +# we need to run

Re: [Freeipa-devel] [PATCH 200] slapi-nis: update configuration to allow external members

2016-02-22 Thread Alexander Bokovoy
On Mon, 22 Feb 2016, Tomas Babej wrote: On 02/22/2016 11:48 AM, Alexander Bokovoy wrote: Hi, attached patch should update compat tree configuration if it exist to follow slapi-nis 0.55 which has support for external members of IPA groups. However, the real work is done in SSSD. These

[Freeipa-devel] [PATCH 547] cacert install: fix trust chain validation

2016-02-22 Thread Jan Cholasta
Hi, the attached patch fixes . Honza -- Jan Cholasta From 7e0ec898cd58647250ed673fa55b98012939e373 Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Mon, 22 Feb 2016 18:14:46 +0100 Subject: [PATCH] cacert install: fix trust

Re: [Freeipa-devel] [PATCH 0135] upgrade: unconditional import of certificate profiles into LDAP

2016-02-22 Thread Fraser Tweedale
On Mon, Feb 22, 2016 at 02:03:49PM +0100, Martin Babinsky wrote: > https://fedorahosted.org/freeipa/ticket/5682 > > -- > Martin^3 Babinsky > Thanks for the patch. Conditional ACK. Patch is tested and works, but I am wary about checking for substring match against RemoteRetrieveError reason

Re: [Freeipa-devel] [PATCH 0135] upgrade: unconditional import of certificate profiles into LDAP

2016-02-22 Thread Jan Cholasta
On 23.2.2016 06:40, Fraser Tweedale wrote: On Mon, Feb 22, 2016 at 02:03:49PM +0100, Martin Babinsky wrote: https://fedorahosted.org/freeipa/ticket/5682 -- Martin^3 Babinsky Thanks for the patch. Conditional ACK. Patch is tested and works, but I am wary about checking for substring match

Re: [Freeipa-devel] [PATCH 0135] upgrade: unconditional import of certificate profiles into LDAP

2016-02-22 Thread Fraser Tweedale
On Tue, Feb 23, 2016 at 07:32:31AM +0100, Jan Cholasta wrote: > On 23.2.2016 06:40, Fraser Tweedale wrote: > >On Mon, Feb 22, 2016 at 02:03:49PM +0100, Martin Babinsky wrote: > >>https://fedorahosted.org/freeipa/ticket/5682 > >> > >>-- > >>Martin^3 Babinsky > >> > >Thanks for the patch.

Re: [Freeipa-devel] [PATCH 0416-0417] fix broken configuration of sidgen and extdom plugins

2016-02-22 Thread Martin Basti
On 22.02.2016 17:05, Martin Basti wrote: On 19.02.2016 15:02, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Petr Vobornik wrote: On 02/19/2016 11:12 AM, Alexander Bokovoy wrote: On Fri, 19 Feb 2016, Martin Basti wrote: WIP patch attached https://fedorahosted.org/freeipa/ticket/5665

Re: [Freeipa-devel] [PATCH 0002] Refactor test_group_plugin

2016-02-22 Thread Filip Skola
- Original Message - > On 02/10/2016 09:17 AM, Milan Kubík wrote: > > On 02/09/2016 04:19 PM, Milan Kubík wrote: > >> On 01/28/2016 10:42 AM, Filip Skola wrote: > >>> > >>> - Original Message - > On 01/25/2016 11:11 AM, Filip Skola wrote: > > - Original Message -