Re: [Freeipa-devel] [PATCH 0405] idviews: Add user certificate attribute to user ID overrides

2016-04-13 Thread Tomas Babej
On 04/07/2016 01:53 PM, Sumit Bose wrote: > On Mon, Apr 04, 2016 at 04:27:02PM +0200, Jan Cholasta wrote: >> Hi, >> >> On 1.4.2016 16:53, Tomas Babej wrote: >>> Hi, >>> >>> this extends the user ID overrides with capability to store the user >>> certificate. >>> >>>

Re: [Freeipa-devel] [PATCH] Added fix for notifying user about account expiration in Web UI

2016-04-13 Thread Abhijeet Kasurde
Ping On 03/22/2016 04:32 PM, Abhijeet Kasurde wrote: Hi All, Please find the updated patches as per review comments. On 03/18/2016 07:39 PM, Petr Vobornik wrote: On 03/18/2016 02:21 PM, Abhijeet Kasurde wrote: Hi All, Please review these patches. Fixes :

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-13 Thread Niranjan
Niranjan wrote: > Petr Viktorin wrote: > > On 04/06/2016 10:55 AM, Niranjan wrote: > > > Greetings, > > > > > > For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have > > > proposed > > > a patch, I think this patch is more of a workaround , than a solution. I > > > would > > >

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 link > to

Re: [Freeipa-devel] [PATCH 0024] ipa-replica-manage: added --suffix option for certain commands

2016-04-13 Thread Martin Basti
On 06.04.2016 14:04, Stanislav Laznicka wrote: On 03/30/2016 04:52 PM, Martin Basti wrote: On 24.03.2016 19:10, Stanislav Laznicka wrote: On 03/23/2016 08:13 PM, Martin Basti wrote: [...] Can you please update design http://www.freeipa.org/page/V4/Manage_replication_topology_4_4 (mainly the

Re: [Freeipa-devel] [PATCH 0405] idviews: Add user certificate attribute to user ID overrides

2016-04-13 Thread Tomas Babej
On 04/13/2016 09:55 AM, Tomas Babej wrote: > On 04/07/2016 01:53 PM, Sumit Bose wrote: >> On Mon, Apr 04, 2016 at 04:27:02PM +0200, Jan Cholasta wrote: >>> Hi, >>> >>> On 1.4.2016 16:53, Tomas Babej wrote: Hi, this extends the user ID overrides with capability to store the user

[Freeipa-devel] [PATCH] 0013 webui: Add ability to convert users from preserved to staged state

2016-04-13 Thread Pavel Vomacka
Hello, This patch adds ability to convert users from preserved to staged state. Fixes this ticket: https://fedorahosted.org/freeipa/ticket/5371 -- Pavel^3 Vomacka >From ac0396b8644fe9cf421a491c35c7eed017dc5cca Mon Sep 17 00:00:00 2001 From: Pavel Vomacka Date: Wed, 13 Apr

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

2016-04-13 Thread Martin Basti
On 12.04.2016 17:23, Martin Babinsky wrote: On 04/12/2016 11:19 AM, Oleg Fayans wrote: ACK. Pushed to: master: 280f1ed85f40a0d123fc891f9ad02a4fff4a363b ipa-4-3: 488dcd3e94a5d6c791baff8cb2f07530072b1d31 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [WIP PATCH] server-del: perform full master removal in managed topology

2016-04-13 Thread Rob Crittenden
Martin Babinsky wrote: This is a WIP patch which moves the `ipa-replica-manage del` subcommand to the 'server-del' API method and exposes it as CLI command[1]. A CI test suite is also included. There are some issues with the patch I would like to discuss in more detail on the list: 1.) In the

Re: [Freeipa-devel] [PATCH 0453 - 0458] host-del: fix updatedns option

2016-04-13 Thread Petr Spacek
On 1.4.2016 18:30, Martin Basti wrote: > https://fedorahosted.org/freeipa/ticket/5675 > > Patches attached. NACK, it breaks if the client does not have any corresponding DNS record. [root@vm-033 git]# ipa host-add host.test. --force -- Added host "host.test"

Re: [Freeipa-devel] [PATCH 0453 - 0458, 0460] host-del: fix updatedns option

2016-04-13 Thread Martin Basti
On 13.04.2016 16:01, Petr Spacek wrote: On 1.4.2016 18:30, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5675 Patches attached. NACK, it breaks if the client does not have any corresponding DNS record. [root@vm-033 git]# ipa host-add host.test. --force --

Re: [Freeipa-devel] [PATCH 0453 - 0458, 0460] host-del: fix updatedns option

2016-04-13 Thread Petr Spacek
On 13.4.2016 16:31, Martin Basti wrote: > > > On 13.04.2016 16:01, Petr Spacek wrote: >> On 1.4.2016 18:30, Martin Basti wrote: >>> https://fedorahosted.org/freeipa/ticket/5675 >>> >>> Patches attached. >> NACK, it breaks if the client does not have any corresponding DNS record. >> >>

Re: [Freeipa-devel] [PATCH 0453 - 0458] host-del: fix updatedns option

2016-04-13 Thread Petr Spacek
On 13.4.2016 16:01, Petr Spacek wrote: > On 1.4.2016 18:30, Martin Basti wrote: >> https://fedorahosted.org/freeipa/ticket/5675 >> >> Patches attached. > > NACK, it breaks if the client does not have any corresponding DNS record. > > [root@vm-033 git]# ipa host-add host.test. --force >

Re: [Freeipa-devel] [DESIGN] Kerberos principal alias handling

2016-04-13 Thread Martin Babinsky
On 04/13/2016 07:50 AM, David Kupka wrote: On 08/04/16 17:10, Martin Babinsky wrote: Hi list, I have put together a draft [1] outlining the effort to reimplement the handling of Kerberos principals in both backend and frontend layers of FreeIPA so that we may have multiple aliases per user,

Re: [Freeipa-devel] [PATCH 0453 - 0458, 0460] host-del: fix updatedns option

2016-04-13 Thread Martin Basti
On 13.04.2016 17:05, Petr Spacek wrote: On 13.4.2016 16:31, Martin Basti wrote: On 13.04.2016 16:01, Petr Spacek wrote: On 1.4.2016 18:30, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5675 Patches attached. NACK, it breaks if the client does not have any corresponding DNS

Re: [Freeipa-devel] [PATCH 0143-0144] different errors/warnings for different LDAP limit type exceeded

2016-04-13 Thread Martin Basti
On 13.04.2016 16:48, Petr Spacek wrote: On 8.4.2016 17:13, Martin Babinsky wrote: On 04/01/2016 05:37 PM, Martin Babinsky wrote: On 03/24/2016 04:14 PM, Martin Babinsky wrote: On 03/22/2016 04:28 PM, Rob Crittenden wrote: Martin Babinsky wrote: On 03/21/2016 12:25 PM, Jan Cholasta wrote:

[Freeipa-devel] [PATCH 0459] use python-netifaces for detection of the local ip addresses

2016-04-13 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5591 Patch attached. From 77d1ead350cd17d2c56706c9f9aa5d2d7d5da73d Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Wed, 13 Apr 2016 16:14:42 +0200 Subject: [PATCH] Use netifaces module instead of 'ip' command Netifaces allows to get

Re: [Freeipa-devel] [PATCH 0453 - 0458] host-del: fix updatedns option

2016-04-13 Thread Martin Basti
On 13.04.2016 17:05, Petr Spacek wrote: On 13.4.2016 16:01, Petr Spacek wrote: On 1.4.2016 18:30, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5675 Patches attached. NACK, it breaks if the client does not have any corresponding DNS record. [root@vm-033 git]# ipa host-add

Re: [Freeipa-devel] [PATCH 0143-0144] different errors/warnings for different LDAP limit type exceeded

2016-04-13 Thread Petr Spacek
On 8.4.2016 17:13, Martin Babinsky wrote: > On 04/01/2016 05:37 PM, Martin Babinsky wrote: >> On 03/24/2016 04:14 PM, Martin Babinsky wrote: >>> On 03/22/2016 04:28 PM, Rob Crittenden wrote: Martin Babinsky wrote: > On 03/21/2016 12:25 PM, Jan Cholasta wrote: >> On 21.3.2016 10:17,

[Freeipa-devel] [WIP PATCH] server-del: perform full master removal in managed topology

2016-04-13 Thread Martin Babinsky
This is a WIP patch which moves the `ipa-replica-manage del` subcommand to the 'server-del' API method and exposes it as CLI command[1]. A CI test suite is also included. There are some issues with the patch I would like to discuss in more detail on the list: 1.) In the original subcommand