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

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



On 07/24/2013 10:06 PM, John Ferlan wrote:
> 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?
> 

Doing it only for the storage driver would move the pool autostart after QEMU
guests autostart and the guests wouldn't be able to use the pools.

Jan


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