On 29 December 2013 05:15, John Griffith <john.griff...@solidfire.com> wrote: > I think Sean made some good recommendations in the review (waiting 24 > hours as well as suggesting ML etc). It seems that cases like this > don't necessarily need mandated time requirements for review but just > need good core reviewers to say "hey, this is a big deal... we should > probably get some feedback here" etc. > > One thing I am curious about however, Gary made a good point about > using the "default_ephemeral_format=" config setting to make this > pretty easy and straight forward. I didn't see any other responses to > that, and it looks like the patch still uses a default of "none". > Quick look at the code it seems like this would be a clean way to go > about things, any reason why this wasn't discussed further?
We make a point of running defaults in TripleO: if the defaults aren't generally production suitable, they aren't suitable defaults. If/when we find a place where there is no sane default, we'll push for having no default and forcing a choice to be made. ext3 wasn't a sane default :). In fact, for CD environments, the ability to set ext3 via config options means this change is easy to convert into an arbitrary-time warning period to users, if a cloud needs to. -Rob -- Robert Collins <rbtcoll...@hp.com> Distinguished Technologist HP Converged Cloud _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
