public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Alexandre Derumier <aderumier@odiso.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH pve-manager] ui: ceph status: add pg warning state
Date: Thu, 22 Jun 2023 12:54:37 +0200	[thread overview]
Message-ID: <20230622105437.2130941-1-aderumier@odiso.com> (raw)

Like ceph mgr dashboard, we need a warning state.

- set degraded && undersized as warning instead criticial

- add "normal" (light blue) color for working state

- use warning (orange) color for warning state

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
---
 www/manager6/ceph/StatusDetail.js | 29 ++++++++++++++++++-----------
 1 file changed, 18 insertions(+), 11 deletions(-)

diff --git a/www/manager6/ceph/StatusDetail.js b/www/manager6/ceph/StatusDetail.js
index d6c0763b..e1bf425a 100644
--- a/www/manager6/ceph/StatusDetail.js
+++ b/www/manager6/ceph/StatusDetail.js
@@ -94,6 +94,7 @@ Ext.define('PVE.ceph.StatusDetail', {
 	colors: [
 	    '#CFCFCF',
 	    '#21BF4B',
+	    '#C2DDF2',
 	    '#FFCC00',
 	    '#FF6C59',
 	],
@@ -152,7 +153,6 @@ Ext.define('PVE.ceph.StatusDetail', {
 	backfilling: 2,
 	creating: 2,
 	deep: 2,
-	degraded: 2,
 	forced_backfill: 2,
 	forced_recovery: 2,
 	peered: 2,
@@ -165,17 +165,20 @@ Ext.define('PVE.ceph.StatusDetail', {
 	snaptrim: 2,
 	snaptrim_wait: 2,
 
-	// error
-	backfill_toofull: 3,
-	backfill_unfound: 3,
-	down: 3,
-	incomplete: 3,
-	inconsistent: 3,
-	recovery_toofull: 3,
-	recovery_unfound: 3,
-	snaptrim_error: 3,
-	stale: 3,
+	//warning
+	degraded: 3,
 	undersized: 3,
+
+	// error
+	backfill_toofull: 4,
+	backfill_unfound: 4,
+	down: 4,
+	incomplete: 4,
+	inconsistent: 4,
+	recovery_toofull: 4,
+	recovery_unfound: 4,
+	snaptrim_error: 4,
+	stale: 4,
     },
 
     statecategories: [
@@ -191,6 +194,10 @@ Ext.define('PVE.ceph.StatusDetail', {
 	},
 	{
 	    text: gettext('Working'),
+	    cls: 'normal',
+	},
+	{
+	    text: gettext('Warning'),
 	    cls: 'warning',
 	},
 	{
-- 
2.39.2




             reply	other threads:[~2023-06-22 10:55 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-22 10:54 Alexandre Derumier [this message]
2023-06-22 13:50 ` Aaron Lauterer
2023-07-21 13:06 ` [pve-devel] [PATCH widget-toolkit 1/3] css: add 'working' css color class Aaron Lauterer
2023-07-21 13:06   ` [pve-devel] [PATCH manager 2/3] ceph: ui: followup: change blue for PGs in working state Aaron Lauterer
2023-07-21 13:06   ` [pve-devel] [PATCH manager 3/3] ceph: ui: rename PG state error to critical Aaron Lauterer
2023-07-21 13:08 ` [pve-devel] [PATCH pve-manager] ui: ceph status: add pg warning state Aaron Lauterer
2023-08-07  7:46   ` DERUMIER, Alexandre
2023-08-08  7:32     ` Aaron Lauterer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230622105437.2130941-1-aderumier@odiso.com \
    --to=aderumier@odiso.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal