• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

stacklok / toolhive / 24240677305
66%

Build:
DEFAULT BRANCH: main
Ran 10 Apr 2026 11:32AM UTC
Jobs 1
Files 618
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

10 Apr 2026 11:26AM UTC coverage: 65.181% (-0.02%) from 65.197%
24240677305

push

github

web-flow
Make transparent proxy health check parameters configurable (#4108)

* Make transparent proxy health check parameters configurable

The health check ping timeout, retry delay, and failure threshold
were hardcoded, making it impossible to tune resilience for
environments with transient network disruptions. Only the check
interval was configurable via TOOLHIVE_HEALTH_CHECK_INTERVAL.

Add three new environment variables following the same pattern:
TOOLHIVE_HEALTH_CHECK_PING_TIMEOUT, TOOLHIVE_HEALTH_CHECK_RETRY_DELAY,
and TOOLHIVE_HEALTH_CHECK_FAILURE_THRESHOLD. Raise the default failure
threshold from 3 to 5 to extend the tolerance window from ~40s to ~60s,
reducing false positives without significantly delaying detection of
genuinely dead servers.

Fixes #4084

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Retrigger CI

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Address review feedback: test hermeticity and doc accuracy

- Always set env vars in getter tests (including empty string for
  default cases) so tests are hermetic against ambient environment
- Clear all health check env vars in TestNewTransparentProxyDefaultValues
- Fix failure window formula in docs to match actual behavior
- Remove reference to not-yet-merged env var name in docs

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Address review feedback from amirejaz

- Fix failure window estimate: 70s → 60s to match the formula
- Document threshold=1 behavior (immediate shutdown, no retries)
- Note that pingTimeout > interval extends the failure window
- Remove pre-existing dead code: retryDelay == 0 guard was
  unreachable before this PR (constructor always set a positive
  value via DefaultHealthCheckRetryDelay)
- Fix test hermeticity: isolate env vars in option function test

Signed-off-by: Greg Katz <gkatz@indeed.com>

---------

Signed-off-by: Greg Katz <gkatz@indeed.com>

59 of 59 new or added lines in 1 file covered. (100.0%)

41 existing lines in 6 files now uncovered.

56870 of 87249 relevant lines covered (65.18%)

60.51 hits per line

Coverage Regressions

Lines Coverage ∆ File
14
74.44
-5.19% pkg/client/config.go
11
68.83
-14.29% pkg/client/discovery.go
8
23.56
-4.6% pkg/client/manager.go
3
79.38
-0.77% pkg/transport/proxy/httpsse/http_proxy.go
3
83.02
0.42% pkg/vmcp/composer/workflow_engine.go
2
94.77
-1.31% pkg/vmcp/composer/dag_executor.go
Jobs
ID Job ID Ran Files Coverage
1 24240677305.1 10 Apr 2026 11:32AM UTC 618
65.18
GitHub Action Run
Source Files on build 24240677305
  • Tree
  • List 618
  • Changed 9
  • Source Changed 1
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #24240677305
  • 63a2f626 on github
  • Prev Build on main (#24229293673)
  • Next Build on main (#24242575708)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc