Open Stack

On Thu, Mar 26, 2015 at 5:09 PM, Deepak Shetty <dpkshetty at gmail.com> wrote:

>
>
> 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
>

Adding Bharat now

thanx,
deepak



>
> 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/f4e083ea/attachment.html>