[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