Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-11 Thread Petr Viktorin 

On 01/10/2013 05:11 PM, John Dennis wrote:

On 01/10/2013 10:23 AM, Martin Kosek wrote:

AFAIU, the API will not change as we do the CSV processing only on
client side
and send processed entries to the server. CSV processing on old
clients should
still work fine.


Is that really true? I thought I remembered CSV parsing logic inside the
plugins.



No. CSV parsing now only happens on the client. For example, when using 
the Web UI no CSV is involved at all.


This would be a CLI change, not an API one. But I guess we need a policy 
for CLI changes, too.


--
Petr³

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread John Dennis 

On 01/10/2013 10:23 AM, Martin Kosek wrote:

AFAIU, the API will not change as we do the CSV processing only on client side
and send processed entries to the server. CSV processing on old clients should
still work fine.


Is that really true? I thought I remembered CSV parsing logic inside the 
plugins.


--
John Dennis 

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Jan Cholasta 

On 10.1.2013 16:23, Martin Kosek wrote:

On 01/10/2013 04:17 PM, Rob Crittenden wrote:

Jan Cholasta wrote:

Hi,

On 10.1.2013 14:14, Petr Spacek wrote:

Hello,

is there any user of CSV support built-in to IPA administration tools
("ipa" command)? Do you consider it sane or even useful? Please reply.



No and no. We should disable CSV in new installs. As you pointed out,
shell does a better job than what we have in IPA.

Honza



We would need some sort of policy on how long to support deprecated API. That
is probably the most important piece.

rob


AFAIU, the API will not change as we do the CSV processing only on client side
and send processed entries to the server. CSV processing on old clients should
still work fine.

Martin



Correct.

--
Jan Cholasta

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Martin Kosek 
On 01/10/2013 04:17 PM, Rob Crittenden wrote:
> Jan Cholasta wrote:
>> Hi,
>>
>> On 10.1.2013 14:14, Petr Spacek wrote:
>>> Hello,
>>>
>>> is there any user of CSV support built-in to IPA administration tools
>>> ("ipa" command)? Do you consider it sane or even useful? Please reply.
>>>
>>
>> No and no. We should disable CSV in new installs. As you pointed out,
>> shell does a better job than what we have in IPA.
>>
>> Honza
>>
> 
> We would need some sort of policy on how long to support deprecated API. That
> is probably the most important piece.
> 
> rob

AFAIU, the API will not change as we do the CSV processing only on client side
and send processed entries to the server. CSV processing on old clients should
still work fine.

Martin

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Rob Crittenden 

Jan Cholasta wrote:

Hi,

On 10.1.2013 14:14, Petr Spacek wrote:

Hello,

is there any user of CSV support built-in to IPA administration tools
("ipa" command)? Do you consider it sane or even useful? Please reply.



No and no. We should disable CSV in new installs. As you pointed out,
shell does a better job than what we have in IPA.

Honza



We would need some sort of policy on how long to support deprecated API. 
That is probably the most important piece.


rob

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Jan Cholasta 

Hi,

On 10.1.2013 14:14, Petr Spacek wrote:

Hello,

is there any user of CSV support built-in to IPA administration tools
("ipa" command)? Do you consider it sane or even useful? Please reply.



No and no. We should disable CSV in new installs. As you pointed out, 
shell does a better job than what we have in IPA.


Honza

--
Jan Cholasta

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Petr Spacek 

Hello,

is there any user of CSV support built-in to IPA administration tools ("ipa" 
command)? Do you consider it sane or even useful? Please reply.



I wanted to add single TXT record with double quotation marks (") inside the 
TXT data.


I spent some time figuring out how it is supposed to work ... and with help of 
Petr^3 I managed to write the command.


The resulting command (for BASH) is absolutely crazy:
ipa dnsrecord-add example.test. newrec --txt-rec='"""created on 13:01:23"""'

Do we really need support for this piece of insanity? Shells can do the same 
thing with much less pain :-)


IPA with CSV support can add multiple attributes at once, e.g.
ipa dnsrecord-add example.test. newrec --txt-rec=1,2,3,4,5,6,7,8,9
will add TXT records with value 1, 2, 3 etc.

BASH can do the same thing (without the escaping hell):
ipa dnsrecord-add example.test. newrec --txt-rec={1,2,3,4,5,6,7,8,9}
and
ipa dnsrecord-add example.test. newrec --txt-rec={1..9}
BASH would expand to
ipa dnsrecord-add example.test. newrec --txt-rec=1 --txt-rec=2 --txt-rec=3 
--txt-rec=4 --txt-rec=5 --txt-rec=6 --txt-rec=7 --txt-rec=8 --txt-rec=9


--
Petr^2 Spacek

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel