commit f433400d0bcb90558c487377bee5fb2fb1442c40
author Austin Schuh <austin.schuh@bluerivertech.com> Sat Oct 16 14:19:51 2021 -0700
committer Austin Schuh <austin.schuh@bluerivertech.com> Sat Oct 16 14:52:21 2021 -0700
tree 3124f23e425dcc10a7981dc81e40c302fb1421a2
parent d78f9782f5691b7959b916f5c082159367ca1e03

Change printed info from aos_starter status

uptime isn't relevant when stopped, so don't print it.  Also, put PID in
there and shorten the titles.  This makes it a lot easier to get what we
want out.

Change-Id: Id6562c5100c9bfb2dc2d2f112fb64ae35e4bd726
Signed-off-by: Austin Schuh <austin.schuh@bluerivertech.com>

aos/starter/starter_cmd.cc

diff --git a/aos/starter/starter_cmd.cc b/aos/starter/starter_cmd.cc
index 95967bb..5f67c4b 100644
--- a/aos/starter/starter_cmd.cc
+++ b/aos/starter/starter_cmd.cc
@@ -24,8 +24,7 @@
                         {"restart", aos::starter::Command::RESTART}};
 
 void PrintKey() {
-  absl::PrintF("%-30s %-30s %s\n\n", "Name", "Time since last started",
-               "State");
+  absl::PrintF("%-30s %-8s %-6s %-9s\n", "Name", "State", "PID", "Uptime");
 }
 
 void PrintApplicationStatus(const aos::starter::ApplicationStatus *app_status,
@@ -34,9 +33,14 @@
       chrono::nanoseconds(app_status->last_start_time()));
   const auto time_running =
       chrono::duration_cast<chrono::seconds>(time - last_start_time);
-  absl::PrintF("%-30s %-30s %s\n", app_status->name()->string_view(),
-               std::to_string(time_running.count()) + 's',
-               aos::starter::EnumNameState(app_status->state()));
+  if (app_status->state() == aos::starter::State::STOPPED) {
+    absl::PrintF("%-30s %-8s\n", app_status->name()->string_view(),
+                 aos::starter::EnumNameState(app_status->state()));
+  } else {
+    absl::PrintF("%-30s %-8s %-6d %-9ds\n", app_status->name()->string_view(),
+                 aos::starter::EnumNameState(app_status->state()),
+                 app_status->pid(), time_running.count());
+  }
 }
 
 bool GetStarterStatus(int argc, char **argv, const aos::Configuration *config) {