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

feat(k8s-infra): introduce prometheus scraper using pod annotations #455

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

prashant-shahi
Copy link
Member

@prashant-shahi prashant-shahi commented Jun 21, 2024

Fixes #445
Fixes #196

This has been requested by lots of OSS and cloud users who wished to scrape metrics from various cluster and send to SigNoz running in different cluster.

This will be disabled in main chart by default where it is imported as dependency chart. Since, this would be redundant to that of metrics scarper in otelCollectorMetrics. That would be required when people choose to not enable the k8s-infra dependency chart itself but require the metrics scraper.

Signed-off-by: Prashant Shahi prashant@signoz.io

Signed-off-by: Prashant Shahi <prashant@signoz.io>
@prashant-shahi prashant-shahi marked this pull request as ready for review June 21, 2024 09:20
@prashant-shahi prashant-shahi requested a review from a team as a code owner June 21, 2024 09:20
Signed-off-by: Prashant Shahi <prashant@signoz.io>
Signed-off-by: Prashant Shahi <prashant@signoz.io>
…s by default

Signed-off-by: Prashant Shahi <prashant@signoz.io>
@srikanthccv
Copy link
Member

One issue with this will only work for minimal workloads. The pod will quickly face OOM issues for non-trivial workloads. When you get some time, please check how otel helm charts enable the Prom target allocator.

@prashant-shahi prashant-shahi marked this pull request as draft November 4, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants