<div dir="ltr">+1<div>common.config should be global and general while agent.config should be local and related to the special back-end.</div><div>Maybe, we can add different prefix to the same option.</div><div><br></div><div>Germy</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 31, 2015 at 11:13 PM, Kevin Benton <span dir="ltr"><<a href="mailto:blak111@gmail.com" target="_blank">blak111@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">neutron.common.config should have general DHCP options that aren't specific to the reference DHCP agent. neutron.agent.dhcp.config should have all of the stuff specific to our agent and dnsmasq. </div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Aug 31, 2015 at 7:54 AM, Gal Sagie <span dir="ltr"><<a href="mailto:gal.sagie@gmail.com" target="_blank">gal.sagie@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Hello all,
<div><br></div><div>I went over the code and noticed that we have default DHCP configuration</div><div>both in neutron/common/config.py  (dhcp_lease_duration , dns_domain and dhcp_agent_notification)</div><div><br></div><div>But also we have it in neutron/agent/dhcp/config.py (DHCP_AGENT_OPTS, DHCP_OPTS)</div><div><br></div><div>I think we should consider merging them (especially the agent configuration)</div><div>into one place so it will be easier to find them.</div><div><br></div><div>I will add a bug on myself to address that, anyone know if this was done in purpose</div><div>for some reason, or anyone have other thoughts regarding this?</div><div><br></div><div>Thanks</div><span><font color="#888888"><div>Gal.</div><div><br></div></font></span></div>
<br></div></div>__________________________________________________________________________<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>
<br></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div>Kevin Benton</div></div>
</font></span></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>
<br></blockquote></div><br></div>