[pve-devel] [PATCH manager 1/3] fix wrong osd calculation in ceph dashboard

Dominik Csapak d.csapak at proxmox.com
Tue Jul 11 14:39:18 CEST 2017


in luminous, the error message is not
'x/y in osds are down' anymore, but
'x osds down'

so we need to adapt the parsing, and it means we cannot check
the number of in osds there anymore (was never really needed, so
we can simply omit it)

when an osd is down but marked as out, those errors disappear

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 www/manager6/ceph/StatusDetail.js | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/www/manager6/ceph/StatusDetail.js b/www/manager6/ceph/StatusDetail.js
index 6517ab7b..f807674e 100644
--- a/www/manager6/ceph/StatusDetail.js
+++ b/www/manager6/ceph/StatusDetail.js
@@ -127,18 +127,13 @@ Ext.define('PVE.ceph.StatusDetail', {
 	var out_osds = total_osds - in_osds;
 	var down_osds = total_osds - up_osds;
 	var downin_osds = 0;
-	var downinregex = /(\d+)\/(\d+) in osds are down/;
+	var downinregex = /(\d+) osds down/;
 	Ext.Array.some(record.data.health.summary, function(item) {
 	    var found = item.summary.match(downinregex);
 
 	    if (found !== null) {
-		// sanity check, test if the message is
-		// consistent with the direct value
-		// for in osds
-		if (found[2] == in_osds) {
 		    downin_osds = parseInt(found[1],10);
 		    return true;
-		}
 	    }
 
 	    return false;
-- 
2.11.0





More information about the pve-devel mailing list