On 03/02/2013 09:30 AM, Christophe Fergeau wrote:
Hey,
On Wed, Feb 27, 2013 at 07:20:56PM +0800, Daniel Veillard wrote:
> I have just tagged git and pushed the tarball. The rpms for F17
> are at the usual place too:
>
ftp://libvirt.org/libvirt/
>
> I gave a try to the set of rpms and this seems to work fine for
> relatively simple tests.
> Please give it more testing and let's keep change to git to
> bug fixes to minimize risks for 1.0.3.
> If everything goes well I will push 1.0.3 on Friday,
I've found a pretty bad issue with rc2/git head and Boxes, creating new
boxes fails with
2013-03-02 14:04:22.028+0000: 15681: error :
qemuDomainCheckDiskPresence:1837 : cannot access file
'/home/teuf/isos/msdn/win7/fr_windows_7_home_premium_n_with_sp1_x64_dvd_u_676833.iso':
Opération non permise
Looking more into it, this seems to be caused by
commit f506a4c115c44003455cb956861836a46425f97b
Date: Thu Jan 31 13:18:45 2013 -0500
util: make virSetUIDGID a NOP only when uid or gid is -1
Yep, I had originally been careful to maintain the previous "0 means
ignore" behavior, but was convinced that, even if it temporarily
uncovered some bugs, it was worth it in the long run to do it correctly.
I had expected there would be some fallout that wasn't uncovered
immediately (I thought it had all been revealed earlier when Doug and
Guido found/fixed a similar problem with session mode libvirt, though.
Sigh.)