[pve-devel] [PATCH manager] fix default text for SCSI Controller Type

[Wolfgang Bumiller]

NAK.

On Fri, Sep 15, 2017 at 10:45:45AM +0200, Thomas Lamprecht wrote:
> On 09/15/2017 10:31 AM, Alwin Antreich wrote:
> > Signed-off-by: Alwin Antreich <a.antreich at proxmox.com>
> > ---
> >   www/manager6/Utils.js | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
> > index 0b850977..09e6be90 100644
> > --- a/www/manager6/Utils.js
> > +++ b/www/manager6/Utils.js
> > @@ -194,7 +194,7 @@ Ext.define('PVE.Utils', { utilities: {
> >       render_scsihw: function(value) {
> >   	if (!value) {
> > -	    return PVE.Utils.defaultText + ' (LSI 53C895A)';
> > +	    return PVE.Utils.defaultText + ' (VirtIO SCSI)';
> 
> Shouldn't that be depended on the OSType?

No. The OS-Type specific default is purely a front-end function, but
this value is the explicit "use the backend's default" value which is
still LSI.