osmith submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
testenv: fix duplicate 2>&1

The pipe variable already has 2>&1, therefore we don't need to add it in
cmd.

Change-Id: Ifeae4c53b57f90242041e61ca5ab7073d155d7a8
---
M _testenv/testenv/daemons.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/_testenv/testenv/daemons.py b/_testenv/testenv/daemons.py
index 9e274d9..c211cd7 100644
--- a/_testenv/testenv/daemons.py
+++ b/_testenv/testenv/daemons.py
@@ -49,7 +49,7 @@
pipe = f"2>&1 | tee {shlex.quote(log)}"
else:
pipe = f">{shlex.quote(log)} 2>&1"
- cmd = ["sh", "-c", f"{program} 2>&1 {pipe}"]
+ cmd = ["sh", "-c", f"{program} {pipe}"]

env = {}
if testenv.args.io_uring:

To view, visit change 37897. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ifeae4c53b57f90242041e61ca5ab7073d155d7a8
Gerrit-Change-Number: 37897
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>