search

rochaporto / collectd-ceph

collectd plugins and dashboards for ceph
GNU General Public License v2.0
60 stars 65 forks source link

read_op_per_sec / write_op_per_sec in Jewel #38

Open maran opened 8 years ago

maran commented 8 years ago

Heya,

It seems that Ceph changed the op_per_sec to read_op_per_sec and write_op_per_sec somewhere in Jewel. I think this is the reason I'm no longer getting op_per_sec stats in my graphite.

I might want to venture a pull request to fix this but I'm unsure how to properly check for Ceph versions. Is there any instance of this somewhere in the code already? Is there a recommended way of doing this?

cfz commented 8 years ago

hi there,

a simple solution is just to append read_op_per_sec and write_op_per_sec. it work for all versions, missing field will be given a 0 https://github.com/cfz/collectd-ceph/commit/eb9de794bbe0829f2d5271d618eaf073a5d43e83