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

Holger Freyther gerrit-no-reply at
Fri Mar 1 15:09:05 UTC 2019

Holger Freyther has submitted this change and it was merged. ( )

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/
M src/osmo_gsm_tester/
2 files changed, 1 insertion(+), 6 deletions(-)

  Pau Espin Pedrol: Looks good to me, approved
  Holger Freyther: Verified

diff --git a/src/osmo_gsm_tester/ b/src/osmo_gsm_tester/
index 5fe8bae..4cdb98e 100644
--- a/src/osmo_gsm_tester/
+++ b/src/osmo_gsm_tester/
@@ -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(
diff --git a/src/osmo_gsm_tester/ b/src/osmo_gsm_tester/
index 29f641e..98651ce 100644
--- a/src/osmo_gsm_tester/
+++ b/src/osmo_gsm_tester/
@@ -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
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iba18a96ff804780e66c69b459e112f9d42b7b2ff
Gerrit-Change-Number: 13072
Gerrit-PatchSet: 2
Gerrit-Owner: Holger Freyther <holger at>
Gerrit-Reviewer: Holger Freyther <holger at>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list