On 7/2/18 4:46 PM, Andrew Neitsch wrote:
> The lxc CreateWizard ignores the pool setting. That makes creating
> containers impossible for users who are restricted to using a pool. This
> patch fixes that.
> 
> Signed-off-by: Andrew Neitsch <and...@neitsch.ca>
> ---
>  www/manager6/lxc/CreateWizard.js | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/www/manager6/lxc/CreateWizard.js 
> b/www/manager6/lxc/CreateWizard.js
> index a913ca0ccf6f..ea86c6aaa665 100644
> --- a/www/manager6/lxc/CreateWizard.js
> +++ b/www/manager6/lxc/CreateWizard.js
> @@ -69,7 +69,6 @@ Ext.define('PVE.lxc.CreateWizard', {
>                   xtype: 'pvePoolSelector',
>                   fieldLabel: gettext('Resource Pool'),
>                   name: 'pool',
> -                 submitValue: false,
>                   value: '',
>                   allowBlank: true
>               },
> @@ -295,6 +294,10 @@ Ext.define('PVE.lxc.CreateWizard', {
>               delete kv.nodename;
>               delete kv.tmplstorage;
>  
> +             if (!kv.pool.length) {
> +                 delete kv.pool;
> +             }
> +
>               if (!kv.password.length && kv['ssh-public-keys']) {
>                   delete kv.password;
>               }
> 

Looks good, thanks!

To allow us to include your patch we'd need a CLA from you.
We use the Harmony CLA, please see:
https://pve.proxmox.com/wiki/Developer_Documentation#Software_License_and_Copyright
for more information and how to send it to us.

cheers,
Thomas

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

Reply via email to