Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: disable keep-alives for internal traffic #3267

Merged
merged 3 commits into from
Sep 13, 2023

Conversation

swiatekm
Copy link

Disable HTTP keep-alives for network traffic from collectors to metadata enrichment. This should make the Kubernetes iptables-based load balancing work better at the cost of increased network traffic.

Checklist

  • Changelog updated or skip changelog label added
  • Documentation updated
  • Template tests added for new features
  • Integration tests added or modified for major features

@swiatekm swiatekm requested a review from a team as a code owner September 13, 2023 10:51
@andrzej-stencel
Copy link
Contributor

Note for future us: This change applies to both logs-collector and metrics-collector. It does not apply to Prometheus as metrics collector, which still needs the remote write proxy for proper load balancing.

@swiatekm swiatekm merged commit d7f7691 into main Sep 13, 2023
43 checks passed
@swiatekm swiatekm deleted the fix/no-persistent-connections branch September 13, 2023 11:13
@swiatekm swiatekm restored the fix/no-persistent-connections branch September 13, 2023 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants