Re: 答复: [PATCH v2] qemu: change monitor.sock from /var/lib/libvirt/qemu/domain-*** to /var/run/libvirt/qemu/domain-***

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

 



On 10/27/2017 10:57 AM, Daniel P. Berrange wrote:
> On Fri, Oct 27, 2017 at 10:25:23AM +0200, Michal Privoznik wrote:
>> On 10/27/2017 03:47 AM, Caoxinhua wrote:
>>> Yes,In qemuxml2argvtest case, mymain call qemuTestDriverInit to init qemu driver, qemuTestDriverInit call mkdtemp() to init stateDir, so stateDir is randomly.  But '-monitor' part of the command line must be a const value. Can I use a const value "/tmp/lib" to instead of random value at qemuxml2argvtest?
>>
>> We will have to. Looks like we don't need stateDir nor configDir in the
>> test anyway.
> 
> I would suggest  '/does-not-exist'  so we see a hard fail if we accidentally
> have something in the test suite that tries to create a file in that
> dir.   Using fixed filenames in /tmp is  a big no-no in general.

Well, we already use /tmp/lib/ for the monitor path. So sticking with it
means we don't have to update all of those ~600 files. Moreover, we
don't really create the dirs/files. But sure, having /does-not-exist may
be more robust.

Michal

--
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]
  Powered by Linux