[Pulp-list] Consumergroup add_keyvalue error handling

[Todd B. Sanders]

  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