Project

General

Profile

Bug #51212

Updated by Patrick Seidensal almost 3 years ago

*OSD Capacity units Grafana widget has bad decimals* 

 Current: 

 !{width:25%}broken.jpg! 

 <pre> 
       "yAxis": { 
         "decimals": 2, 
 </pre> 
 !{width:25%}fixed.jpg! 

 *Health Status widget breaks if query returns two rows* 

 The health Status widget breaks if query returns two rows, which happens when a mgr instance fails (over). If a time frame in Grafana is selected in which that happened, the widget will not be able to return the health status correctly. 

 This is due to the widget type being used in combination with the Prometheus query: 

 <pre> 
 ceph_health_status{instance=~'$instance'} 
 </pre> 

 !{width:50%}health_status.jpg! 

 *Hard-coded time zone* 

 A hard-coded time zone in the Grafana dashboard specifications (JSON files) results in the administrator being unable to change the time zone displayed by Grafana for those dashboards, as those dashboards are automatically imported using a Grafana internal mechanism, which prevent administrators to change them (which they shouldn't anyway). Only if the time zone is not set in those dashboards, the value of the global Grafana settings is used. Few dashboards have a hard-coded time zone set. 

 *Convert newlines from CRLF to LF* 

 A single dashboard specification (rbd-details.json) has different line endings (CRLF) than all the other dashboards (LF). 


 *Some widgets are not filterable by the instance label* 

 Basically, this one is about promql queries which do not contain the instance label for some widgets, but there's a discussion about if it should be fixed to be consistent or, for some dashboards/widgets, completely removed. The idea to completely remove the filtering by the instance label stems from the fact that the mgr module is the only way to provide Ceph related metrics, so there'll ever only be one address/node to query, which provides data that is not related to the queried node. So the instance label has no meaningful use. 

 See discussion in the PR.

Back