On Tue, Dec 04, 2012 at 03:32:45PM +0000, Daniel P. Berrange wrote: > On Mon, Dec 03, 2012 at 03:17:23PM -0500, Dave Allan wrote: > > On Mon, Dec 03, 2012 at 05:12:59PM +0100, Martin Kletzander wrote: > > > This patch changes the default checkbox-state of "Delete all > > > associated storage" to be checked, but adds a prompt with a warning > > > for users to be sure they notice this change and they know what they > > > are doing (hopefully). > > > > I really think that changing the default from preserve user data to > > destroy user data is asking for trouble. > > That was my thought too, but this is done based on Cole's explicit > request against the V1 posting, so I'm acking it. Ok, I'm willing to see how it works in practice. The warning prompt should help, and I've certainly heard people complain about the current behavior. Dave > Daniel > -- > |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- http://virt-manager.org :| > |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
