From 1b190930a62ff5e9e7165ce7eeb3c44d13c340bb Mon Sep 17 00:00:00 2001 From: Brent Baude Date: Thu, 18 Jan 2024 09:34:20 -0600 Subject: [PATCH] Add log-file to proxy builder The original PR only enabled log-file on the command line. This PR adds log-file to gvproxy's builder (for consumption) and adds trivial test. Signed-off-by: Brent Baude --- pkg/types/gvproxy_command.go | 8 ++++++++ test/basic_test.go | 2 ++ 2 files changed, 10 insertions(+) diff --git a/pkg/types/gvproxy_command.go b/pkg/types/gvproxy_command.go index 8aa72e7ea..13df0982b 100644 --- a/pkg/types/gvproxy_command.go +++ b/pkg/types/gvproxy_command.go @@ -22,6 +22,9 @@ type GvproxyCommand struct { // Map of different sockets provided by user (socket-type flag:socket) sockets map[string]string + // Logfile where gvproxy should redirect logs + LogFile string + // File where gvproxy's pid is stored PidFile string @@ -179,6 +182,11 @@ func (c *GvproxyCommand) ToCmdline() []string { args = append(args, "-pid-file", c.PidFile) } + // log-file + if c.LogFile != "" { + args = append(args, "-log-file", c.LogFile) + } + return args } diff --git a/test/basic_test.go b/test/basic_test.go index ad96b9007..9d3faf09b 100644 --- a/test/basic_test.go +++ b/test/basic_test.go @@ -204,6 +204,7 @@ var _ = ginkgo.Describe("command-line format", func() { command.Debug = true command.AddQemuSocket("tcp://0.0.0.0:1234") command.PidFile = "~/gv-pidfile.txt" + command.LogFile = "~/gv.log" command.AddForwardUser("demouser") cmd := command.ToCmdline() @@ -215,6 +216,7 @@ var _ = ginkgo.Describe("command-line format", func() { "-listen-qemu", "tcp://0.0.0.0:1234", "-forward-user", "demouser", "-pid-file", "~/gv-pidfile.txt", + "-log-file", "~/gv.log", })) }) })