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

Peter Krempa pkrempa at redhat.com
Tue Mar 24 14:37:56 UTC 2015


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150324/fdf57686/attachment-0001.sig>


More information about the libvir-list mailing list