[openstack-dev] [cinder] LVM snapshot performance issue -- why isn't thin provisioning the default?

Chris Friesen chris.friesen at windriver.com
Fri Sep 18 18:32:57 UTC 2015


On 09/18/2015 12:11 PM, John Griffith wrote:
>
>
> On Fri, Sep 18, 2015 at 11:31 AM, Chris Friesen <chris.friesen at windriver.com
> <mailto:chris.friesen at windriver.com>> wrote:
>
>     On 09/18/2015 11:01 AM, John Griffith wrote:
>
>
>
>         On Fri, Sep 18, 2015 at 9:06 AM, Chris Friesen
>         <chris.friesen at windriver.com <mailto:chris.friesen at windriver.com>
>         <mailto:chris.friesen at windriver.com
>         <mailto:chris.friesen at windriver.com>>> wrote:
>
>              On 09/18/2015 06:57 AM, Eric Harney wrote:
>
>                  On 09/17/2015 06:06 PM, John Griffith wrote:
>
>
>                      Having the "global conf" settings intermixed with the
>         backend sections
>                      caused a number of issues when we first started working on
>         this.  That's
>                      part of why we require the "self.configuration" usage all
>         over in the
>                      drivers.  Each driver instantiation is it's own independent
>         entity.
>
>
>                  Yes, each driver instantiation is independent, but that would
>         still be
>                  the case if these settings inherited values set in [DEFAULT]
>         when they
>                  aren't set in the backend section.
>
>
>              Agreed.  If I explicitly set something in the [DEFAULT] section,
>         that should
>              carry through and apply to all the backends unless overridden in the
>              backend-specific section.
>
>
>
>         Bottom line "yes", ideally in the case of drivers we would check
>         global/default
>         setting, and then override it if something was provided in the driver
>         specific
>         setting, or if the driver itself set a different default.  That seems
>         like the
>         right way to be doing it anyway.  I've looked at that a bit this
>         morning, the
>         issue is that currently we don't even pass any of those higher level conf
>         settings in to the drivers init methods anywhere.  Need to figure out how to
>         change that, then it should be a relatively simple fix.
>
>
>     Actually, I think it should be slightly different.  If I set a variable in
>     the global/default section of the config file, then that should override any
>     defaults in the code for a driver.
>
> Hmm... well, on the bright side that might be easier to implement at least :). I
> guess I don't agree on the meaning of "DEFAULT", to me a "DEFAULT" section
> means, "these are the defaults if you don't specify something else", no?  Your
> proposal seems really counter-intuitive to me.

That's what I proposed.

If you specify anything in the driver-specific portion of the config file, that 
takes priority over everything.  If you specify something in the DEFAULT portion 
of the config file, then that takes priority over the in-code defaults.  If you 
specify a default value in the driver-specific code, that takes priority over 
any defaults specified in the global/generic code.

Chris




More information about the OpenStack-dev mailing list