Re: [libvirt] [PATCH] Fix vm define error with back compat console device

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

 



"Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
...
>>   + virsh --connect qemu:///session define devs.xml
>
> Shouldn't use qemu:///session for test cases like this - this is what
> the test:///default driver is for, avoiding the fragility & danger of
> using the daemon & live hypervisor drivers.

There's no failure with test:///default.
But I'll adjust it to use the new unix_sock_dir config option I proposed.
However, when run by root, it'll still leave logs in the wrong place,
which is currently hard-coded, so I'll have to do more.
Either add a config. directive to change the root (log/sockdir,etc)
directory, or yet another config setting, just for the log.
Preference: log_dir or log_file?

> Can you pass that XML through xmllint -format so is has more

Good idea.  Done.

> readable indentation - good to remove all the disk/interface/hostdev
> devices too, since they're not relevant to the test in question.

too much hubris ;-)
Done, too.

>> +libvirtd > log 2>&1 & pid=$!
>> +sleep 1
>
> No need for the libvirtd daemon if using the test driver.

--
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]