feat: display os info in the rollapp status output (#1093)

dymensionxyz · Oct 29, 2024 · 9bdac2e · 9bdac2e
1 parent a9340a8
commit 9bdac2e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/cmd/rollapp/start/start.go b/cmd/rollapp/start/start.go
@@ -6,6 +6,7 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"runtime"
 	"slices"
 	"strconv"
 	"strings"
@@ -203,7 +204,9 @@ func PrintOutput(
 	if withProcessInfo {
 		pterm.DefaultSection.WithIndentCharacter("💈").
 			Println("Process Info:")
-		fmt.Println("PID: ", pid)
+		fmt.Println("PID:", pid)
+		fmt.Println("OS:", runtime.GOOS)
+		fmt.Println("Architecture:", runtime.GOARCH)
 	}
 
 	if isHealthy {

diff --git a/cmd/rollapp/status/status.go b/cmd/rollapp/status/status.go
@@ -31,7 +31,6 @@ func Cmd() *cobra.Command {
 			pid, err := os.ReadFile(pidFilePath)
 			if err != nil {
 				fmt.Println("failed to read pid file:", err)
-				return
 			}
 
 			nodeID, err := dymint.GetNodeID(home)
@@ -42,7 +41,8 @@ func Cmd() *cobra.Command {
 
 			ok, msg := healthagent.IsEndpointHealthy("http://localhost:26657/health")
 			if !ok {
-				start.PrintOutput(rollerConfig, string(pid), true, false, false, false, nodeID)
+				// TODO: use options pattern, this is ugly af
+				start.PrintOutput(rollerConfig, string(pid), true, false, true, false, nodeID)
 				fmt.Println("Unhealthy Message: ", msg)
 				return
 			}