[MERGED] osmo-gsm-tester[master]: log state dir

[Neels Hofmeyr]

Neels Hofmeyr has submitted this change and it was merged.

Change subject: log state dir
......................................................................


log state dir

On the log output, clearly show which state dir is used. Since all runs should
use the same state dir, this is important and/or reassuring to see.

Change-Id: Ie92f1879a35d8bb399ee916b7ef72f9ee7d47409
---
M selftest/suite_test.ok
M src/osmo_gsm_tester/suite.py
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Pau Espin Pedrol: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/selftest/suite_test.ok b/selftest/suite_test.ok
index bb04e49..fda77dc 100644
--- a/selftest/suite_test.ok
+++ b/selftest/suite_test.ok
@@ -25,7 +25,7 @@
 cnf -: DBG: Found config file resources.conf as [PATH]/selftest/suite_test/resources.conf in ./suite_test which is [PATH]/selftest/suite_test
 cnf -: DBG: Found path state_dir as [PATH]/selftest/suite_test/test_work/state_dir
 tst test_suite: Suite run start
-tst test_suite: reserving resources...
+tst test_suite: reserving resources in [PATH]/selftest/suite_test/test_work/state_dir ...
 tst test_suite: DBG: {combining='resources'}  [test_suite↪test_suite]
 tst test_suite: DBG: {definition_conf={bts=[{'times': '1'}], modem=[{'times': '2'}], nitb_iface=[{'times': '1'}]}}  [test_suite↪(combining_scenarios='resources')↪test_suite]
 tst test_suite: Reserving 1 x bts (candidates: 3)  [test_suite↪test_suite]
diff --git a/src/osmo_gsm_tester/suite.py b/src/osmo_gsm_tester/suite.py
index affb9ad..43e55af 100644
--- a/src/osmo_gsm_tester/suite.py
+++ b/src/osmo_gsm_tester/suite.py
@@ -186,7 +186,7 @@
     def reserve_resources(self):
         if self.reserved_resources:
             raise RuntimeError('Attempt to reserve resources twice for a SuiteRun')
-        self.log('reserving resources...')
+        self.log('reserving resources in', self.resources_pool.state_dir, '...')
         with self:
             self.reserved_resources = self.resources_pool.reserve(self, self.resource_requirements())
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie92f1879a35d8bb399ee916b7ef72f9ee7d47409
Gerrit-PatchSet: 3
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>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>