mbox series

[ovs-dev,v3,0/2] Partial cluster support in Python IDL client

Message ID 20180710192616.31277-1-nusiddiq@redhat.com
Headers show
Series Partial cluster support in Python IDL client | expand

Message

Numan Siddique July 10, 2018, 7:26 p.m. UTC 
From: Numan Siddique <nusiddiq@redhat.com>

Python IDL library is lacking the functionality to connect to the
clustered db servers by providing multiple remotes (like -
"tcp:10.0.0.1:6641, tcp:10.0.0.2:6641, tcp:10.0.0.3:6641") in the
connection string.

This patch adds this functionality to the python idl library.
It still lacks the feature to connect to the master of the cluster.
To add this
  - python idl client should monitor and read the '_Server' schema
  - connect to the master of the cluster.

I will submit the patch once that is ready. But for now I think this
is good enough for the clients to connect to the cluster dbs.


v2 -> v3
--------
Addressed the review comments from Ben to parse the remote in
db/idl.py

v1 -> v2
--------
Deleted the debug code which I forgot to cleanup when sending v1.

Numan Siddique (2):
  ovs python: ovs.stream.open_block() returns success even if the remote
    is unreachable
  python jsonrpc: Allow jsonrpc_session to have more than one remote.

 python/ovs/db/idl.py      | 20 ++++++++++-
 python/ovs/jsonrpc.py     | 39 +++++++++++++++++-----
 python/ovs/socket_util.py | 34 +++++++++++++++++++
 python/ovs/stream.py      | 16 +++++++--
 tests/automake.mk         |  1 +
 tests/ovsdb-idl.at        | 70 +++++++++++++++++++++++++++++++++++++++
 tests/test-ovsdb.py       | 13 ++++++--
 tests/test-stream.py      | 32 ++++++++++++++++++
 8 files changed, 210 insertions(+), 15 deletions(-)
 create mode 100644 tests/test-stream.py