<br><br><div class="gmail_quote">On Wed, Jul 25, 2012 at 12:23 PM, Robert Kukura <span dir="ltr"><<a href="mailto:rkukura@redhat.com" target="_blank">rkukura@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 07/24/2012 01:24 PM, Gary Kotton wrote:<br>
> Hi,<br>
</div><div class="im">> I have made a first patch - <a href="https://review.openstack.org/#/c/10234" target="_blank">https://review.openstack.org/#/c/10234</a>. It<br>
> was actually simpler than I had expected. By a fluke the change also<br>
> imported better logging for the agents. This should also move to the<br>
> openstack-common logging.<br>
> I still need to do some additional work:<br>
> 1. ensure agent tests are OK<br>
> 2. update devstack to support the configurations<br>
<br>
</div>I have begun reviewing patch set 5, and so far it looks good. I'd like<br>
to see this merged quickly to resolve the plugin unit test issues<br>
currently on master.<br>
<br>
But I do have one significant concern about the direction the quantum<br>
configuration has taken. My understanding is that with this patch,<br>
neither the quantum-server nor the agents are intended to load the<br>
plugin-specific .ini file unless a second --config-file option<br>
referencing it is passed in the command line. Unless I am missing<br>
something, this means that the quantum-server command line will now need<br>
to be plugin-specific in order for the plugin-specific .ini file to be<br>
loaded. Although this can be handled in devstack, it seems to be<br>
problematic for downstream systemd/init/upstart scripts which should<br>
generally not require customization once they are installed.<br>
<br>
I'm not very familiar with the common config mechanisms yet, but would<br>
it be possible for both the quantum-server/plugin and the agent to<br>
automatically load their .ini file(s), to have the quantum.conf file<br>
somehow include the .ini file(s), or to have a variable in quantum.conf<br>
specify which plugin-specific .ini file(s) need(s) to be loaded? This<br>
way, the quantum-server command line in the systemd/init/upstart scripts<br>
would only need to reference quantum.conf, and would not need to be<br>
customized based on which plugin is chosen.<br></blockquote><div><br></div><div>Jumping into this thread late, so please ignore this if the ideas I'm suggesting have already been considered and rejected. Bob, I tend to share your viewpoint on this. I do think its nice to be able to specify a particular config file for the plugin (currently, this is impossible, the plugin will just search a set of paths), however, I wonder if this could be achieved either by putting all plugin config into separate sections of the quantum.conf, or by having an entry of quantum.conf point to plugin config in some way. </div>
<div><br></div><div>dan</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-Bob<br>
<div class="im"><br>
> Thanks<br>
> Gary<br>
><br>
> On 07/24/2012 08:15 PM, Gary Kotton wrote:<br>
>> Hi,<br>
>> Please see my inline below.<br>
>> Thanks<br>
>> Gary<br>
>><br>
>> On 07/24/2012 07:47 PM, Salvatore Orlando wrote:<br>
>>> I +1 what Mark and Gary agreed on the IRC.<br>
>>> That's exactly what I meant by 'namespaces' on yesterday's meeting.<br>
>>><br>
>>> Cheers,<br>
>>> Salvatore<br>
>>><br>
>>> On 24 July 2012 16:30, Sumit Naiksatam (snaiksat) <<a href="mailto:snaiksat@cisco.com">snaiksat@cisco.com</a><br>
</div><div class="im">>>> <mailto:<a href="mailto:snaiksat@cisco.com">snaiksat@cisco.com</a>>> wrote:<br>
>>><br>
>>> Hi Gary,<br>
>>><br>
>>><br>
>>><br>
>>> Are you asking this just in the context of the RPC related<br>
>>> configurations, or also for moving all the existing configuration<br>
>>> from the plugin-specific configuration files (the ones under<br>
>>> etc/quantum/plugins) to a single configuration file?<br>
>>><br>
>><br>
>> The RPC was the motivation for this. This is information that is<br>
>> required by both the agents and the plugins. In addition to this due<br>
>> to the fact that is is from the openstack-common (which used the<br>
>> global cfg.CONF) complicates things.<br>
>><br>
>> There is still the option to use multiple files with input like:<br>
>> --config-file=/etc/quantum.conf --config-file=/etc/quantum/quantum.conf --config-dir=/etc/quantum/plugins.d<br>
>><br>
>> Thanks<br>
>> Gary<br>
>><br>
>><br>
>>><br>
>>><br>
>>> Moving plugin-independent common configuration to a common file<br>
>>> makes sense to me (including the agent and/or RPC related<br>
>>> configurations as discussed in the IRC meeting yesterday).<br>
>>> However, I am not in favor of moving all plugin-specific<br>
>>> configurations to a single/common file.<br>
>>><br>
>>><br>
>>><br>
>>> Thanks,<br>
>>><br>
>>> ~Sumit.<br>
>>><br>
>>><br>
>>><br>
</div>>>> *From:*Gary Kotton [mailto:<a href="mailto:gkotton@redhat.com">gkotton@redhat.com</a><br>
>>> <mailto:<a href="mailto:gkotton@redhat.com">gkotton@redhat.com</a>>]<br>
>>> *Sent:* Tuesday, July 24, 2012 6:03 AM<br>
>>> *To:* <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>
>>> <mailto:<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
>>> *Subject:* Re: [openstack-dev] [Quantum] Agents and configuration<br>
<div class="im">>>> files<br>
>>><br>
>>><br>
>>><br>
>>> Hi,<br>
>>> Can people please raise their concerns about moving all of the<br>
>>> plugin.ini files into quantum.conf.<br>
>>> This will solve us a number of problems. Please let me know if<br>
>>> you have any reservations.<br>
>>> Thanks<br>
>>> Gary<br>
>>><br>
>>> On 07/24/2012 09:17 AM, Gary Kotton wrote:<br>
>>><br>
>>> Hi,<br>
>>><br>
</div>>>> *Background*: The quantum service makes use of the configuration<br>
<div class="im">>>> file quantum.conf. In addition to this there is a common<br>
>>> configuration file shared between the plugin and the agent<br>
>>> (plugin.ini). The aforementioned files are loaded by the common<br>
>>> configuration files.<br>
>>><br>
</div>>>> *Current developments*: A number of current developments, for<br>
<div class="im">>>> example notifications (<a href="https://review.openstack.org/#/c/9835/" target="_blank">https://review.openstack.org/#/c/9835/</a>)<br>
>>> and scalable agents (<a href="https://review.openstack.org/#/c/9591/" target="_blank">https://review.openstack.org/#/c/9591/</a>)<br>
>>> require RPC configurations.<br>
>>><br>
>>> From the discussion last night the community the consensus is to<br>
>>> have the common configuration items that may be relevant to one<br>
>>> or more agent/plugin in the quantum configuration file. This<br>
>>> change will require the agents to also load the quantum.conf<br>
>>> configuration files.<br>
>>><br>
</div>>>> *Open issues*:<br>
<div class="im">>>> 1. I do not think that the common configuration supports mapping<br>
>>> of name spaces to configuration files.<br>
>>> 2. Configuration file loading. With the plugins the configuration<br>
>>> files are loaded in two stages. If possible it would be great<br>
>>> that the files could be loaded at once - this can be done by<br>
>>> listing the plugin/agent ini file(s) in the quantum.conf. When<br>
>>> this is loaded then the other files will be loaded.<br>
>>><br>
>>> Please let me know if you have any comments or objections. I<br>
>>> would like to move ahead and start to work on this.<br>
>>><br>
>>> Thanks<br>
>>> Gary<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>><br>
>>> OpenStack-dev mailing list<br>
>>><br>
</div>>>> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a> <mailto:<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a>><br>
<div class="im">>>><br>
>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> OpenStack-dev mailing list<br>
>>> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
</div>>>> <mailto:<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a>><br>
<div class="HOEnZb"><div class="h5">>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>>><br>
>>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> OpenStack-dev mailing list<br>
>> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <div>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a><br><div>twitter: danwendlandt<br>
~~~~~~~~~~~~~~~~~~~~~~~~~~~<br></div></div><br>