<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 17, 2015 at 11:31 AM, Eric Harney <span dir="ltr"><<a href="mailto:eharney@redhat.com" target="_blank">eharney@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 09/17/2015 05:00 AM, Duncan Thomas wrote:<br>
> On 16 September 2015 at 23:43, Eric Harney <<a href="mailto:eharney@redhat.com" target="_blank">eharney@redhat.com</a>> wrote:<br>
><br>
>> Currently, at least some options set in [DEFAULT] don't apply to<br>
>> per-driver sections, and require you to set them in the driver section<br>
>> as well.<br>
>><br>
><br>
> This is extremely confusing behaviour. Do you have any examples? I'm not<br>
> sure if we can fix it without breaking people's existing configs but I<br>
> think it is worth trying. I'll add it to the list of things to talk about<br>
> briefly in Tokyo.<br>
><br>
<br>
</span>The most recent place this bit me was with iscsi_helper.<br>
<br>
If cinder.conf has:<br>
<br>
[DEFAULT]<br>
iscsi_helper = lioadm<br>
enabled_backends = lvm1<br>
<br>
[lvm1]<br>
volume_driver = ...LVMISCSIDriver<br>
# no iscsi_helper setting<br>
<br>
<br>
You end up with c-vol showing "iscsi_helper = lioadm", and<br>
"lvm1.iscsi_helper = tgtadm", which is the default in the code, and not<br>
the default in the configuration file.<br>
<br>
I agree that this is confusing, I think it's also blatantly wrong.  I'm<br>
not sure how to fix it, but I think it's some combination of your<br>
suggestions above and possibly having to introduce new option names.<br>
<div><div><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><div class="gmail_default" style="font-family:monospace,monospace">​</div><div class="gmail_default" style="font-family:monospace,monospace">I'm not sure why that's "blatantly wrong', this is a side effect of having multiple backends enabled, it's by design really.  Any option that is defined in driver.py needs to be set in the actual enabled-backend stanza IIRC.  This includes iscsi_helper, volume_clear etc.</div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace">Having the "global conf" settings intermixed with the backend sections caused a number of issues when we first started working on this.  That's part of why we require the "self.configuration" usage all over in the drivers.  Each driver instantiation is it's own independent entity.<br></div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace">I haven't looked at this for a long time, but if something has changed or I'm missing something my apologies.  We can certainly consider changing it, but because of the way we do multi-backend I'm not exactly sure how you would do this, or honestly why you would want to.</div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace">John</div><br></div></div>