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

Add XFS block mapping and block map B-tree stats #575

Merged
merged 1 commit into from
Jul 7, 2017
Merged

Add XFS block mapping and block map B-tree stats #575

merged 1 commit into from
Jul 7, 2017

Conversation

mdlayher
Copy link
Contributor

@mdlayher mdlayher commented May 3, 2017

A continuation of my previous XFS collector work. Many stats left, but I'll keep adding them in chunks to make it more manageable.

{
name: "block_mapping_extent_list_compares_total",
desc: "Number of extent list compares for a filesystem.",
value: float64(0),
Copy link
Member

Choose a reason for hiding this comment

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

Err, why add a counter if it's always going to be 0?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops. :) Totally forgot to fill these in when I was copying/pasting slice elements. Will fix.

@mdlayher
Copy link
Contributor Author

mdlayher commented May 3, 2017

@SuperQ updated.

Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM

@SuperQ SuperQ merged commit 6e82fd1 into prometheus:master Jul 7, 2017
@mdlayher mdlayher deleted the xfs-add branch July 7, 2017 20:00
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
tamcore pushed a commit to gitgrave/node_exporter that referenced this pull request Oct 22, 2024
Enable CI testing of Go 1.21.
* Bump Go modules.

Signed-off-by: SuperQ <superq@gmail.com>
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.

2 participants