Re: [PATCH 2/7] conf: Add support for storage state directory

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

 



On Tue, Mar 24, 2015 at 10:16:25 -0400, John Ferlan wrote:
> 
> 
> On 03/24/2015 06:06 AM, Erik Skultety wrote:
> > Before introducing necessary changes to storage_driver.c, first prepare
> > our structures for storage state XML support.
> > 
> > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1177733
> > ---
> >  src/conf/storage_conf.h      | 1 +
> >  src/storage/storage_driver.c | 1 +
> >  2 files changed, 2 insertions(+)
> > 
> 
> I had started down this path before for a different bug/issue:
> 
> http://www.redhat.com/archives/libvir-list/2015-January/msg00465.html
> 
> but abandoned the stateDir because it was felt it wasn't necessary.  I
> recall running into a few "interesting" issues with stateDir including
> fixing one issue seen during testing that didn't hit the list.  Good

The patches above are for status XMLs for storage volumes while this
series does the same for storage pools. While volumes can theoretically
be reloaded from existing pools this series fixes storage pools that
vanish during start of libvirt.

> news is I still have the patches in a branch somewhere if you're
> interested.  1 & 2 are on list... The 3rd one in the archive was a
> solution to the particular problem - that was rejected and a different
> solution was pushed.
> 
> In any case, I do suggest looking at 1 & 2, plus I can send you an
> adjustment to 1 to resolve some condition I ran into, but cannot recall
> the details.

Peter

Attachment: signature.asc
Description: Digital signature

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