Change in ovirt-hosted-engine-ha[master]: rpc: Custom made RPC replaced with xmlrpc

Code Review gerrit at ovirt.org
Wed Aug 30 15:26:54 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged. ( https://gerrit.ovirt.org/81083 )

Change subject: rpc: Custom made RPC replaced with xmlrpc
......................................................................


rpc: Custom made RPC replaced with xmlrpc

Broker link was just a home made implementation of
a remote procedure calls. This patch removes BrokerLink
and introduces standard pathing xmlrpc as a replacement.

https://github.com/oVirt/ovirt-site/pull/1157

Change-Id: I28c31844394df7dfe6012d4b8d3421ad8a69a2ab
Signed-off-by: Denis Chaplygin <dchaplyg at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1457468
---
M ovirt_hosted_engine_ha/agent/constants.py.in
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/broker/listener.py
M ovirt_hosted_engine_ha/broker/notifications.py
M ovirt_hosted_engine_ha/broker/storage_broker.py
M ovirt_hosted_engine_ha/client/client.py
M ovirt_hosted_engine_ha/lib/Makefile.am
M ovirt_hosted_engine_ha/lib/brokerlink.py
D ovirt_hosted_engine_ha/lib/brokerlink_test.py
A ovirt_hosted_engine_ha/lib/unixrpc.py
10 files changed, 165 insertions(+), 431 deletions(-)

Approvals:
  Denis Chaplygin: Verified
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/81083
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I28c31844394df7dfe6012d4b8d3421ad8a69a2ab
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list