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

Updating collect-timeout will not change scrape_timeout configuration #207

Open
dashmage opened this issue Mar 27, 2024 · 1 comment
Open
Labels
bug Something isn't working

Comments

@dashmage
Copy link
Contributor

The scrape_timeout for Prometheus is set as part of the charm's init method when instantiating the CosAgentProvider class. This value is obtained from the charm's collect-timeout config.

But the problem is that once the charm is running and the user changes the value of collect-timeout, this will not change the scrape_timeout value again since the charm's init method won't be called. So Prometheus will always be configured with the default collect-timeout value of 10s.

@dashmage
Copy link
Contributor Author

Since we're updating the relation data when config change event occurs (due to the refresh_events configuration), it would look like the scrape_timeout would also be updated along with that.

This would require some manual verification to be 100% certain. Maybe we could have a new test for this as part of the COS integration tests?

@Pjack Pjack added the bug Something isn't working label Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants