[pve-devel] [PATCH manager] ceph install wizard: fix #3597: don't autofill network

[Stefan Hrdlicka]

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.hrdlicka at 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.lauterer at 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 at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>