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 <[email protected]>
containers · Jan 18, 2024 · 1b19093 · 1b19093
1 parent 012bc90
commit 1b19093
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git 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
@@ -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",
 		}))
 	})
 })