Skip to content
This repository has been archived by the owner on Nov 12, 2018. It is now read-only.

update #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

update #18

wants to merge 2 commits into from

Conversation

phonglh79
Copy link

No description provided.

…on to calculate percent values and rename function to create zabbix configuration file
@thelan
Copy link
Owner

thelan commented Nov 14, 2017

Hello,

Thanks for the PR and the python script.
It seems it doesn't work with latest ceph version.
I've currently tested with 10.2.7 (50e863e0f4bc8f4b9e31156de690d765af245185)
Which version of ceph did your script support.

I've got this error

Traceback (most recent call last):
  File "./ceph-status.py", line 261, in <module>
    main()
  File "./ceph-status.py", line 253, in main
    Info()
  File "./ceph-status.py", line 199, in Info
    for s in data['pg_stats']:
KeyError: 'pg_stats'

Here is the ceph pg stat -f json-pretty:

{
    "num_pg_by_state": [
        {
            "name": "active+clean",
            "num": 1968
        }
    ],
    "version": 29542931,
    "num_pgs": 1968,
    "num_bytes": 251757018805,
    "raw_bytes_used": 548103057408,
    "raw_bytes_avail": 16917757952000,
    "raw_bytes": 17465861009408,
    "read_bytes_sec": 1316,
    "write_bytes_sec": 8339,
    "io_sec": 7
}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants