[pve-devel] [PATCH manager] OSD: rename 'remove' to 'destroy'

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Jun 6 15:30:51 CEST 2017


To have consistency with the CLI tools `pveceph destroyosd` command.
This is also more clear about the resulting action, as 'Remove' could
also just mean 'Remove from Config', for example.

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---

some user mentioned that this was a bit missleading, imo he is rigth.


 www/manager6/ceph/OSD.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/manager6/ceph/OSD.js b/www/manager6/ceph/OSD.js
index 8170f17f..b184e2ea 100644
--- a/www/manager6/ceph/OSD.js
+++ b/www/manager6/ceph/OSD.js
@@ -150,7 +150,7 @@ Ext.define('PVE.CephRemoveOsd', {
 
 	me.isCreate = true;
 
-	me.title = gettext('Remove') + ': ' + 'Ceph OSD osd.' + me.osdid.toString();
+	me.title = gettext('Destroy') + ': ' + 'Ceph OSD osd.' + me.osdid.toString();
 
         Ext.applyIf(me, {
 	    url: "/nodes/" + me.nodename + "/ceph/osd/" + me.osdid.toString()
@@ -401,7 +401,7 @@ Ext.define('PVE.node.CephOsdTree', {
 	});
 
 	var remove_btn = new Ext.Button({
-	    text: gettext('Remove'),
+	    text: gettext('Destroy'),
 	    disabled: true,
 	    handler: function(){
 		var rec = sm.getSelection()[0];
-- 
2.11.0





More information about the pve-devel mailing list