-
Notifications
You must be signed in to change notification settings - Fork 42
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
Brupop is Not Sending Metrics to Datadog #644
Comments
Hello. Can you share which version of Brupop you are attempting to configure? The ability to create a ServiceMonitor via helm values was recently released in Brupop 1.4.0 You can see the relevant helm values here |
I am using version 1.3.0, but I have installed ServiceMonitor already, is it only work with version 1.4.0 ? openmetrics_endpoint": "http://%%host%%:8080/metrics An annotation is missing in the controller template, can I create a PR for the same? |
It should work with I've noticed a section in the DataDog documentation mentioning autodiscovery via Prometheus annotations. Is that something that could possibly work for you?
I think rather than adding additional annotations here, we probably instead want to expose the annotations field wholesale as we've done for e.g. placement settings. For compatibility, we'd want the default to maintain the existing prometheus annotations. A PR for that would be welcome. I cut #646 as a separate issue for that, as I think it's a worthwhile feature even if you can resolve your issue using the DataDog autodiscovery feature. |
Metrics are coming to the data dog and working fine. Let me know if I can add the datadog integration block in "README", So that people get benefit from it. I have one general question. I deployed Brupop in my EKS cluster with three environments: Dev, Test, and Prod. What I observed is that as soon as a new BR image is available in the AWS public repository, my Brupop operator starts upgrading the instances. Is it possible to hold these changes to be rollout in Prod for the time being to ensure safety? |
That could be useful, although the README is pretty expansive. Mind sharing the configuration here? We could discuss adding it to the README, but it might also help if this thread turns up in web searches. It would also be helpful to inform the design for #646.
What this means is that you can apply a version lock to all of your instances. Brupop will ensure that these instances remain at the locked version. You would then need to instrument a system to update If you implement this, you may wish to also take a look at the ignore waves setting, which impacts when your host is able to see an update. |
Despite making the necessary changes to expose OpenMetrics from the brupop-controller and configuring the Datadog Agent to scrape these metrics, we are not seeing any metrics data in Datadog.
Adding annotation is not possible with provided brupop helm chart, I added the annotation to my container manually [FYI]
My files looks like -
Please let me know if I am missing anything on my end or sending metrics is not possible via open metrics to datadog ?
The text was updated successfully, but these errors were encountered: