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

[mongo] Updated metric list to match V 3.0 #1798

Merged
merged 1 commit into from
Aug 12, 2015

Conversation

avaughan89
Copy link
Contributor

Also added notes that references the deprecated metrics in latest update.

List of metrics now matches the return of db.serverStatus() found here:

http://docs.mongodb.org/manual/reference/command/serverStatus/

@benmccann
Copy link
Contributor

+1. We really need both this change and #1823 to be merged

@remh remh added this to the 5.5.0 milestone Aug 10, 2015
@remh
Copy link
Contributor

remh commented Aug 10, 2015

@yannmh do you think it can join the 5.5.0 release ?

@yannmh
Copy link
Member

yannmh commented Aug 10, 2015

Yes, it sounds reasonable.

@benmccann
Copy link
Contributor

It'd be great to merge this PR. I added some additional metrics in a new PR - #1825

@yannmh yannmh self-assigned this Aug 12, 2015
@benmccann
Copy link
Contributor

Any reason not to merge this yet? I'd be happy to help test it if there's anything else you need to see done still

@avaughan89 avaughan89 changed the title Updated metric list to include the metrics added in V 3.0.0 of Mongo [mongo.py] Updated metric list to match V 3.0.0 of Mongo Aug 12, 2015
@avaughan89 avaughan89 changed the title [mongo.py] Updated metric list to match V 3.0.0 of Mongo [mongo] Updated metric list to match V 3.0 Aug 12, 2015
@yannmh
Copy link
Member

yannmh commented Aug 12, 2015

Let's merge it.

yannmh added a commit that referenced this pull request Aug 12, 2015
[mongo] Updated metric list to match V 3.0
@yannmh yannmh merged commit 3fb1653 into master Aug 12, 2015
@remh remh deleted the andrew/update_mongo_metrics branch May 3, 2016 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants