ovsdb: session.rpc never initialized

Previously, if idl.Transaction.commit_block() returned a status of
TRY_AGAIN we would check self.api.idl._session.rpc.status which would
result in an attribute error as rpc is None.

This patch fixes this attribute error by removing this unneeded check. In
addtion, the force_reconnect() is also removed as ovs.jsonrpc handles
reconnecting automatically for us.

Change-Id: Ibf3ce5cd3432845f8938a1d83637ecf59b14b5ca
Closes-bug: 1465889

diff --git a/neutron/agent/ovsdb/impl_idl.py b/neutron/agent/ovsdb/impl_idl.py
index 57399fc01af2583266f5d4c4a29ba298df468ca9..51ea96f17a31f82e11e1efe4d472f28c52d0188c 100644 (file)
--- a/neutron/agent/ovsdb/impl_idl.py
+++ b/neutron/agent/ovsdb/impl_idl.py
@@ -93,9 +93,6 @@ class Transaction(api.Transaction):
             status = txn.commit_block()
             if status == txn.TRY_AGAIN:
                 LOG.debug("OVSDB transaction returned TRY_AGAIN, retrying")
-                if self.api.idl._session.rpc.status != 0:
-                    LOG.debug("Lost connection to OVSDB, reconnecting!")
-                    self.api.idl.force_reconnect()
                 idlutils.wait_for_change(
                     self.api.idl, self.timeout - elapsed_time,
                     seqno)