[ovs-dev] [PATCH 2/2] ovs-sandbox: Fix ovs-appctl for ovn-northd and ovn-controller.

Ben Pfaff blp at ovn.org
Fri Jul 6 21:04:55 UTC 2018


On Sun, Jul 01, 2018 at 06:23:01PM -0700, Justin Pettit wrote:
> Commits 1e8eeb66db2e7 ("ovs-sandbox: Support starting multiple
> ovn-northds.") and 047458de40391 ("ovs-sandbox: Add option to support
> multiple ovn-controllers.") allowed starting multiple instances of
> ovn-northd and ovn-controller, respectively.  It did this by assigning a
> sequence number to to the pidfile name.  Unfortunately, this breaks the
> method ovs-appctl uses to determine to which process it should connect.
> This commit changes the behavior so that a sequence number is not added
> to the first instance, so ovs-appctl will connect to that be default.
> 
> This commit also uses the same convention for naming the log file.
> 
> Signed-off-by: Justin Pettit <jpettit at ovn.org>

Acked-by: Ben Pfaff <blp at ovn.org>


More information about the dev mailing list