Tried it and works as expected.

I can't answer the original mail, since I joined the mailing list a bit later 
:).

Tested-by: Stefan Hrdlicka<s.hrdli...@proxmox.com>

On 6/1/22 12:27, Aaron Lauterer wrote:
Can someone take a look at this? Patch should still apply.

On 5/2/22 16:05, Aaron Lauterer wrote:
By not auto filling the Ceph public network we can avoid accidential
clicks on 'Next' which will cause the first Mon to be created with a
potentially wrong network. While that is fixable, it is tedious and can
be easily avoided by making the user always select the network to use.

Signed-off-by: Aaron Lauterer <a.laute...@proxmox.com>
---
  www/manager6/ceph/CephInstallWizard.js | 1 +
  1 file changed, 1 insertion(+)

diff --git a/www/manager6/ceph/CephInstallWizard.js b/www/manager6/ceph/CephInstallWizard.js
index 59458b0d..bebfc319 100644
--- a/www/manager6/ceph/CephInstallWizard.js
+++ b/www/manager6/ceph/CephInstallWizard.js
@@ -372,6 +372,7 @@ Ext.define('PVE.ceph.CephInstallWizard', {
              name: 'network',
              value: '',
              fieldLabel: 'Public Network IP/CIDR',
+            autoSelect: false,
              bind: {
              allowBlank: '{configuration}',
              },


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


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

Reply via email to