On Tue, Jun 28, 2016 at 5:24 PM, Dan Kenigsberg
<danken(a)redhat.com> wrote:
> On Tue, Jun 28, 2016 at 04:08:51PM +0200, Simone Tiraboschi wrote:
>> On Tue, Jun 28, 2016 at 3:02 PM, Dan Kenigsberg <danken(a)redhat.com>
wrote:
>> > On Mon, Jun 27, 2016 at 10:08:33AM +0200, Stefano Danzi wrote:
>> >> Hi!
>> >
>> >
>> > Thanks for the detailed logging!
>> >
>> >>
>> >> The broker error is:
>> >>
>> >> ==> /var/log/ovirt-hosted-engine-ha/agent.log <==
>> >> MainThread::INFO::2016-06-27
09:27:03,311::brokerlink::140::ovirt_hosted_engine_ha.lib.brokerlink.BrokerLink::(start_monitor)
>> >> Success, id 140293563619152
>> >>
>> >> ==> /var/log/ovirt-hosted-engine-ha/broker.log <==
>> >> Thread-25::ERROR::2016-06-27
09:27:03,314::listener::182::ovirt_hosted_engine_ha.broker.listener.ConnectionHandler::(handle)
>> >> Error while serving connection
>> >> Traceback (most recent call last):
>> >> File
>> >>
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/broker/listener.py",
>> >> line 166, in handle
>> >> data)
>> >> File
>> >>
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/broker/listener.py",
>> >> line 299, in _dispatch
>> >> .set_storage_domain(client, sd_type, **options)
>> >> File
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/broker/storage_broker.py",
>> >> line 66, in set_storage_domain
>> >> self._backends[client].connect()
>> >> File
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/lib/storage_backends.py",
>> >> line 400, in connect
>> >> volUUID=volume.volume_uuid
>> >> File
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/lib/storage_backends.py",
>> >> line 245, in _get_volume_path
>> >> volUUID
>> >
>> > We have two issues here. First is that
>> >
https://gerrit.ovirt.org/gitweb?p=ovirt-hosted-engine-ha.git;a=blob;f=ovi...
>> > is still using vdscli to contact vdsm, instead of the preferred
>> > jsonrpccli.
>>
>> We already have this one:
>>
https://bugzilla.redhat.com/show_bug.cgi?id=1101554
>>
> Bug 1101554 - [RFE] HE-setup: use vdsm api instead of vdsClient
>
> It does not seem to be the same thing; here I'm referring to
> ovirt-hosted-engine-ha (not -setup). -ha *is* using vdsm API, but in one
> condition it's the soon-to-be-deprecated xmlrpc.
ovirt-hosted-engine-setup is already using just API via jsonrpc.
The issue is that ovirt-hosted-engine-ha somewhere uses vdsClient
which AFAIK internally still uses xmlrpc