<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/10194">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bts: Recreate resources.conf trx_list with len based on num_trx<br><br>We are already doing this for defaults.cfg, but not for resources.conf.<br>As a result, if we have a trx_list with 2 trx but we have set num_trx=1<br>(default), parsing will fail later in bsc.cfg.tmpl because<br>conf_for_bsc_prepare() will pass a trx_list with 2 trx.<br><br>Change-Id: I28ed34abeedaa0ee2e7862ced45a46042192d831<br>---<br>M src/osmo_gsm_tester/bts.py<br>1 file changed, 7 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/94/10194/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo_gsm_tester/bts.py b/src/osmo_gsm_tester/bts.py</span><br><span>index ca33eb4..0f3ef1c 100644</span><br><span>--- a/src/osmo_gsm_tester/bts.py</span><br><span>+++ b/src/osmo_gsm_tester/bts.py</span><br><span>@@ -20,6 +20,7 @@</span><br><span> import os</span><br><span> import pprint</span><br><span> import tempfile</span><br><span style="color: hsl(120, 100%, 40%);">+import copy</span><br><span> from abc import ABCMeta, abstractmethod</span><br><span> from . import log, config, util, template, process, schema, pcu_osmo</span><br><span> </span><br><span>@@ -112,7 +113,12 @@</span><br><span>             config.overlay(values, { 'cell_identity': self.cellid })</span><br><span>         if self.bvci is not None:</span><br><span>             config.overlay(values, { 'bvci': self.bvci })</span><br><span style="color: hsl(0, 100%, 40%);">-        config.overlay(values, self.conf)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+        conf = copy.deepcopy(self.conf)</span><br><span style="color: hsl(120, 100%, 40%);">+        trx_list = conf.get('trx_list')</span><br><span style="color: hsl(120, 100%, 40%);">+        if trx_list and len(trx_list) != self.num_trx():</span><br><span style="color: hsl(120, 100%, 40%);">+            conf['trx_list'] = Bts._trx_list_recreate(trx_list, self.num_trx())</span><br><span style="color: hsl(120, 100%, 40%);">+        config.overlay(values, conf)</span><br><span> </span><br><span>         sgsn_conf = {} if self.sgsn is None else self.sgsn.conf_for_client()</span><br><span>         config.overlay(values, sgsn_conf)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10194">change 10194</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/10194"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I28ed34abeedaa0ee2e7862ced45a46042192d831 </div>
<div style="display:none"> Gerrit-Change-Number: 10194 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>