Re: [libvirt] PATCH: Switch openvz driver to domain APIs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> The biggest flaw I see currently is that the
openvz driver doesn't load the existing device config for networks or
filesystems of existing VMs, so you can't see that info in the XML dump
Yes. Good note. I will implement it.


-        /*get VCPU*/
-        ret = openvzReadConfigParam(veid, "CPUS", temp, sizeof(temp));
-        if (ret < 0) {
-             openvzError(conn, VIR_ERR_INTERNAL_ERROR,
-                            _("Cound not read config for container %d"), veid);
-             goto error;
-        } else if (ret > 0) {
-             vmdef->vcpus = strtoI(temp);
-        }
Why did you remove loading VCPUs and always set = 1 ?
+ + dom->def->vcpus = 1;


--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]