<p>ninjab3s has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/25132">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Fix Amarisoft local execution<br><br>Passing an asterisk as a string into a local process does not work<br>because it is interpreted as a string. To make this work we have<br>to adress the binaries directly by name instead of using something<br>like 'lteue-*'.<br><br>Change-Id: I50adcef36694f05613c2c7ce10ad5828290d6e9c<br>---<br>M src/osmo_gsm_tester/obj/epc_amarisoft.py<br>M src/osmo_gsm_tester/obj/ms_amarisoft.py<br>2 files changed, 6 insertions(+), 4 deletions(-)<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/32/25132/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo_gsm_tester/obj/epc_amarisoft.py b/src/osmo_gsm_tester/obj/epc_amarisoft.py</span><br><span>index 1534d87..e2aad4e 100644</span><br><span>--- a/src/osmo_gsm_tester/obj/epc_amarisoft.py</span><br><span>+++ b/src/osmo_gsm_tester/obj/epc_amarisoft.py</span><br><span>@@ -101,7 +101,7 @@</span><br><span>         self.process.launch()</span><br><span> </span><br><span>     def start_locally(self):</span><br><span style="color: hsl(0, 100%, 40%);">-        binary = self.inst.child('', BINFILE)</span><br><span style="color: hsl(120, 100%, 40%);">+        binary = self.inst.child('', AmarisoftEPC.BINFILE)</span><br><span> </span><br><span>         env = {}</span><br><span>         # setting capabilities will later disable use of LD_LIBRARY_PATH from ELF loader -> modify RPATH instead.</span><br><span>diff --git a/src/osmo_gsm_tester/obj/ms_amarisoft.py b/src/osmo_gsm_tester/obj/ms_amarisoft.py</span><br><span>index 7e0cc13..27be14e 100644</span><br><span>--- a/src/osmo_gsm_tester/obj/ms_amarisoft.py</span><br><span>+++ b/src/osmo_gsm_tester/obj/ms_amarisoft.py</span><br><span>@@ -180,14 +180,16 @@</span><br><span> </span><br><span>         # setting capabilities will later disable use of LD_LIBRARY_PATH from ELF loader -> modify RPATH instead.</span><br><span>         self.log('Setting RPATH for lteue')</span><br><span style="color: hsl(0, 100%, 40%);">-        util.change_elf_rpath(binary, util.prepend_library_path(self.inst), self.run_dir.new_dir('patchelf'))</span><br><span style="color: hsl(120, 100%, 40%);">+        util.change_elf_rpath(binary, str(util.prepend_library_path(self.inst)), self.run_dir.new_dir('patchelf'))</span><br><span>         # We also need to patch the arch-optimized binaries that lteue() will exec() into:</span><br><span style="color: hsl(0, 100%, 40%);">-        util.change_elf_rpath(self.inst.child('', 'lteue-*'), util.prepend_library_path(self.inst), self.run_dir.new_dir('patchelf2'))</span><br><span style="color: hsl(120, 100%, 40%);">+        util.change_elf_rpath(self.inst.child('', 'lteue-avx'), str(util.prepend_library_path(self.inst)), self.run_dir.new_dir('patchelf2'))</span><br><span style="color: hsl(120, 100%, 40%);">+        util.change_elf_rpath(self.inst.child('', 'lteue-avx2'), str(util.prepend_library_path(self.inst)), self.run_dir.new_dir('patchelf2'))</span><br><span> </span><br><span>         # lteue requires CAP_NET_ADMIN to create tunnel devices: ioctl(TUNSETIFF):</span><br><span>         self.log('Applying CAP_NET_ADMIN capability to lteue')</span><br><span>         util.setcap_net_admin(binary, self.run_dir.new_dir('setcap_net_admin'))</span><br><span style="color: hsl(0, 100%, 40%);">-        util.setcap_net_admin(self.inst.child('', 'lteue-*'), self.run_dir.new_dir('setcap_net_admin2'))</span><br><span style="color: hsl(120, 100%, 40%);">+        util.setcap_net_admin(self.inst.child('', 'lteue-avx'), str(self.run_dir.new_dir('setcap_net_admin2')))</span><br><span style="color: hsl(120, 100%, 40%);">+        util.setcap_net_admin(self.inst.child('', 'lteue-avx2'), str(self.run_dir.new_dir('setcap_net_admin2')))</span><br><span> </span><br><span>         args = (binary, os.path.abspath(self.config_file))</span><br><span>         self.dbg(run_dir=self.run_dir, binary=binary, env=env)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/25132">change 25132</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/+/25132"/><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: I50adcef36694f05613c2c7ce10ad5828290d6e9c </div>
<div style="display:none"> Gerrit-Change-Number: 25132 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: ninjab3s <nils.fuerste@softwareradiosystems.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>