[openstack-dev] [ALL][PTLs] [Community goal] Toggle the debug option at runtime

ChangBo Guo glongwave at gmail.com
Mon Mar 26 12:15:53 UTC 2018


2018-03-22 16:12 GMT+08:00 Sławomir Kapłoński <slawek at kaplonski.pl>:

> Hi,
>
> I took care of implementation of [1] in Neutron and I have couple
> questions to about this goal.
>
> 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?
>

     Yes , let's the only  thing.  we need test if that if it works .

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

    good question, we indeed thought this question when we proposal  the
goal.  But It seems difficult to test  that consuming projects like Neutron
automatically.

>
> 3. Should we add any automatic tests for such change also? Any examples of
> such tests in other projects maybe?
>
     There is no example for tests now, we only have some unit tests  in
oslo.service .

>
> [1] https://governance.openstack.org/tc/goals/rocky/enable-
> mutable-configuration.html
> [2] https://docs.openstack.org/oslo.config/latest/reference/mutable.html
> [3] https://review.openstack.org/#/c/554259/
>
>> Best regards
> Slawek Kaplonski
> slawek at kaplonski.pl
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



-- 
ChangBo Guo(gcb)
Community Director @EasyStack
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180326/977c5f4c/attachment.html>


More information about the OpenStack-dev mailing list