[ovs-dev] [PATCH] rhel: Add support for "systemctl reload openvswitch"

Russell Bryant russell at ovn.org
Tue Aug 8 18:05:08 UTC 2017


On Tue, Aug 8, 2017 at 1:32 PM, Guru Shetty <guru at ovn.org> wrote:
> On 8 August 2017 at 07:42, Timothy Redaelli <tredaelli at redhat.com> wrote:
>
>> The reload procedure will trigger a script that saves the flows and tlv
>> maps then it restarts ovsdb-server, it stops ovs-vswitchd, it sets
>> other_config:flow-restore-wait=true (to wait till flow restore is
>> finished), it starts ovs-vswitchd, it restore the backupped flows/tlv
>> maps and it removes other_config:flow-restore-wait=true (logic mostly
>> ripped
>> from ovs-ctl).
>>
>> It uses systemctl with --job-mode=ignore-dependencies to restart
>> ovsdb-server
>> and stop and start ovs-vswitchd in order to avoid systemd to restart the
>> other components due to dependencies (as explained in
>> rhel/README.RHEL.rst).
>>
>> It also uses --bundle, when available, in order to minimize the downtime.
>>
>
> The core script can probably be added to ovs-ctl, reuse some functions and
> then you can call it? This way, other platforms can benefit too.

Sounds like a good suggestion.

What would you think about updating the stop/start behavior to do
this, as well?  It looks like this makes "reload" actually just an
"intelligent restart".  systemctl has a "restart" command too, which
looks like it just does stop/start, and it'd be nice to make that work
better too.

-- 
Russell Bryant


More information about the dev mailing list