[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