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

Cleanup NFS metrics #834

Merged
merged 3 commits into from
Feb 21, 2018
Merged

Cleanup NFS metrics #834

merged 3 commits into from
Feb 21, 2018

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented Feb 19, 2018

  • Update nfs metric names to match nfsd.
  • Remove unneeded tcp label from TCP connections metric.
  • Remove unneeded v on nfsd metrics.
  • Enable all nfs v4 client metrics.
  • Remove nfs metric name overrides.

* Update `nfs` metric names to match `nfsd`.
* Remove uneeded `tcp` label from TCP connections metric.
* Remove uneeded `v` on `nfsd` metrics.
* Enable all `nfs` v4 client metrics.
* Remove `nfs` metric name overrides.
Copy link
Contributor

@mdlayher mdlayher left a comment

Choose a reason for hiding this comment

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

LGTM after typo fix.

prometheus.BuildFQName(namespace, nfsSubsystem, "net_reads_total"),
"Number of reads at the network layer.",
prometheus.BuildFQName(namespace, nfsSubsystem, "packets_total"),
"Total NFSd network packets (sent+recieved) by protocol type.",
Copy link
Contributor

Choose a reason for hiding this comment

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

*received

@SuperQ SuperQ force-pushed the superq/nfs_refactor branch from f12adb2 to 3fd3189 Compare February 20, 2018 21:49
@SuperQ SuperQ merged commit e0d54a5 into master Feb 21, 2018
@SuperQ SuperQ deleted the superq/nfs_refactor branch February 21, 2018 06:25
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Cleanup NFS metrics

* Update `nfs` metric names to match `nfsd`.
* Remove uneeded `tcp` label from TCP connections metric.
* Remove uneeded `v` on `nfsd` metrics.
* Enable all `nfs` v4 client metrics.
* Remove `nfs` metric name overrides.

* Add ppc64le fixture.

* Fix typo.
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