<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">2018-03-22 16:12 GMT+08:00 Sławomir Kapłoński <span dir="ltr"><<a href="mailto:slawek@kaplonski.pl" target="_blank">slawek@kaplonski.pl</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I took care of implementation of [1] in Neutron and I have couple questions to about this goal.<br>
<br>
1. Should we only change "restart_method" to mutate as is described in [2] ? I did already something like that in [3] - is it what is expected?<br></blockquote><div><br></div><div>     Yes , let's the only  thing.  we need test if that if it works .<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2. How I can check if this change is fine and config option are mutable exactly? For now when I change any config option for any of neutron agents and send SIGHUP to it it is in fact "restarted" and config is reloaded even with this old restart method.<br></blockquote><div> <br></div><div>    good question, we indeed thought this question when we proposal  the goal.  But It seems difficult to test  that consuming projects like Neutron automatically. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
3. Should we add any automatic tests for such change also? Any examples of such tests in other projects maybe?<br></blockquote><div>     There is no example for tests now, we only have some unit tests  in oslo.service .<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
[1] <a href="https://governance.openstack.org/tc/goals/rocky/enable-mutable-configuration.html" rel="noreferrer" target="_blank">https://governance.openstack.<wbr>org/tc/goals/rocky/enable-<wbr>mutable-configuration.html</a><br>
[2] <a href="https://docs.openstack.org/oslo.config/latest/reference/mutable.html" rel="noreferrer" target="_blank">https://docs.openstack.org/<wbr>oslo.config/latest/reference/<wbr>mutable.html</a><br>
[3] <a href="https://review.openstack.org/#/c/554259/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/554259/</a><br>
<br>
—<br>
Best regards<br>
Slawek Kaplonski<br>
<a href="mailto:slawek@kaplonski.pl">slawek@kaplonski.pl</a><br>
<br>
<br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>ChangBo Guo(gcb)<br></div><div>Community Director @EasyStack<br></div></div></div></div></div>
</div></div>