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] Add WiredTiger metrics #1980

Closed
wants to merge 3 commits into from
Closed

[mongo] Add WiredTiger metrics #1980

wants to merge 3 commits into from

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Oct 16, 2015

Rebase of #1825 on #1979, to collect WiredTiger metrics behind a additional_metrics option.

Thanks again @benmccann !

@yannmh yannmh added this to the 5.6.0 milestone Oct 16, 2015
@yannmh yannmh changed the title Yann/wiredtiger option [mongo] Add WiredTiger metrics Oct 16, 2015
@yannmh yannmh self-assigned this Oct 16, 2015
@yannmh yannmh force-pushed the yann/wiredtiger-option branch from 0433c67 to 7e7011a Compare October 16, 2015 12:03
"tcmalloc.tcmalloc.transfer_cache_free_bytes": GAUGE,
}

WIREDTIGER_METRICS = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about these metric names...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The metric names were created by MongoDB. We're simply collecting the metrics that they export. I think it'd be really confusing to remap them to new names because then for every metric we have to go back and figure out what the MongoDB name was to really understand what it's doing.

@hkaj
Copy link
Member

hkaj commented Oct 16, 2015

@yannmh let's talk about the metric names. Everything else is 👍

@yannmh yannmh force-pushed the yann/wiredtiger-option branch from 7e7011a to 33b328c Compare October 29, 2015 22:47
Scott Bessler and others added 3 commits October 30, 2015 17:03
By default, the check collects a sample of metrics from MongoDB.
The (optional) `additional_metrics` parameter instructs the check to
collect additional
metrics on specific topics.
Available options are:
* `durability` - Journaling-related operations and performance
* `locks` - Locks
* `metrics.commands` - Use of database commands
* `tcmalloc` -  TCMalloc memory allocator
[[email protected]] rebased on #1979 → `collect_wiredtiger_metrics`
option.
@yannmh
Copy link
Member Author

yannmh commented Nov 2, 2015

Closing in favor of #2020

@yannmh yannmh closed this Nov 2, 2015
@yannmh yannmh deleted the yann/wiredtiger-option branch November 3, 2015 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants