make default_cols consistent to avoid ci surprises

johnstcn · johnstcn · commit 0f3254a64640 · 2023-06-13T18:23:38.000+01:00
diff --git a/cli/stat.go b/cli/stat.go
@@ -13,11 +13,7 @@ import (
 
 func (*RootCmd) stat() *clibase.Cmd {
 	fs := afero.NewReadOnlyFs(afero.NewOsFs())
-	defaultCols := []string{"host_cpu", "host_memory", "home_disk"}
-	if ok, err := clistat.IsContainerized(fs); err == nil && ok {
-		// If running in a container, we assume that users want to see these first. Prepend.
-		defaultCols = append([]string{"container_cpu", "container_memory"}, defaultCols...)
-	}
+	defaultCols := []string{"host_cpu", "host_memory", "home_disk", "container_cpu", "container_memory"}
 	formatter := cliui.NewOutputFormatter(
 		cliui.TableFormat([]statsRow{}, defaultCols),
 		cliui.JSONFormat(),
diff --git a/docs/cli/stat.md b/docs/cli/stat.md
@@ -17,7 +17,7 @@ coder stat [flags]
 |         |                                                                            |
 | ------- | -------------------------------------------------------------------------- |
 | Type    | <code>string-array</code>                                                  |
-| Default | <code>container_cpu,container_memory,host_cpu,host_memory,home_disk</code> |
+| Default | <code>host_cpu,host_memory,home_disk,container_cpu,container_memory</code> |
 
 Columns to display in table output. Available columns: host cpu, host memory, home disk, container cpu, container memory.
 
