[libvirt] [PATCH v5 4/5] storage: Support "chap" authentication for iscsi pool
John Ferlan
jferlan at redhat.com
Tue Jul 23 17:15:35 UTC 2013
On 07/23/2013 12:03 PM, Daniel P. Berrange wrote:
> On Tue, Jul 23, 2013 at 11:50:57AM -0400, John Ferlan wrote:
>> On 07/23/2013 10:56 AM, Daniel P. Berrange wrote:
>> <...snip...>
>>>> +++ b/src/storage/storage_backend_iscsi.c
>>>> @@ -714,7 +714,8 @@ virStorageBackendISCSISetAuth(const char *portal,
>>>>
>>>> if (!conn) {
>>>> virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
>>>> - _("iscsi 'chap' authentication requires connection"));
>>>> + _("iscsi 'chap' authentication not supported "
>>>> + "for autostarted pools"));
>>>> return -1;
>>>> }
>>>
>>> I noticed that the nwfilter already unconditionally calls virConnectOpen("qemu://system"); so we're already in fact suffering from the problem with
>>> autostart having a qemu dependency.
>>>
>>> Given this, I'd support a patch which simply did
>>>
>>> conn = virConnectOpen(privilege ? "qemu:///system" : "qemu:///session");
>>>
>>> in storageDriverAutostart, provided that we ignore any errors from
>>> virConnectOpen, and fallback to use NULL for the connection in that
>>> case.
>>>
>>> Obviously this is something we'll still need to fix properly in a
>>> future release, but at least it'll make autostart of storage pools
>>> with auth work in the common case in the short term for this release.
>>>
>>> Regards,
>>> Daniel
>>>
>>
>> So resurrect the following:
>>
>> https://www.redhat.com/archives/libvir-list/2013-July/msg00962.html
>
> Yep, ACK to that patch, but add "XXX Remove hardcoding of QEMU URI"
> as a comment just before the virConnectOpen call.
>
> Daniel
>
OK - I resurrected the patch, added the comment, and pushed.
John
More information about the libvir-list
mailing list