[Pulp-list] Consumergroup add_keyvalue error handling

[Sayli Karmarkar]

Todd B. Sanders wrote:
>  On 09/27/2010 01:28 PM, Sayli Karmarkar wrote:
>> As we decided in key-value attributes deep dive, I am making changes 
>> to consumergroup to make key_value_pair a property of consumergroup 
>> (not just consumer). That will make sure that consumers added to the 
>> consumergroup AFTER setting a key-value property of the group will 
>> also inherit that property. This complicates error handling a little 
>> bit.
>>
>> Let's say I have consumer1 and consumer2 registered to the 
>> pulp-server and I add key-value {location : rdu} to consumer1. 
>> consumer1 and consumer2 belong to consumergroup "testgroup". When I 
>> try to add key-value {location : sea} to this group, should it -
>> 1. fail saying consumer1 already has  different value for that key ?
>> 2. override key-value for consumer1 to {location : sea} ? (basically 
>> delete location key from consumer1, so there is no conflict)
>> 3. fail, unless --override flag is set ?
>>
>> Same error handling will need to be added when adding a consumer to 
>> consumergroup.
>>
>> Thoughts? Suggestions?
>>
> Definitely either 2. or 3.   Do we have the notion of --override or 
> --force anywhere else?
>
> -Todd
No. We don't have --override or --force anywhere else.

-- 
Sayli Karmarkar
skarmark at redhat.com

Systems Management
http://www.redhat.com