Skip to content

feat(load-balancer): support http timeout-idle annotation - #1301

Open
lukasmetzner wants to merge 4 commits into
mainfrom
feat/lb-support-timeout-idle
Open

feat(load-balancer): support http timeout-idle annotation#1301
lukasmetzner wants to merge 4 commits into
mainfrom
feat/lb-support-timeout-idle

Conversation

@lukasmetzner

@lukasmetzner lukasmetzner commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Add the load-balancer.hetzner.cloud/http-timeout-idle annotation to
configure the idle timeout for the client and server side of an HTTP/HTTPS
Load Balancer service.

It is documented as the new duration type (a Go duration string, e.g. 30s).

Closes #1233

feat(load-balancer): support http timeout-idle annotation
feat(load-balancer): improve error logs on invalid input

@lukasmetzner
lukasmetzner requested a review from a team as a code owner July 23, 2026 11:50
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.75000% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.85%. Comparing base (7e1f686) to head (c830d90).

Files with missing lines Patch % Lines
internal/hcops/load_balancer.go 50.00% 9 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1301      +/-   ##
==========================================
+ Coverage   71.63%   71.85%   +0.22%     
==========================================
  Files          26       27       +1     
  Lines        2778     2800      +22     
==========================================
+ Hits         1990     2012      +22     
  Misses        613      613              
  Partials      175      175              
Flag Coverage Δ
e2e 44.52% <15.62%> (-0.40%) ⬇️
unit 68.21% <68.75%> (+0.25%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lukasmetzner
lukasmetzner marked this pull request as draft July 23, 2026 11:51
Comment thread docs/reference/load_balancer_annotations.md Outdated
@lukasmetzner
lukasmetzner marked this pull request as ready for review July 28, 2026 15:18
Comment thread internal/hcops/errors.go

// withInvalidInputFields adds the validation errors of an 'invalid_input' API
// error to the error message.
func withInvalidInputFields(err error) error {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately we can't use the hcloud helper here as long as we use klog.

Add the `load-balancer.hetzner.cloud/http-timeout-idle` annotation to
configure the idle timeout for the client and server side of an HTTP/HTTPS
Load Balancer service.

It is documented as the new `duration` type (a Go duration string, e.g. `30s`).

Closes #1233
@lukasmetzner
lukasmetzner force-pushed the feat/lb-support-timeout-idle branch from 32cb59f to c830d90 Compare July 28, 2026 15:45
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.

Support timeout_idle in Load Balancer implementation

2 participants