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.

Sign up for GitHub

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

Jump to bottom

[inputs.ceph] added the read and write op per second fields #5210

Merged
merged 5 commits into from
Jan 2, 2019
Merged

[inputs.ceph] added the read and write op per second fields #5210

merged 5 commits into from
Jan 2, 2019

Conversation

svenwiltink
Copy link
Contributor

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@svenwiltink
Copy link
Contributor Author

the op_per_sec field is no longer exported by ceph, but I am reluctant to remove it as older version might still do it.

@svenwiltink svenwiltink changed the title added the read and write op per second fields [inputs.ceph] added the read and write op per second fields Jan 2, 2019
@svenwiltink
Copy link
Contributor Author

I will also add it to the osd pool stats output

@svenwiltink
Copy link
Contributor Author

requires #5213

danielnelson
danielnelson suggested changes Jan 2, 2019
View reviewed changes
plugins/inputs/ceph/ceph.go Show resolved Hide resolved
plugins/inputs/ceph/README.md Outdated Show resolved Hide resolved
@svenwiltink
Copy link
Contributor Author

svenwiltink commented Jan 2, 2019 •
edited
Loading

The commit that changed this was ceph/ceph@c01a314. This belongs to version v10, so I will update the readme accordingly

@danielnelson danielnelson added this to the 1.10.0 milestone Jan 2, 2019
@danielnelson danielnelson added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jan 2, 2019
@danielnelson danielnelson merged commit bf7a426 into influxdata:master Jan 2, 2019
@svenwiltink svenwiltink deleted the ceph-read-write-ops branch January 4, 2019 13:21
trevorwhitney pushed a commit to trevorwhitney/telegraf that referenced this pull request Feb 14, 2019
@svenwiltink
Add read and write op per second fields (influxdata#5210)
516d41d
danielnelson pushed a commit that referenced this pull request Feb 25, 2019
@svenwiltink @danielnelson
Add read and write op per second fields (#5210)
ee31f0a 
(cherry picked from commit bf7a426)
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@svenwiltink @otherpirate
Add read and write op per second fields (influxdata#5210)
cb53cc4
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@svenwiltink @otherpirate
Add read and write op per second fields (influxdata#5210)
e90a67a
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
@svenwiltink
Add read and write op per second fields (influxdata#5210)
af2e633
bitcharmer pushed a commit to bitcharmer/telegraf that referenced this pull request Oct 18, 2019
@svenwiltink @bitcharmer
Add read and write op per second fields (influxdata#5210)
14fdff9
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@svenwiltink
Add read and write op per second fields (influxdata#5210)
73a65eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielnelson danielnelson danielnelson approved these changes

Assignees
No one assigned
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
1.10.0
Development

Successfully merging this pull request may close these issues.
2 participants
@svenwiltink @danielnelson