On Thu, Mar 26, 2015 at 5:09 PM, Deepak Shetty <dpkshe...@gmail.com> wrote:
> > > On Thu, Mar 26, 2015 at 4:40 PM, Sean Dague <s...@dague.net> wrote: > >> On 03/25/2015 09:26 AM, Dean Troyer wrote: >> > On Wed, Mar 25, 2015 at 6:04 AM, Deepak Shetty <dpkshe...@gmail.com >> > <mailto:dpkshe...@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 > >
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev