Re: [RFC] Add flag for virsh undefine to remove/wipe the disk devices

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

 



On Thu, Mar 31, 2011 at 11:30:03AM +0100, Daniel P. Berrange wrote:
> On Wed, Mar 30, 2011 at 09:50:19PM +0800, Daniel Veillard wrote:
> > On Wed, Mar 30, 2011 at 09:39:14PM +0800, Osier Yang wrote:
> > >    For these situations, we need to do checking and throw
> > >    straightforward warnings to tell user why it can't be
> > >    removed/wiped.
> > 
> >   I would rather make this a flag of virDomainUndefine(), except
> > there is no flag argument for it :(
> 
> I don't think this is a good idea. Applications should directly call
> the storage APIs for this, so that when we adding RBAC support to
> our APIs, we get correct access control checks on *each* volume
> being deleted / wiped. We won't want the 'undefine' API for a VM
> to be side-stepping the volume access controls.

  Well if the user don't have the resource to wipe all the volumes,
fail the operation. Simple thing should stay simple even if we allow
for more complex ones.

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

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