Dashboard information not displayed in 12.1.1 luminous
See original GitHub issueHi, I am currently using the docker you are providing here with a proof of concept cluster running luminous.
~]$ ceph tell mon.* version
mon.ceph1: ceph version 12.1.1 (f3e663a190bf2ed12c7e3cda288b9a159572c800) luminous (rc)
mon.ceph2: ceph version 12.1.1 (f3e663a190bf2ed12c7e3cda288b9a159572c800) luminous (rc)
mon.ceph3: ceph version 12.1.1 (f3e663a190bf2ed12c7e3cda288b9a159572c800) luminous (rc)
The first problem encountered is with the infamous line 317 :
timechekMons = data['health']['timechecks']['mons'];
As of 12.1.1, running ceph status --format=json
returns :
,"health":{"checks":{"PG_NOT_SCRUBBED":{"severity":"HEALTH_WARN","message":"6 pgs not scrubbed for 86400"}}
The variable name seems to have changed.
So changing the JS to timechekMons = data['health']['checks']['mons'];
solves that problem.
Onto the next problem : line 229
// CREATE A ALERT MESSAGE {{{
function message(severity, msg) {
if (severity == 'success') { icon = 'ok' }
if (severity == 'warning') { icon = 'flash' }
if (severity == 'danger') { icon = 'remove' }
return '<div class="alert alert-' + severity + '"><strong><span class="glyphicon glyphicon-' + icon + '"> </span>' + msg + '</strong></div>';
}
// }}}
At this point, severity is undefined, I do not understant why since I see : {"severity":"HEALTH_WARN"
in the json, which is a case you handle at line 246.
To bypass the problem, i set an else {icon = 'ok'}
in the message function.
Then the next error arising is : TypeError: a is undefined
at jquery-2.1.4.min.js:2:2724
Starting there, it is beyond my capabilities to keep going since I have basically no knowledge in web development.
Did I miss something about which versions of Ceph to use? What would you recommand?
Thanks a bunch!
Issue Analytics
- State:
- Created 6 years ago
- Reactions:2
- Comments:27 (14 by maintainers)
Top GitHub Comments
Hi all,
I’ve just pushed a new branch called luminous, which as it is right now ONLY works for the luminous release. Can you guys please try if this is working for you? If yes I will try to merge it so it works for all versions.
Cheers, Christian
@ENACaseyBoone thank you for taking the time to test this! I am going to merge it then and create a new release.