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

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

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 :-)


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