fix: CLI - Add log for daemon start (#464)

bryphe-coder · web-flow · commit 8b12e470d9ce · 2022-03-16T13:39:57.000-07:00
While going through the manual CLI flow with some people on the team, there was some confusion with the `coder daemon` command - the fact there was no output to confirm that the daemon started: ``` coder ~/coder (bryphe/fix/daemon-log) $ dist/coder_linux_amd64/coder daemon ``` This PR just adds a simple log to confirm that the daemon has started: ``` coder ~/coder (bryphe/fix/daemon-log) $ dist/coder_linux_amd64/coder daemon 2022-03-16 17:57:20.358 [INFO] <daemon.go:53> daemon started {"url": "http://127.0.0.1:3000"} ``` Just throwing this out there. Feel free to reject if you have concerns about adding this @kylecarbs !
diff --git a/cli/daemon.go b/cli/daemon.go
@@ -50,6 +50,8 @@ func daemon() *cobra.Command {
 			}
 			defer listener.Close()
 
+			logger.Info(cmd.Context(), "daemon started", slog.F("url", accessURL.String()))
+
 			client := codersdk.New(accessURL)
 			daemonClose, err := newProvisionerDaemon(cmd.Context(), client, logger)
 			if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,8 @@ func daemon() *cobra.Command {`
`50`	`50`	`}`
`51`	`51`	`defer listener.Close()`
`52`	`52`
	`53`	`+ logger.Info(cmd.Context(), "daemon started", slog.F("url", accessURL.String()))`
	`54`	`+`
`53`	`55`	`client := codersdk.New(accessURL)`
`54`	`56`	`daemonClose, err := newProvisionerDaemon(cmd.Context(), client, logger)`
`55`	`57`	`if err != nil {`