diff mbox series

[ovs-dev,1/2] ovs-sandbox: Use different log file names for ovn-controllers.

Message ID 20180702012301.36193-1-jpettit@ovn.org
State Accepted
Headers show
Series [ovs-dev,1/2] ovs-sandbox: Use different log file names for ovn-controllers. | expand

Commit Message

Justin Pettit July 2, 2018, 1:23 a.m. UTC
Commit 047458de40391 ("ovs-sandbox: Add option to support multiple
ovn-controllers.") allowed creating multiple instances of
ovn-controller.  However, all instances would use the same log file
name.  This commit uses the sequence number to name the log file.

Signed-off-by: Justin Pettit <jpettit@ovn.org>
---
 tutorial/ovs-sandbox | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Ben Pfaff July 6, 2018, 9:04 p.m. UTC | #1
On Sun, Jul 01, 2018 at 06:23:00PM -0700, Justin Pettit wrote:
> Commit 047458de40391 ("ovs-sandbox: Add option to support multiple
> ovn-controllers.") allowed creating multiple instances of
> ovn-controller.  However, all instances would use the same log file
> name.  This commit uses the sequence number to name the log file.
> 
> Signed-off-by: Justin Pettit <jpettit@ovn.org>

Acked-by: Ben Pfaff <blp@ovn.org>
diff mbox series

Patch

diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox
index 781bedbc6e35..3fab1c0ccc98 100755
--- a/tutorial/ovs-sandbox
+++ b/tutorial/ovs-sandbox
@@ -537,7 +537,8 @@  if $ovn; then
     for i in $(seq $n_controllers); do
         rungdb $gdb_ovn_controller $gdb_ovn_controller_ex ovn-controller \
                $OVN_CTRLR_PKI --detach --no-chdir -vsyslog:off \
-               --pidfile=ovn-controller$i.pid -vconsole:off --log-file
+               --log-file=ovn-controller$i.log \
+               --pidfile=ovn-controller$i.pid -vconsole:off
     done
     rungdb $gdb_ovn_controller_vtep $gdb_ovn_controller_vtep_ex \
         ovn-controller-vtep --detach --no-chdir --pidfile -vconsole:off \