stacklok / toolhive / 20307224613

51%

Build Type

push

github

Committed by web-flow

Commit Message

fix: update workload status when health check fails for remote MCP servers (#3077)

* Update workload status on health check failure

When a remote MCP server becomes unreachable, the health check fails
and the proxy is stopped. However, the workload status was not being
updated, causing thv list to report the server as running even though
nothing was listening on the port.

This fix adds a callback mechanism that notifies the runner when a
health check fails, allowing it to update the workload status to
unhealthy.

Closes #3079

Signed-off-by: Frederic Le Feurmou <fredericlefeurmou@gmail.com>

* Update pkg/transport/types/transport.go

Removed the unused `OnHealthCheckFailed HealthCheckFailedCallback` code

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: Frederic Le Feurmou <fredericlefeurmou@gmail.com>
Co-authored-by: Frédéric LE FEURMOU <fredericlefeurmou@users.noreply.github.com>
Co-authored-by: Muhammad Amir Ejaz <amir@stacklok.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Run Details

11 of 33 new or added lines in 4 files covered. (33.33%)

3 existing lines in 1 file now uncovered.

27907 of 55587 relevant lines covered (50.2%)

57.08 hits per line