Review at  https://gerrit.osmocom.org/2485

ofono_client: drop explicit exception catching

For some reason the code there catches exceptions happening while interfaces
are being established. That's not a good idea, drop it.

Related: OS#2220
Change-Id: Ida7731ed6ff3cad44ff437137abe2d6a424fcaa4
---
M src/osmo_gsm_tester/ofono_client.py
1 file changed, 2 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/85/2485/1

diff --git a/src/osmo_gsm_tester/ofono_client.py 
b/src/osmo_gsm_tester/ofono_client.py
index 1f0380a..9a018e1 100644
--- a/src/osmo_gsm_tester/ofono_client.py
+++ b/src/osmo_gsm_tester/ofono_client.py
@@ -119,16 +119,10 @@
         self._interfaces = now
         for iface in removals:
             with log.Origin('modem.disable(%s)' % iface):
-                try:
-                    self._on_interface_disabled(iface)
-                except:
-                    self.log_exn()
+                self._on_interface_disabled(iface)
         for iface in additions:
             with log.Origin('modem.enable(%s)' % iface):
-                try:
-                    self._on_interface_enabled(iface)
-                except:
-                    self.log_exn()
+                self._on_interface_enabled(iface)
 
     def _on_interface_enabled(self, interface_name):
         self.dbg('Interface enabled:', interface_name)

-- 
To view, visit https://gerrit.osmocom.org/2485
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida7731ed6ff3cad44ff437137abe2d6a424fcaa4
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>

Reply via email to