Open Stack

On Thu, Mar 26, 2015 at 4:40 PM, Sean Dague <sean at dague.net> wrote:

> On 03/25/2015 09:26 AM, Dean Troyer wrote:
> > On Wed, Mar 25, 2015 at 6:04 AM, Deepak Shetty <dpkshetty at gmail.com
> > <mailto:dpkshetty at gmail.com>> wrote:
> >
> >     Had a question here, why is this source in the end ?
> >
> >
> > More often than not, you will want the variables defined by the other
> > plugins (including the built-ins), this is really the first case we've
> > had to deviate from that.  The right solution is to add an
> > 'override_plugins' phase that runs before the built-ins are sourced so
> > you can override the built-in defaults.
>
> Ok, we did a quick discussion at the QA Sprint yesterday on this and the
> result is - https://review.openstack.org/#/c/167933/
>
> Please see if that would work in the glusterfs case.
>

Thanks Sean.

+Bharat

Bharat,
  Pls check the below scenarios with sean's patch:

1) enable_plugin glusterfs set & CINDER_ENABLED_BACKENDS unset - it should
pick from plugin
2) enable_plugin glusterfs set & CINDER_ENABLED_BACKENDS set - it should
pick from localrc
3) enable_plugin glusterfs set & set some backend-specific var not touched
by lib/cinder (eg: GLUSTERFS_LOOPBACK_SIZE) and see if it picks up correctly
4) enable_plugin glusterfs unset - it should pick cinder default

thanx,
deepak
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150326/f4410ab1/attachment.html>