Pau Espin Pedrol has posted comments on this change. ( 
https://gerrit.osmocom.org/12209 )

Change subject: ms_driver: Add a virtual bts and make use of it
......................................................................


Patch Set 1:

(4 comments)

https://gerrit.osmocom.org/#/c/12209/1/example/resources.conf.prod
File example/resources.conf.prod:

https://gerrit.osmocom.org/#/c/12209/1/example/resources.conf.prod@4
PS1, Line 4: - addr: 10.42.42.2
           : - addr: 10.42.42.3
           : - addr: 10.42.42.4
           : - addr: 10.42.42.5
           : - addr: 10.42.42.6
           : - addr: 10.42.42.7
           : - addr: 10.42.42.8
           : - addr: 10.42.42.9
           : - addr: 10.42.42.10
> neels/pau: Any idea how this can end with the following error? […]
jenkins osmo-gsm-tester job 1686 running the wole test battery failed 
(connection lost between jenkins master and slave) and unfortunately what I 
observed so far is that jenkins sends a kill -9 in that case, not allowing 
osmo-gsm-tester to cleanup resources properly. This kind of cases require 
manual intervention (rm /var/tmp/osmo-gsm-tester/state/*).

I fixed the corrupted state, but you'll need to set up a proper IP addr in the 
osmo-bts-virtual below to have it working properly.


https://gerrit.osmocom.org/#/c/12209/1/example/resources.conf.prod@115
PS1, Line 115:   ipa_unit_id: 11
This ipa_unit_id is already used by LimeSDR-USB (line 54), use another one. 13 
is fine.


https://gerrit.osmocom.org/#/c/12209/1/example/resources.conf.prod@116
PS1, Line 116:   addr: 10.42.42.123
This IP addr doesn't seem to be set up in the osmo-gsm-tester main unit. 
Furthremore we use this 120ish range for DHCP based BTS (like nanobts, see 
osmo-ci.git). Better use .55 for this one.

You need to submit a patch against osmo-ci.git adding the IP addr so ansible 
can set it up: osmo-ci/ansible/roles/gsm-tester-network/templates/interface.j2

Ping me if you want me to set the IP addr into the main unit.


https://gerrit.osmocom.org/#/c/12209/1/example/resources.conf.prod@118
PS1, Line 118:   num_trx: 1
num_trx: 1 is the default (see bsc_bts in example/defaults.conf), no need to 
put it here.



--
To view, visit https://gerrit.osmocom.org/12209
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9f5a2501eb4473ccf2287c902ee207c6a45a1bc5
Gerrit-Change-Number: 12209
Gerrit-PatchSet: 1
Gerrit-Owner: Holger Freyther <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-CC: Pau Espin Pedrol <[email protected]>
Gerrit-Comment-Date: Mon, 10 Dec 2018 09:37:24 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No

Reply via email to