Change in osmo-gsm-tester[master]: pyflakes: Remove occurrences of using undefined variables

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/.

Holger Freyther gerrit-no-reply at lists.osmocom.org
Wed Feb 27 07:15:10 UTC 2019


Holger Freyther has uploaded this change for review. ( https://gerrit.osmocom.org/13072


Change subject: pyflakes: Remove occurrences of using undefined variables
......................................................................

pyflakes: Remove occurrences of using undefined variables

Mention the attribute right away and remove a method that looks
to have been broken (more vs. more_rules).

Change-Id: Iba18a96ff804780e66c69b459e112f9d42b7b2ff
---
M src/osmo_gsm_tester/bts_nanobts.py
M src/osmo_gsm_tester/resource.py
2 files changed, 1 insertion(+), 6 deletions(-)



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

diff --git a/src/osmo_gsm_tester/bts_nanobts.py b/src/osmo_gsm_tester/bts_nanobts.py
index 5fe8bae..4cdb98e 100644
--- a/src/osmo_gsm_tester/bts_nanobts.py
+++ b/src/osmo_gsm_tester/bts_nanobts.py
@@ -92,7 +92,7 @@
 
     def start(self, keepalive=False):
         if self.conf.get('ipa_unit_id') is None:
-            raise log.Error('No attribute %s provided in conf!' % attr)
+            raise log.Error('No attribute ipa_unit_id provided in conf!')
         self.run_dir = util.Dir(self.suite_run.get_test_run_dir().new_dir(self.name()))
         self._configure()
 
diff --git a/src/osmo_gsm_tester/resource.py b/src/osmo_gsm_tester/resource.py
index 29f641e..98651ce 100644
--- a/src/osmo_gsm_tester/resource.py
+++ b/src/osmo_gsm_tester/resource.py
@@ -395,11 +395,6 @@
             raise RuntimeError('adding a list of resources to itself?')
         config.add(self, copy.deepcopy(more))
 
-    def combine(self, more_rules):
-        if more_rules is self:
-            raise RuntimeError('combining a list of resource rules with itself?')
-        config.combine(self, copy.deepcopy(more))
-
     def mark_reserved_by(self, origin_id):
         for key, item_list in self.items():
             for item in item_list:

-- 
To view, visit https://gerrit.osmocom.org/13072
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iba18a96ff804780e66c69b459e112f9d42b7b2ff
Gerrit-Change-Number: 13072
Gerrit-PatchSet: 1
Gerrit-Owner: Holger Freyther <holger at freyther.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190227/d63ee8eb/attachment.htm>


More information about the gerrit-log mailing list