ListWorkloads shouldn't return workloads that can't be restarted
Open
@dmjb is already working on this.
Since Jul 8, 2025.
- Dominant language
- Go
- Stars
- 2.2k
- Forks
- 300
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 184
Description
Currently RestartWorkloads needs to know about the runconfig of a server in order to restart it, but ListWorkloads doesn't look at the runconfig.
If the toolhive directory is removed, the user gets into a bad state where ListWorkloads returns several stopped workloads, but they cannot be restarted by calling RestartWorkloads.
Instead, we should only return workloads that we have the runconfig for.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.