Re: [libvirt] [PATCH v5 4/5] storage: Support "chap" authentication for iscsi pool

On 07/24/2013 05:57 AM, Daniel P. Berrange wrote: >> On Wed, Jul 24, 2013 at 10:25:06AM +0200, Ján Tomko wrote: <...snip...> >> >> Both secret and qemu drivers are registered after the storage driver on >> libvirtd startup, so autostarting these pools will only work on storage driver >> reload. On libvirtd startup it fails with: >> qemuConnectOpen:1033 : internal error qemu state driver is not active >> >> (And it seems nwfilter only opens the qemu:// connection on reload) > > Oh damn, yes, that pretty much dooms us. We can't change the order of > the drivers either, because autostarting of QEMU guests, requires that > the storage pools be autostarted already. > > To fix this would require that we split virStateInitialize into two > parts, virStateInitialize() and virStateAutoStart(). That's too big > a change todo for this release, but we could do it for next release > without too much trouble. > > > Daniel > Could we just do it for storage driver? It seems you are indicating that the following would suffice, right? Or does this problem go deeper?