Re: [PATCH v4 0/9] Rework storage migration

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

 



On Thu, Feb 21, 2013 at 14:39:17 +0100, Michal Privoznik wrote:
> Since there are many hidden problems with auto-creating storage for the domain
> in here, as previous rounds of reviewing the series has shown, I've decided to
> not do anything about it for now and the auto allocation is dropped completely.
> So we are back to the assumption we already have - users need to make sure files
> already exists and have the right size. Hawk.

OK, I think that's wise considering how things are getting complicated
on the storage pre-creation side. I guess you'll be working on
implementing that in a follow-up series.

> diff to v4:
> -drop storage auto-allocation
> -include John's and Jirka's reviews
> 
> Note, that most of the patches has been ACKed already.

Could you consider putting the changelog into individual patches next
time? Especially in case the previous version of a particular patch was
acked and no changes were made in it since then. You can't expect
reviewers will remember what patches they already acked :-)

Jirka

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