Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-18 Thread Petr Viktorin
On 05/14/2012 04:00 PM, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-18 Thread Rob Crittenden
Petr Viktorin wrote: On 05/14/2012 04:00 PM, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-18 Thread Petr Viktorin
On 05/18/2012 03:49 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 05/14/2012 04:00 PM, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote:

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-14 Thread Martin Kosek
On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-14 Thread Martin Kosek
On Mon, 2012-05-14 at 09:36 +0200, Martin Kosek wrote: On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-14 Thread Rob Crittenden
Martin Kosek wrote: On Thu, 2012-05-10 at 15:19 +0200, Petr Viktorin wrote: On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-05-10 Thread Petr Viktorin
On 04/10/2012 07:53 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 19:25 +0200, Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: [snip] Like you said above, we should either not

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-12 Thread Petr Viktorin
On 04/11/2012 09:42 AM, Jan Cholasta wrote: On 11.4.2012 09:27, Martin Kosek wrote: On Wed, 2012-04-11 at 09:18 +0200, Jan Cholasta wrote: On 10.4.2012 19:56, Dmitri Pal wrote: ... The use case I would see is the extensibility. Say a customer wants to extend a schema and add an attribute X

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-11 Thread Martin Kosek
On Tue, 2012-04-10 at 13:56 -0400, Dmitri Pal wrote: On 04/10/2012 01:48 PM, Rob Crittenden wrote: [snip] The use case I would see is the extensibility. Say a customer wants to extend a schema and add an attribute X to the user object. He would still be able to manage users using CLI without

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-11 Thread Jan Cholasta
On 10.4.2012 19:56, Dmitri Pal wrote: On 04/10/2012 01:48 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-11 Thread Jan Cholasta
On 11.4.2012 09:27, Martin Kosek wrote: On Wed, 2012-04-11 at 09:18 +0200, Jan Cholasta wrote: On 10.4.2012 19:56, Dmitri Pal wrote: On 04/10/2012 01:48 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-11 Thread Dmitri Pal
On 04/11/2012 03:42 AM, Jan Cholasta wrote: On 11.4.2012 09:27, Martin Kosek wrote: On Wed, 2012-04-11 at 09:18 +0200, Jan Cholasta wrote: On 10.4.2012 19:56, Dmitri Pal wrote: On 04/10/2012 01:48 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On

[Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Petr Viktorin
I'm aware that we have backwards compatibility requirements so we have to stick with unfortunate decisions, but I wanted you to know what I think. Please tell me I'm wrong! It is not clear what --{set,add,del}attr and friends should do. On the one hand they should be powerful -- presumably

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Jan Cholasta
On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to stick with unfortunate decisions, but I wanted you to know what I think. Please tell me I'm wrong! It is not clear what --{set,add,del}attr and friends should do. On the one hand

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Petr Viktorin
On 04/10/2012 05:03 PM, Jan Cholasta wrote: To be functionally complete, we should also add validated equivalents of --{add,del}attr to *-mod commands for all multivalue params (think --add-param and --del-param for each --param). We need something like that anyway. Requiring users to learn

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Petr Spacek
On 04/10/2012 05:31 PM, Petr Viktorin wrote: On 04/10/2012 05:03 PM, Jan Cholasta wrote: On 04/10/2012 05:31 PM, Petr Viktorin wrote: tl;dr: --setattr work on IPA-managed attributes (with validation) is a mistake. +1 It adds no functionality, only complexity. We don't want people to use

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Martin Kosek
On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to stick with unfortunate decisions, but I wanted you to know what I think. Please tell me I'm wrong! It is not

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Petr Viktorin
On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to stick with unfortunate decisions, but I wanted you to know what I think. Please

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Rob Crittenden
Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to stick with unfortunate decisions, but I wanted you to know

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Stephen Ingram
On Tue, Apr 10, 2012 at 10:25 AM, Petr Viktorin pvikt...@redhat.com wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to

Re: [Freeipa-devel] [RANT] --setattr validation is a minefield.

2012-04-10 Thread Dmitri Pal
On 04/10/2012 01:48 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 04/10/2012 07:07 PM, Martin Kosek wrote: On Tue, 2012-04-10 at 17:03 +0200, Jan Cholasta wrote: On 10.4.2012 16:00, Petr Viktorin wrote: I'm aware that we have backwards compatibility requirements so we have to stick with