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

fixed variable name #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed variable name #34

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 16, 2016

ceph_pool_plugin threw an exeption which shows in syslog as


Feb 16 08:29:29 ceph00 collectd[4416]: ceph-pool: failed to ceph pool stats :: global name 'ceph_dfcmdline' is not defined :: Traceback (most recent call last):#012  File "/usr/share/collectd/python/ceph_pool_plugin.py", line 57, in get_stats#012    df_output = subprocess.check_output(ceph_dfcmdline, shell=True)#012NameError: global name 'ceph_dfcmdline' is not defined
Feb 16 08:29:29 ceph00 collectd[4416]: ceph: failed to retrieve stats

I found that "ceph_dfcmdline" was actually called "cephdf_cmdline".

@waipeng
Copy link

waipeng commented Apr 7, 2016

Can confirm this works. Please merge this, thanks!

@A-Dechorgnat
Copy link

+1

@cgimeno
Copy link

cgimeno commented May 10, 2016

Confirmed

@Kallio
Copy link

Kallio commented Nov 16, 2016

Please merge this. IT has also duplicate req #42 and #41

@dbiazus
Copy link

dbiazus commented Nov 28, 2016

👍

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.

5 participants