[ovs-dev] [PATCH v2] utilities/ovs-ctl.in: Only add_managers with vswitchd

[Ben Pfaff]

On Fri, May 20, 2016 at 10:50:46AM -0400, Aaron Conole wrote:
> The ovs-ctl script was changed recently to have per-service start/stop
> control. However, when that change was made the add_managers() call was
> overlooked. This results in calls to `ovs-ctl --no-ovs-vswitchd start`
> telling the ovsdb-server to connect to the remote controllers.
> 
> The fix presented will defer signaling to remote managers until the
> following are both true:
> 1. At least one of OVSDB_SERVER or OVS_VSWITCHD was told to start
> 2. Both daemons are running.
> 
> Fixes: 7fc28c50c012 ("ovs-ctl: Allow selective start for db and switch")
> Signed-off-by: Aaron Conole <aconole at redhat.com>
> ---
> v1->v2:
> - refactored to only start when both daemons are up, and only when a start
>   or restart command includes at least one of the daemons.

Applied, thanks!