Re: [PATCH 3/6] storage: fs: Don't attempt directory creation if it already exists

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

 




On 07/16/2015 07:39 PM, Christophe Fergeau wrote:
On Thu, Jul 16, 2015 at 02:02:39PM +0800, tzheng wrote:
Hi,Cole

After this patch is included in libvirt on rhel7.2,virt-manager can not
create dir pool with existing directory.
There is no default pool created in a fresh rhel7.2 system,so if I try to
create default pool,the below error shows:
Error creating pool: Could not build storage pool: failed to create
directory '/var/lib/libvirt/images': File exists

Do you think it's acceptable for this change since the
directory(/var/lib/libvirt/images) is created by libvirt,if users want to
create pool with /var/lib/libvirt/images,they need to delete images
directory firstly,if so I think maybe some notes in doc is preferred,thanks
very much.
This part of the code was then changed by
https://libvirt.org/git/?p=libvirt.git;a=commit;h=c8661a1a7ee8ef11010
Are you 100% sure it's this patch which causes the issue you describe?

I just searched this patch which is related to the change for build pool with existing directory.


GNOME Boxes is having very similar issues, see
https://bugzilla.gnome.org/show_bug.cgi?id=752417

Thanks,it's the similar issue,I file a rhel7 bug:https://bugzilla.redhat.com/show_bug.cgi?id=1244080


Christophe

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