<p>pespin <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17686">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">srsenb: No need to generates values dict for every template file<br><br>Let's only do all the dictionary processing once and not for each<br>template file.<br><br>Change-Id: I4684d26375fce1e2ae28d3eccdc4f607fa5411a9<br>---<br>M src/osmo_gsm_tester/srs_enb.py<br>1 file changed, 12 insertions(+), 14 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo_gsm_tester/srs_enb.py b/src/osmo_gsm_tester/srs_enb.py</span><br><span>index 1df8074..d4bdcaa 100644</span><br><span>--- a/src/osmo_gsm_tester/srs_enb.py</span><br><span>+++ b/src/osmo_gsm_tester/srs_enb.py</span><br><span>@@ -181,9 +181,15 @@</span><br><span>         self.suite_run.remember_to_stop(self.process)</span><br><span>         self.process.launch()</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    def gen_conf_file(self, path, filename):</span><br><span style="color: hsl(0, 100%, 40%);">-        self.dbg(config_file=path)</span><br><span style="color: hsl(120, 100%, 40%);">+    def gen_conf_file(self, path, filename, values):</span><br><span style="color: hsl(120, 100%, 40%);">+        self.dbg('srsENB ' + filename + ':\n' + pprint.pformat(values))</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+        with open(path, 'w') as f:</span><br><span style="color: hsl(120, 100%, 40%);">+            r = template.render(filename, values)</span><br><span style="color: hsl(120, 100%, 40%);">+            self.dbg(r)</span><br><span style="color: hsl(120, 100%, 40%);">+            f.write(r)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+    def configure(self):</span><br><span>         values = dict(enb=config.get_defaults('enb'))</span><br><span>         config.overlay(values, dict(enb=config.get_defaults('srsenb')))</span><br><span>         config.overlay(values, dict(enb=self.suite_run.config().get('enb', {})))</span><br><span>@@ -214,14 +220,6 @@</span><br><span>                         + ',id=enb,base_srate=' + str(base_srate)</span><br><span>             config.overlay(values, dict(enb=dict(rf_dev_args=rf_dev_args)))</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        self.dbg('srsENB ' + filename + ':\n' + pprint.pformat(values))</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-        with open(path, 'w') as f:</span><br><span style="color: hsl(0, 100%, 40%);">-            r = template.render(filename, values)</span><br><span style="color: hsl(0, 100%, 40%);">-            self.dbg(r)</span><br><span style="color: hsl(0, 100%, 40%);">-            f.write(r)</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    def configure(self):</span><br><span>         self.config_file = self.run_dir.child(srsENB.CFGFILE)</span><br><span>         self.config_sib_file = self.run_dir.child(srsENB.CFGFILE_SIB)</span><br><span>         self.config_rr_file = self.run_dir.child(srsENB.CFGFILE_RR)</span><br><span>@@ -229,10 +227,10 @@</span><br><span>         self.log_file = self.run_dir.child(srsENB.LOGFILE)</span><br><span>         self.pcap_file = self.run_dir.child(srsENB.PCAPFILE)</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        self.gen_conf_file(self.config_file, srsENB.CFGFILE)</span><br><span style="color: hsl(0, 100%, 40%);">-        self.gen_conf_file(self.config_sib_file, srsENB.CFGFILE_SIB)</span><br><span style="color: hsl(0, 100%, 40%);">-        self.gen_conf_file(self.config_rr_file, srsENB.CFGFILE_RR)</span><br><span style="color: hsl(0, 100%, 40%);">-        self.gen_conf_file(self.config_drb_file, srsENB.CFGFILE_DRB)</span><br><span style="color: hsl(120, 100%, 40%);">+        self.gen_conf_file(self.config_file, srsENB.CFGFILE, values)</span><br><span style="color: hsl(120, 100%, 40%);">+        self.gen_conf_file(self.config_sib_file, srsENB.CFGFILE_SIB, values)</span><br><span style="color: hsl(120, 100%, 40%);">+        self.gen_conf_file(self.config_rr_file, srsENB.CFGFILE_RR, values)</span><br><span style="color: hsl(120, 100%, 40%);">+        self.gen_conf_file(self.config_drb_file, srsENB.CFGFILE_DRB, values)</span><br><span> </span><br><span>     def ue_add(self, ue):</span><br><span>         if self.ue is not None:</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17686">change 17686</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17686"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4684d26375fce1e2ae28d3eccdc4f607fa5411a9 </div>
<div style="display:none"> Gerrit-Change-Number: 17686 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>