For DB and WAL disks, not only partitions will show up now, but one
more type of disk, that didn't show up before: Namely, GPT-partitioned
disks with any partitions detected as used.

It's confusing as the size shown is of the full disk, with no
indication that a new partition will be appended at the end. This
problem was already present before, but only affected GPT-partitioned
disks where no usage on a partition was detected.

Signed-off-by: Fabian Ebner <f.eb...@proxmox.com>
---

No changes from v1.

RFC, because of the weird situation with the DB/WAL disks...Even if we
choose to extend the support in the back-end, it might make sense to
limit what we show in the front-end.

Dependency bump for widget-toolkit needed.

 www/manager6/ceph/OSD.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/www/manager6/ceph/OSD.js b/www/manager6/ceph/OSD.js
index aea38d6c..3e775606 100644
--- a/www/manager6/ceph/OSD.js
+++ b/www/manager6/ceph/OSD.js
@@ -38,6 +38,7 @@ Ext.define('PVE.CephCreateOsd', {
                            name: 'dev',
                            nodename: me.nodename,
                            diskType: 'unused',
+                           includePartitions: true,
                            fieldLabel: gettext('Disk'),
                            allowBlank: false,
                        },
@@ -48,6 +49,7 @@ Ext.define('PVE.CephCreateOsd', {
                            name: 'db_dev',
                            nodename: me.nodename,
                            diskType: 'journal_disks',
+                           includePartitions: true,
                            fieldLabel: gettext('DB Disk'),
                            value: '',
                            autoSelect: false,
@@ -101,6 +103,7 @@ Ext.define('PVE.CephCreateOsd', {
                            name: 'wal_dev',
                            nodename: me.nodename,
                            diskType: 'journal_disks',
+                           includePartitions: true,
                            fieldLabel: gettext('WAL Disk'),
                            value: '',
                            autoSelect: false,
-- 
2.30.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to