[MERGED] osmo-gsm-tester[master]: less verbose origins on interface enable/disable

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Sun May 7 01:22:38 UTC 2017


Neels Hofmeyr has submitted this change and it was merged.

Change subject: less verbose origins on interface enable/disable
......................................................................


less verbose origins on interface enable/disable

Change-Id: Ia59b3e6a9617f1b9e8dc0c488846a9fb0ec8f566
---
M src/osmo_gsm_tester/ofono_client.py
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo_gsm_tester/ofono_client.py b/src/osmo_gsm_tester/ofono_client.py
index bb044ff..5b7d0d2 100644
--- a/src/osmo_gsm_tester/ofono_client.py
+++ b/src/osmo_gsm_tester/ofono_client.py
@@ -145,11 +145,9 @@
         removals = self._interfaces - now
         self._interfaces = now
         for iface in removals:
-            with log.Origin('modem.disable(%s)' % iface):
-                self._on_interface_disabled(iface)
+            self._on_interface_disabled(iface)
         for iface in additions:
-            with log.Origin('modem.enable(%s)' % iface):
-                self._on_interface_enabled(iface)
+            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/2505
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia59b3e6a9617f1b9e8dc0c488846a9fb0ec8f566
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>



More information about the gerrit-log mailing list