Sandro Bonazzola has uploaded a new change for review. Change subject: nit: fix pep8 issues ......................................................................
nit: fix pep8 issues fixed pep8 issues causing make distcheck failure. Change-Id: I5d20dcacbd239ac00fab93fcf1939dfd681faf29 Signed-off-by: Sandro Bonazzola <[email protected]> --- M ovirt_hosted_engine_ha/agent/states.py M ovirt_hosted_engine_ha/broker/listener.py 2 files changed, 18 insertions(+), 12 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-hosted-engine-ha refs/changes/78/33178/1 diff --git a/ovirt_hosted_engine_ha/agent/states.py b/ovirt_hosted_engine_ha/agent/states.py index 2cb2cf4..f40107f 100644 --- a/ovirt_hosted_engine_ha/agent/states.py +++ b/ovirt_hosted_engine_ha/agent/states.py @@ -725,8 +725,9 @@ :type new_data: HostedEngineData :type logger: logging.Logger """ - return new_data._replace(migration_result= - fsm.actions.MONITOR_MIGRATION()) + return new_data._replace( + migration_result=fsm.actions.MONITOR_MIGRATION() + ) @check_global_maintenance(GlobalMaintenance) @check_local_vm_unknown(UnknownLocalVmState) diff --git a/ovirt_hosted_engine_ha/broker/listener.py b/ovirt_hosted_engine_ha/broker/listener.py index 9b6a0d3..039c960 100644 --- a/ovirt_hosted_engine_ha/broker/listener.py +++ b/ovirt_hosted_engine_ha/broker/listener.py @@ -296,26 +296,31 @@ to a thread and uses this to track monitors accordingly. """ with self.server.sp_listener.conn_monitors_access_lock: - self.server.sp_listener.conn_monitors[threading - .current_thread().ident] = [] + self.server.sp_listener.conn_monitors[ + threading.current_thread().ident + ] = [] def _get_monitors_for_conn(self): with self.server.sp_listener.conn_monitors_access_lock: - ret = self.server.sp_listener.conn_monitors[threading - .current_thread().ident] + ret = self.server.sp_listener.conn_monitors[ + threading.current_thread().ident + ] return ret def _add_monitor_for_conn(self, id): with self.server.sp_listener.conn_monitors_access_lock: - self.server.sp_listener.conn_monitors[threading - .current_thread().ident].append(id) + self.server.sp_listener.conn_monitors[ + threading.current_thread().ident + ].append(id) def _remove_monitor_for_conn(self, id): with self.server.sp_listener.conn_monitors_access_lock: - self.server.sp_listener.conn_monitors[threading - .current_thread().ident].remove(id) + self.server.sp_listener.conn_monitors[ + threading.current_thread().ident + ].remove(id) def _remove_monitor_conn_entry(self): with self.server.sp_listener.conn_monitors_access_lock: - del self.server.sp_listener.conn_monitors[threading - .current_thread().ident] + del self.server.sp_listener.conn_monitors[ + threading.current_thread().ident + ] -- To view, visit http://gerrit.ovirt.org/33178 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I5d20dcacbd239ac00fab93fcf1939dfd681faf29 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Sandro Bonazzola <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
