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