Skip to content

Minor: listVmsForImport API fix #8414

@andrijapanicsb

Description

@andrijapanicsb

ACS 4.19 RC1

  • minor update

listVmsForImport API states (based on the docs) that the username and password parameters are required (besides zoneid, host IP and hypervisor=KVM), while in reality, this API seems to connect to remote libvirt via TCP unauthenticated connections and lists stopped VMs.

Using CMK if the "username" parameter is omitted - it gives error and says it's mandatory,

(localcloud)  > listVmsForImport zoneid=fbfd1039-ca49-436d-b17d-e03f39663047 hypervisor=KVM host=10.0.35.106
 Error: (HTTP 431, error code 4350) Username need to be provided.

...however any random username can be passed and it will work:

(localcloud)  > listVmsForImport username=aaaaaaaaaaaaaaaaa zoneid=fbfd1039-ca49-436d-b17d-e03f39663047 hypervisor=KVM host=10.0.35.106 
{
  "count": 1,
  "unmanagedinstance": [
    {
      "cpucorepersocket": -1,
      "disk": [
        {
          "capacity": 17179869184,
          "controller": "virtio",
          "datastorepath": "/var/lib/libvirt/images/MasteringKVM01.qcow2",
          "datastoretype": "file",
          "id": "0",
          "label": "vda",
          "position": 0
        },
        {

If fixed, the docs also needs to be updated to remove the statement that the username/password are mandatory

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions