Change in docker-playground[master]: osmo-gsm-tester: Point config parameter to file instead of its dirname

2020-05-12 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/18203 )

Change subject: osmo-gsm-tester: Point config parameter to file instead of its 
dirname
..


Patch Set 1: Verified+1 Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/18203
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
Gerrit-Change-Number: 18203
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 12 May 2020 12:53:30 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in docker-playground[master]: osmo-gsm-tester: Point config parameter to file instead of its dirname

2020-05-12 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/18203 )

Change subject: osmo-gsm-tester: Point config parameter to file instead of its 
dirname
..

osmo-gsm-tester: Point config parameter to file instead of its dirname

Since osmo-gsm-tester.git Change-Id Ieca65b71b543c44cfcec8e83efd0fe053c432e55,
the -c parameter holds the config file and not the directory where it is 
contained.

Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
---
M osmo-gsm-tester/jenkins.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved; Verified



diff --git a/osmo-gsm-tester/jenkins.sh b/osmo-gsm-tester/jenkins.sh
index 89f7116..5b1541b 100755
--- a/osmo-gsm-tester/jenkins.sh
+++ b/osmo-gsm-tester/jenkins.sh
@@ -80,7 +80,7 @@
/bin/sh -c "/data/osmo-gsm-tester-slave.sh >/data/sshd.log 2>&1"

 echo Starting container with osmo-gsm-tester main unit
-OSMO_GSM_TESTER_CONF=${OSMO_GSM_TESTER_CONF:-/tmp/osmo-gsm-tester/sysmocom}
+OSMO_GSM_TESTER_CONF=${OSMO_GSM_TESTER_CONF:-/tmp/osmo-gsm-tester/sysmoco/main.conf}
 OSMO_GSM_TESTER_OPTS=${OSMO_GSM_TESTER_OPTS:--T -l dbg -s 
4g:srsenb-rftype@zmq+srsue-rftype@zmq -t ping}
 docker run --rm \
--cap-add=NET_ADMIN \

--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/18203
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
Gerrit-Change-Number: 18203
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in docker-playground[master]: osmo-gsm-tester: Point config parameter to file instead of its dirname

2020-05-11 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/18203 )

Change subject: osmo-gsm-tester: Point config parameter to file instead of its 
dirname
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/18203
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
Gerrit-Change-Number: 18203
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 11 May 2020 20:01:48 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in docker-playground[master]: osmo-gsm-tester: Point config parameter to file instead of its dirname

2020-05-11 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/docker-playground/+/18203 )


Change subject: osmo-gsm-tester: Point config parameter to file instead of its 
dirname
..

osmo-gsm-tester: Point config parameter to file instead of its dirname

Since osmo-gsm-tester.git Change-Id Ieca65b71b543c44cfcec8e83efd0fe053c432e55,
the -c parameter holds the config file and not the directory where it is 
contained.

Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
---
M osmo-gsm-tester/jenkins.sh
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/docker-playground 
refs/changes/03/18203/1

diff --git a/osmo-gsm-tester/jenkins.sh b/osmo-gsm-tester/jenkins.sh
index 89f7116..5b1541b 100755
--- a/osmo-gsm-tester/jenkins.sh
+++ b/osmo-gsm-tester/jenkins.sh
@@ -80,7 +80,7 @@
/bin/sh -c "/data/osmo-gsm-tester-slave.sh >/data/sshd.log 2>&1"

 echo Starting container with osmo-gsm-tester main unit
-OSMO_GSM_TESTER_CONF=${OSMO_GSM_TESTER_CONF:-/tmp/osmo-gsm-tester/sysmocom}
+OSMO_GSM_TESTER_CONF=${OSMO_GSM_TESTER_CONF:-/tmp/osmo-gsm-tester/sysmoco/main.conf}
 OSMO_GSM_TESTER_OPTS=${OSMO_GSM_TESTER_OPTS:--T -l dbg -s 
4g:srsenb-rftype@zmq+srsue-rftype@zmq -t ping}
 docker run --rm \
--cap-add=NET_ADMIN \

--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/18203
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5aa0507d0e82616ee3cca74573fea6bdb7459b53
Gerrit-Change-Number: 18203
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-MessageType: newchange