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

Introduce experimental.externalLogCache helm values #3817

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

danail-branekov
Copy link
Member

Is there a related GitHub Issue?

#3754

What is this change about?

Introduce experimental.externalLogCache helm values

Tag your pair, your PM, and/or team

@uzabanov

danail-branekov and others added 2 commits February 17, 2025 14:18
* Introduce LogCacheGaugesCollector that uses the standard logcache
  client to get gauges.
* The log cache client is backed up by an http client that inserts the
  raw vaule of the stats request `Authorization` header when sending
  logcache requests.
* Add `RawAuthHeader` to the `authorization.Info` object so that the
  logache client can get it from the request context
* Stats handlers short-circuit to `localhost` to get metrics. This is
  temporary solution until we extract logache component into a separate
  deployment. Until then we configure the localhost client to not verify
  certificates intentionally.

Co-authored-by: Georgi Sabev <[email protected]>
Co-authored-by: Danail Branekov <[email protected]>
Co-authored-by: Yusmen Zabanov <[email protected]>

fixes #3755

Co-authored-by: Yusmen Zabanov <[email protected]>
Co-authored-by: Danail Branekov <[email protected]>
Co-authored-by: Yusmen Zabanov <[email protected]>
fixes #3754

Co-authored-by: Danail Branekov <[email protected]>
Co-authored-by: Yusmen Zabanov <[email protected]>
@danail-branekov danail-branekov enabled auto-merge (rebase) February 17, 2025 15:53
@danail-branekov danail-branekov merged commit 0477416 into main Feb 17, 2025
10 checks passed
@danail-branekov danail-branekov deleted the issues/3754-external-logache-configuration branch February 17, 2025 16:32
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.

2 participants