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

Enable metrics for ratelimiter along with the other metrics #2439

Open
sgayangi opened this issue Aug 28, 2024 · 0 comments · May be fixed by #2648
Open

Enable metrics for ratelimiter along with the other metrics #2439

sgayangi opened this issue Aug 28, 2024 · 0 comments · May be fixed by #2648
Assignees

Comments

@sgayangi
Copy link
Contributor

Problem

Currently the Prometheus metrics are enabled using the metrics.enabled flag in the values.yaml, but there needs to be a separate statsd configuration for enabling metrics for ratelimiter. The ratelimiter metrics should also be enabled via the metrics.enabled flag without needing a separate config for it.

Solution

Set default values for the statsd configuration and overwrite those values if a statsd config is provided in the values.yaml. The resource limits for the statsd pod also need to be set.

Affected Component

None

Version

1.1.0

Implementation

No response

Related Issues

No response

Suggested Labels

No response

@sgayangi sgayangi self-assigned this Aug 28, 2024
@sgayangi sgayangi moved this from Backlog to In-Progress in APK Development Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In-Progress
Development

Successfully merging a pull request may close this issue.

1 participant