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 <hol...@freyther.de>