-
Notifications
You must be signed in to change notification settings - Fork 814
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
[mongo] Add WiredTiger metrics #1825
Conversation
2296edd
to
5b04379
Compare
It looks great, thanks a lot @benmccann ! We froze the 5.5.0 agent state in order to focus on polishing the release, and fixing the last bugs. As you are introducing a lot of new metrics, we think it'd be more reasonable to schedule it for the 5.6.0 agent release. We'll likely add more tests, in the meantime, to assert that all metrics are properly collected. Of course, you can still run this check as a custom one to replace the default, until everything is released. |
Happy to help. I understand waiting, though I certainly wouldn't mind getting this in 5.5.0 either, so if there's anything I can do as far as testing goes or anything else that would give you confidence in the change to get it in now, I'd be happy to do so. It doesn't seem particular high risk compared to other changed going in, so happy to help vet it. I've already deployed the change to our own servers and it's working well there |
I'm so in love with Datadog right now. Just got weeks of persistent outages figured out due to the data we gathered with it. Some of the data in this PR was a big part of that, so really hoping we can get it in :-) How long do you have between releases typically? |
Thank you so much for your support @benmccann, it's much appreciated. I'll transmit your words to the team 😃 We typically try to keep a 6-7 weeks release schedule. We mainly focus on our new features during the first 3-4 weeks, then 'freeze' the release to polish and bring bugfixes. As your changes are adding a lot of metrics, we think it's' more reasonable to postpone it to the 5.6.0 agent release. It'd give us some time to assess the changes. @irabinovitch any thoughts ? Best, |
Ok, I'll trim this down to the most useful. Posting the whole list here in case we want to revisit in the future
|
@yannmh I toned this down a lot and only added the really useful ones. Many of the others seem to be always 0 or monotonically increasing counters in the form of # of times operation x has been done since the server was started. If Datadog can transform that such that each minute it looks at what the increase is from the last datapoint into operations per minute then that may be more useful. But as it stands now most of them don't provide value in this form. |
780e286
to
7cc307d
Compare
Any thoughts about getting this in now that 5.5 has been released? I'd like to get it in while it's still somewhat fresh on my mind |
This adds new metrics from MongoDB 3.0 that aren't already included in #1798