<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 26, 2015 at 5:09 PM, Deepak Shetty <span dir="ltr"><<a href="mailto:dpkshetty@gmail.com" target="_blank">dpkshetty@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Mar 26, 2015 at 4:40 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 03/25/2015 09:26 AM, Dean Troyer wrote:<br>
> On Wed, Mar 25, 2015 at 6:04 AM, Deepak Shetty <<a href="mailto:dpkshetty@gmail.com" target="_blank">dpkshetty@gmail.com</a><br>
</span><span>> <mailto:<a href="mailto:dpkshetty@gmail.com" target="_blank">dpkshetty@gmail.com</a>>> wrote:<br>
><br>
>     Had a question here, why is this source in the end ?<br>
><br>
><br>
> More often than not, you will want the variables defined by the other<br>
> plugins (including the built-ins), this is really the first case we've<br>
> had to deviate from that.  The right solution is to add an<br>
> 'override_plugins' phase that runs before the built-ins are sourced so<br>
> you can override the built-in defaults.<br>
<br>
</span>Ok, we did a quick discussion at the QA Sprint yesterday on this and the<br>
result is - <a href="https://review.openstack.org/#/c/167933/" target="_blank">https://review.openstack.org/#/c/167933/</a><br>
<br>
Please see if that would work in the glusterfs case.<br></blockquote><div><br></div></div></div><div>Thanks Sean. <br><br>+Bharat <br></div></div></div></div></blockquote><div><br></div><div>Adding Bharat now<br><br></div><div>thanx,<br></div><div>deepak<br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Bharat,<br></div><div>  Pls check the below scenarios with sean's patch:<br><br></div><div>1) enable_plugin glusterfs set & CINDER_ENABLED_BACKENDS unset - it should pick from plugin<br>2) enable_plugin glusterfs set & CINDER_ENABLED_BACKENDS set - it should pick from localrc <br></div><div>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<br>4) enable_plugin glusterfs unset - it should pick cinder default<br><br></div><div>thanx,<br></div><div>deepak<br></div></div><br></div></div>
</blockquote></div><br></div></div>