1
50%
main: 50%

Ran 24 Oct 2025 04:55PM UTC

Files 298

Run time 10s

Badge

Embed ▾

Committed 24 Oct 2025 04:49PM UTC coverage: 47.271% (+0.1%) from 47.142%

Job # 18786402642.1

Build Type

push

github

Committed by

web-flow

Commit Message

fix: prevent zombie supervisor processes on restart (#2306)

Fixes #2274

When running 'thv restart' on an already-running workload, the command
would return early with 'Container is already running' but would NOT
stop the old supervisor process. This caused supervisor processes to
accumulate over time.

Changes:
- restartContainerWorkload now always stops supervisor + container when
  workload is already running
- restartRemoteWorkload applies the same logic for remote workloads
- Handles edge cases: dead supervisor, dead container, or both

The fix ensures restart always performs a proper stop→start cycle:
1. Stops old supervisor (kills proxy process)
2. Stops container if running
3. Cleans up client configurations
4. Starts fresh supervisor + container

System is resilient - stopProcess gracefully handles missing PIDs,
and supervisor auto-exits when container dies.

Added comprehensive unit tests and E2E test to verify no process
accumulation occurs on multiple restarts.

Run Details

18883 of 39946 relevant lines covered (47.27%)

15.57 hits per line

stacklok / toolhive / 18786402642 / 1
50%
main: 50%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 18786402642.1

stacklok / toolhive / 18786402642 / 1 50% main: 50%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 18786402642.1

stacklok / toolhive / 18786402642 / 1
50%
main: 50%

README BADGES
x