Skip to content

Improved stretch_body_server --status and --restart UX by providing more info and removing tail logging.#99

Open
hello-robot-shehab wants to merge 1 commit into
mainfrom
feature/sbs-better-restart-ux
Open

Improved stretch_body_server --status and --restart UX by providing more info and removing tail logging.#99
hello-robot-shehab wants to merge 1 commit into
mainfrom
feature/sbs-better-restart-ux

Conversation

@hello-robot-shehab

@hello-robot-shehab hello-robot-shehab commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Changes in this PR:

  • --restart now restarts --launch if the daemon is not installed.
  • -- restart restarts the daemon if it is installed.
  • --daemon and --restart do not tail the log. However, the user can now pass --daemon --print or --restart --print to tail the log.
  • --status now shows the daemon status too.
  • --status now shows the user owning the server.

To test

It might be a good idea to pull in PR100's changes before doing this tests too.. If you see commands hanging, it's because of this.

  1. Run stretch_body_server --daemon and observe that the log is not tailed
  2. Run stretch_body_server --daemon --print and observe that the log is tailed
  3. Repeat 1 and 2 with --restart
  4. Run stretch_body_server --status and observe that the daemon status is printed alongside the normal status. Also the normal status has the owning user
  5. Run stretch_body_server --uninstall_daemon then stretch_body_server --restart and observe server is launched in the terminal (same as using --launch)
  6. Run stretch_body_server --launch in one terminal, and run stretch_body_server --restart in another terminal and observe the first instance getting killed, while a new one is started in the second terminal.

…wise restarts the daemon. --daemon and --restart do not tail the log. However, the user can now pass --daemon --print or --restart --print to tail the log. --status now shows the daemon status too. --status now shows the user owning the server.
@hello-robot-shehab
hello-robot-shehab requested review from hello-bharadwaj and hello-binit and removed request for hello-binit July 14, 2026 22:27
@hello-bharadwaj

hello-bharadwaj commented Jul 17, 2026

Copy link
Copy Markdown
Contributor
  1. Test 1: log not tailed. PASS
  2. Test 2: log tailed. PASS
  3. Test 3: restart behaves same. PASS
  4. Test 4: user printed with a tail log. daemon service status displayed. PASS
  5. Test 5: no daemon restart behaves like launch. PASS
  6. Test 6: throws an error. FAIL image

@hello-robot-shehab

hello-robot-shehab commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Ah, number 6 should pass with the fix in #100 I believe because it cleans up stop logic. Thank you for testing Brad!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants