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

Justin Pettit jpettit at ovn.org
Fri Jul 6 22:10:02 UTC 2018


> On Jul 6, 2018, at 5:04 PM, Ben Pfaff <blp at ovn.org> wrote:
> 
> 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>

Thanks.  I pushed the series to master.

--Justin




More information about the dev mailing list