Open Stack

On 01/05/13 11:46 +0200, Thierry Carrez wrote:
>Clint Byrum wrote:
>> On 2013-04-30 14:52, Russell Bryant wrote:
>>> I'm really worried about the transition of ownership going from the
>>> submitter to the project as a whole much earlier in the process.  I'd
>>> like to have a high confidence level that it's worth having this code
>>> for the long run before it starts coming in.  That to me requires it
>>> baking out of tree and getting completed before we accept it.
>>
>> What is the negative impact of having disabled code in a release? I
>> think we would need a good reason before we would back accepted code
>> out. If the code is just sitting there, undocumented and turned off, not
>> interfering with anything, I'm having a hard time thinking of a logical
>> reason to back it out.
>
>I think the root of the problem with shipping incomplete features
>disabled by default is the stable maintenance expectations. Is that
>incomplete feature considered a bug in the stable branch ? The code
>might be disabled, it can still be enabled... Could we mark that code in
>a way that make it sufficiently clear it's not supported by the stable
>branch team ? I don't really want to make it easier for CD at the
>expense of the stable branch team. Or turn it into a public cloud
>deployer vs. private cloud distros argument.
>

I agree with Thierry and Russell here. 

Having incomplete features (even if disabled) doesn't feel right. Thierry
made good points about what we consider stable and what we actually
maintain. We should also consider that reviewing an incomplete feature
doesn't give the full view of how the code is meant to be used, or
possible bugs or areas that could be improved. It would just make the
review process more fragile and harder.

Personally, I don't see the benefit of having incomplete features in
the tree, instead of just parking it somewhere else until it is
complete.

FF

-- 
{ name: "Flavio Percoco",
   gpg: "87112EC1", 
   internal: "8261386",
   phone: "+390687502386",
   irc: ["fpercoco", "flaper87"]}