Skip to content

vmstoragemodel doesn't work for VMs on ppc64le hypervisor #2139

@zet809

Description

@zet809

When using vmstoragemodel to specify the stroagemodel for a diskfile, it will be replaced by 'scsi' for a ppc64 or ppc64le hypervisor.

 # lsdef testvm -i vmstoragemodel
 Object name: testvm
    vmstoragemodel=virtio
<disk type='file' device='disk'>
      <driver name='qemu' type='qcow2' cache='none'/>
      <source file='/var/lib/libvirt/images/testvm.sda.qcow2'/>
      <backingStore/>
      <target dev='sda' bus='scsi'/>             ==> Shall be dev='vda' and bus='virtio'
      <alias name='scsi0-0-0-0'/>
      <address type='drive' controller='0' bus='0' target='0' unit='0'/>
    </disk>

It is wrongly modified within our code when it detect the hypervisor is a ppc64/ppc64le model. Changing to 'ppc64$' only will work.

        if ($confdata->{$hyp}->{cpumodel} and $confdata->{$hyp}->{cpumodel} =~ /ppc64/i) {
            $confdata->{vm}->{$node}->[0]->{storagemodel} = "scsi";
        }

It is approved that 'virtio' also works for rhev4.x on ppc64le hypervisor, so need more test for vm related functions.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions