Change in osmo-gsm-tester[master]: 4g: add checks for received PRACHs for ping tests

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22043 )

Change subject: 4g: add checks for received PRACHs for ping tests
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22043
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib23056b1ad4b18a7dde6cd0c997f5114ac2acb61
Gerrit-Change-Number: 22043
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:22:31 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: enb: add stdout counter helper to eNB objects

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042 )

Change subject: enb: add stdout counter helper to eNB objects
..


Patch Set 1: Code-Review-1

(3 comments)

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/enb.py
File src/osmo_gsm_tester/obj/enb.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/enb.py@349
PS1, Line 349: def get_counter(self, counter_name):
I'd rather leave it as @abstractmethod here and make sure all implementations 
either implement it correctly or simply implement them by raising an exception 
like yo did here.

This way it's easier to find out when implementing a subclass what's missing 
before running into it (thanks to ABCMeta)


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/enb_srs.py
File src/osmo_gsm_tester/obj/enb_srs.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/enb_srs.py@89
PS1, Line 89: # Make sure we give time to tear down
Not related to this commit, submit in a separate commit please.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/srslte_common.py
File src/osmo_gsm_tester/obj/srslte_common.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042/1/src/osmo_gsm_tester/obj/srslte_common.py@102
PS1, Line 102: def _get_counter_stdout(self, keyword):
What about moving this function to Process class? as in 
process.count_lines(match=keyword, stream=stdout) or something alike.



--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I434f072b8aa6f4dce9f90889c6b40832f6798ff8
Gerrit-Change-Number: 22042
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:21:03 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsenb.conf.tmpl: set nof_of_prach_threds to 0 for ZMQ

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22041 )

Change subject: srsenb.conf.tmpl: set nof_of_prach_threds to 0 for ZMQ
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22041
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I97eddb5f9ca682251ded1f73093c2d0a49462bd6
Gerrit-Change-Number: 22041
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:15:09 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsepc.conf.tmpl: increase default value of T3413 (paging timer)

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22040 )

Change subject: srsepc.conf.tmpl: increase default value of T3413 (paging timer)
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22040
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Icc75ee048c2cc605ab26c4c87a039519998f7e57
Gerrit-Change-Number: 22040
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:14:25 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: enb_srs: adopt throughput thresholds for QAM256

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22039 )

Change subject: enb_srs: adopt throughput thresholds for QAM256
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22039
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I0e38fe28002fd68c768cc8dcffcf74f4f190df02
Gerrit-Change-Number: 22039
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:14:08 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: enb_srs: only use QAM64 tables if both UE and eNB are configured for ...

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22038 )

Change subject: enb_srs: only use QAM64 tables if both UE and eNB are 
configured for QAM64
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22038
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I599df92d69eeb56a5d44327de08f004222cff073
Gerrit-Change-Number: 22038
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:13:24 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: iperf3: return 0 if iperf results can't be read back

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22037 )

Change subject: iperf3: return 0 if iperf results can't be read back
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22037
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5a8fe32f0a2bb676dbb80dad500bc6c63c176485
Gerrit-Change-Number: 22037
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:11:00 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsue.conf.tmpl: disable TTI stats for ZMQ

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22036 )

Change subject: srsue.conf.tmpl: disable TTI stats for ZMQ
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22036
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5c2cb0abcfce0ee67806f6611356f4d5d180541d
Gerrit-Change-Number: 22036
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:10:08 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: intraenb-intrafreq-ho.conf: add eNB to cell config

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22035 )

Change subject: intraenb-intrafreq-ho.conf: add eNB to cell config
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22035
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Id494ac1984846e78a6456bb6a17e72d63c9dacf5
Gerrit-Change-Number: 22035
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:09:47 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsue.conf.tmpl: reorder conditions for setting RRC release

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22034 )

Change subject: srsue.conf.tmpl: reorder conditions for setting RRC release
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22034
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I2fa49f0cb5d80db412a811ceeb380359c8ad67a7
Gerrit-Change-Number: 22034
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:09:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: enb: include name label and run_addr in enb remote path

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22033 )

Change subject: enb: include name label and run_addr in enb remote path
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22033
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I9c979de852a23041e43692e97ebcbfb389b52fdb
Gerrit-Change-Number: 22033
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:07:36 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsenb_drb.conf.tmpl: add QCI1 DRB config

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22032 )

Change subject: srsenb_drb.conf.tmpl: add QCI1 DRB config
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22032
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I43f408b1a3c172a5eb8327f03df49801e063e5cd
Gerrit-Change-Number: 22032
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:07:19 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: jenkins-build-srslte.sh: allow custom binaries

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22031 )

Change subject: jenkins-build-srslte.sh: allow custom binaries
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22031
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I63f5799328435ce32b5bfd6dae6fd0b37e1e5397
Gerrit-Change-Number: 22031
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 17:07:08 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: 4g: add checks for received PRACHs for ping tests

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22043 )


Change subject: 4g: add checks for received PRACHs for ping tests
..

4g: add checks for received PRACHs for ping tests

the tests already check for sent PRACHs, this will
also check for received PRACHs on the eNB to make
sure we don't detect any fake PRACHs during the tests.

Change-Id: Ib23056b1ad4b18a7dde6cd0c997f5114ac2acb61
---
M sysmocom/suites/4g/rrc_idle_mo_ping.py
M sysmocom/suites/4g/rrc_idle_mt_ping.py
2 files changed, 19 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/43/22043/1

diff --git a/sysmocom/suites/4g/rrc_idle_mo_ping.py 
b/sysmocom/suites/4g/rrc_idle_mo_ping.py
index ced798d..389e5a2 100755
--- a/sysmocom/suites/4g/rrc_idle_mo_ping.py
+++ b/sysmocom/suites/4g/rrc_idle_mo_ping.py
@@ -31,9 +31,16 @@
 output = proc.get_stdout()

 # Check PRACH transmissions
+num_prachs = 2
 num_prach_sent = ue.get_counter('prach_sent')
-if num_prach_sent != 2:
-raise Exception("Expected to have sent exactly 2 PRACHs, but in fact sent 
{}".format(num_prach_sent))
+if num_prach_sent != num_prachs:
+raise Exception("Expected to have sent exactly {} PRACHs, but in fact sent 
{}".format(num_prachs, num_prach_sent))

+# Check PRACH receptions
+num_prach_received = enb.get_counter('prach_received')
+if num_prach_sent != num_prachs:
+raise Exception("Expected to have received exactly {} PRACHs, but in fact 
received {}".format(num_prachs, num_prach_received))
+
+output += 
"\nnum_prach_sent={}\nnum_prach_received={}\n".format(num_prach_sent, 
num_prach_received)
 print(output)
 test.set_report_stdout(output)
diff --git a/sysmocom/suites/4g/rrc_idle_mt_ping.py 
b/sysmocom/suites/4g/rrc_idle_mt_ping.py
index dcabe21..bdebb26 100755
--- a/sysmocom/suites/4g/rrc_idle_mt_ping.py
+++ b/sysmocom/suites/4g/rrc_idle_mt_ping.py
@@ -26,7 +26,7 @@
 # Wait a bit
 sleep(5)

-# Generate MO traffic, send single ping
+# Generate MT traffic, send single ping
 proc = epc.prepare_process('ping', ('ping', '-c', '1', ue.get_assigned_addr()))
 proc.launch_sync()
 output = proc.get_stdout()
@@ -37,9 +37,16 @@
 raise Exception("Expected to receive exactly 1 paging message, but in fact 
received {}".format(num_paging_received))

 # Check PRACH transmissions
+num_prachs = 2
 num_prach_sent = ue.get_counter('prach_sent')
-if num_prach_sent != 2:
-raise Exception("Expected to have sent exactly 2 PRACHs, but in fact sent 
{}".format(num_prach_sent))
+if num_prach_sent != num_prachs:
+raise Exception("Expected to have sent exactly {} PRACHs, but in fact sent 
{}".format(num_prachs, num_prach_sent))

+# Check PRACH receptions
+num_prach_received = enb.get_counter('prach_received')
+if num_prach_sent != num_prachs:
+raise Exception("Expected to have received exactly {} PRACHs, but in fact 
received {}".format(num_prachs, num_prach_received))
+
+output += 
"\nnum_prach_sent={}\nnum_prach_received={}\n".format(num_prach_sent, 
num_prach_received)
 print(output)
 test.set_report_stdout(output)

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22043
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib23056b1ad4b18a7dde6cd0c997f5114ac2acb61
Gerrit-Change-Number: 22043
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: enb: add stdout counter helper to eNB objects

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042 )


Change subject: enb: add stdout counter helper to eNB objects
..

enb: add stdout counter helper to eNB objects

this patch adds the stdout counter to count events happening
on the stdout (known from the UE already) to the common
srslte class so they can also be used from the eNB.

In addition, we add a PRACH counter to be used for tests.

Change-Id: I434f072b8aa6f4dce9f90889c6b40832f6798ff8
---
M src/osmo_gsm_tester/obj/enb.py
M src/osmo_gsm_tester/obj/enb_amarisoft.py
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
M src/osmo_gsm_tester/obj/srslte_common.py
5 files changed, 33 insertions(+), 11 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/42/22042/1

diff --git a/src/osmo_gsm_tester/obj/enb.py b/src/osmo_gsm_tester/obj/enb.py
index 3944459..2eac0fe 100644
--- a/src/osmo_gsm_tester/obj/enb.py
+++ b/src/osmo_gsm_tester/obj/enb.py
@@ -346,4 +346,7 @@
 def addr(self):
 return self._run_node.run_addr()

+def get_counter(self, counter_name):
+raise log.Error('get_counter() not implemented!')
+
 # vim: expandtab tabstop=4 shiftwidth=4
diff --git a/src/osmo_gsm_tester/obj/enb_amarisoft.py 
b/src/osmo_gsm_tester/obj/enb_amarisoft.py
index ef28100..62c5b8a 100644
--- a/src/osmo_gsm_tester/obj/enb_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/enb_amarisoft.py
@@ -247,6 +247,20 @@
 def running(self):
 return not self.process.terminated()

+def get_counter(self, counter_name):
+if counter_name == 'prach_received':
+return self._get_counter_stdout('PRACH:')
+raise log.Error('counter %s not implemented!' % counter_name)
+
+def _get_counter_stdout(self, keyword):
+# Match stdout against keyword
+n = 0
+stdout_lines = (self.process.get_stdout() or '').splitlines()
+for l in stdout_lines:
+if keyword in l:
+n += 1
+return n
+
 def get_rfemu(self, cell=0, dl=True):
 cell_list = self.gen_conf['enb'].get('cell_list', None)
 if cell_list is None or len(cell_list) < cell + 1:
diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 6afc873..221259a 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -86,7 +86,7 @@
 if self._run_node.is_local():
 return

-# Make sure we give the UE time to tear down
+# Make sure we give time to tear down
 self.sleep_after_stop()

 # copy back files (may not exist, for instance if there was an early 
error of process):
@@ -258,6 +258,11 @@
 def running(self):
 return not self.process.terminated()

+def get_counter(self, counter_name):
+if counter_name == 'prach_received':
+return self._get_counter_stdout('RACH:')
+raise log.Error('counter %s not implemented!' % counter_name)
+
 def get_rfemu(self, cell=0, dl=True):
 cell_list = self.gen_conf['enb'].get('cell_list', None)
 if cell_list is None or len(cell_list) < cell + 1:
diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index 610d3e6..a89ba1b 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -382,15 +382,6 @@
 proc.launch_sync()
 return proc

-def _get_counter_stdout(self, keyword):
-# Match stdout against keyword
-n = 0
-stdout_lines = (self.process.get_stdout() or '').splitlines()
-for l in stdout_lines:
-if keyword in l:
-n += 1
-return n
-
 def get_counter(self, counter_name):
 if counter_name == 'handover_success':
 return self._get_counter_stdout('HO successful')
diff --git a/src/osmo_gsm_tester/obj/srslte_common.py 
b/src/osmo_gsm_tester/obj/srslte_common.py
index e221940..3acad9e 100644
--- a/src/osmo_gsm_tester/obj/srslte_common.py
+++ b/src/osmo_gsm_tester/obj/srslte_common.py
@@ -97,4 +97,13 @@
 return analyzer.get_num_phy_errors(kpi)
 except ImportError:
 self.log("Can't load KPI analyzer module.")
-return 0
\ No newline at end of file
+return 0
+
+def _get_counter_stdout(self, keyword):
+# Match stdout against keyword
+n = 0
+stdout_lines = (self.process.get_stdout() or '').splitlines()
+for l in stdout_lines:
+if keyword in l:
+n += 1
+return n

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22042
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I434f072b8aa6f4dce9f90889c6b40832f6798ff8
Gerrit-Change-Number: 

Change in osmo-gsm-tester[master]: srsenb.conf.tmpl: set nof_of_prach_threds to 0 for ZMQ

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22041 )


Change subject: srsenb.conf.tmpl: set nof_of_prach_threds to 0 for ZMQ
..

srsenb.conf.tmpl: set nof_of_prach_threds to 0 for ZMQ

this executed the PRACH search in the main thread so
PHY workers and PRACH processing don't diverge in loaded
scenarios

Change-Id: I97eddb5f9ca682251ded1f73093c2d0a49462bd6
---
M src/osmo_gsm_tester/templates/srsenb.conf.tmpl
1 file changed, 3 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/41/22041/1

diff --git a/src/osmo_gsm_tester/templates/srsenb.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
index 5df432b..7ac75bb 100644
--- a/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
@@ -271,6 +271,9 @@
 #
 #
 [expert]
+% if enb.rf_dev_type == 'zmq':
+nof_prach_threads = 0
+% endif
 #pusch_max_its= 8 # These are half iterations
 #pusch_8bit_decoder   = false
 #nof_phy_threads = 3

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22041
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I97eddb5f9ca682251ded1f73093c2d0a49462bd6
Gerrit-Change-Number: 22041
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: enb_srs: adopt throughput thresholds for QAM256

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22039 )


Change subject: enb_srs: adopt throughput thresholds for QAM256
..

enb_srs: adopt throughput thresholds for QAM256

it turned out that we have to reduce the MCS when using QAM256,
especially for 6 PRB as subframe 0 and 5 contains PBCH and PSS
signals, so the available REs are reduced.

The eNB scheduler now has this limitation in mind and lowers the MCS.

Change-Id: I0e38fe28002fd68c768cc8dcffcf74f4f190df02
---
M src/osmo_gsm_tester/obj/enb_srs.py
1 file changed, 8 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/39/22039/1

diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 1b17c3c..6afc873 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -276,14 +276,14 @@
 def ue_max_rate(self, downlink=True, num_carriers=1):
 # The max rate for a single UE per PRB configuration in TM1 with MCS 28
 if 'dl_qam256' in self.ue.features():
-max_phy_rate_tm1_dl = {6: 5.9e6,
-   15: 14.6e6,
-   25: 24.5e6,
-   50: 48.9e6,
-   75: 75.3e6,
-   100: 97.8e6}
+max_phy_rate_tm1_dl = {6: 4.4e6,
+   15: 14e6,
+   25: 24e6,
+   50: 49e6,
+   75: 75e6,
+   100: 98e6}
 else:
-max_phy_rate_tm1_dl = {6: 3.5e6,
+max_phy_rate_tm1_dl = {6: 3.3e6,
15: 11e6,
25: 18e6,
50: 36e6,
@@ -296,7 +296,7 @@
 25 : 14e6,
 50 : 32e6,
 75 : 34e6,
-100 : 70e6 }
+100 : 71e6 }
 else:
 max_phy_rate_tm1_ul = { 6 : 1.7e6,
 15 : 4.7e6,

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22039
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I0e38fe28002fd68c768cc8dcffcf74f4f190df02
Gerrit-Change-Number: 22039
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: srsepc.conf.tmpl: increase default value of T3413 (paging timer)

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22040 )


Change subject: srsepc.conf.tmpl: increase default value of T3413 (paging timer)
..

srsepc.conf.tmpl: increase default value of T3413 (paging timer)

we've seen that the default value of 2s might be a bit tight.
the value is network dependent so no real default value exists.
however, 6s seems to be a good compromise.

Change-Id: Icc75ee048c2cc605ab26c4c87a039519998f7e57
---
M src/osmo_gsm_tester/templates/srsepc.conf.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/40/22040/1

diff --git a/src/osmo_gsm_tester/templates/srsepc.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsepc.conf.tmpl
index 830cd32..62f9977 100644
--- a/src/osmo_gsm_tester/templates/srsepc.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsepc.conf.tmpl
@@ -31,7 +31,7 @@
 dns_addr = 8.8.8.8
 encryption_algo = EEA0
 integrity_algo = EIA1
-paging_timer = 2
+paging_timer = 6

 #
 # HSS configuration

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22040
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Icc75ee048c2cc605ab26c4c87a039519998f7e57
Gerrit-Change-Number: 22040
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: enb_srs: only use QAM64 tables if both UE and eNB are configured for ...

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22038 )


Change subject: enb_srs: only use QAM64 tables if both UE and eNB are 
configured for QAM64
..

enb_srs: only use QAM64 tables if both UE and eNB are configured for QAM64

this was causing failed tests because to achieve QAM64 rates both
eNB and UE need to support it and have it activated.

Change-Id: I599df92d69eeb56a5d44327de08f004222cff073
---
M src/osmo_gsm_tester/obj/enb_srs.py
1 file changed, 5 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/38/22038/1

diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 7fde719..1b17c3c 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -72,6 +72,7 @@
 self.remote_log_file = None
 self.remote_pcap_file = None
 self.enable_pcap = False
+self.enable_ul_qam64 = False
 self.metrics_file = None
 self.stop_sleep_time = 6 # We require at most 5s to stop
 self.testenv = testenv
@@ -192,7 +193,9 @@
 self.enable_pcap = util.str2bool(values['enb'].get('enable_pcap', 
'false'))
 config.overlay(values, dict(enb={'enable_pcap': self.enable_pcap}))

-config.overlay(values, dict(enb={'enable_ul_qam64': 
util.str2bool(values['enb'].get('enable_ul_qam64', 'false'))}))
+self.enable_ul_qam64 = 
util.str2bool(values['enb'].get('enable_ul_qam64', 'false'))
+config.overlay(values, dict(enb={'enable_ul_qam64': 
self.enable_ul_qam64}))
+
 config.overlay(values, dict(enb={'enable_dl_awgn': 
util.str2bool(values['enb'].get('enable_dl_awgn', 'false'))}))
 config.overlay(values, dict(enb={'rf_dev_sync': 
values['enb'].get('rf_dev_sync', None)}))

@@ -287,7 +290,7 @@
75: 55e6,
100: 75e6}

-if 'ul_qam64' in self.ue.features():
+if self.enable_ul_qam64 and 'ul_qam64' in self.ue.features():
 max_phy_rate_tm1_ul = { 6 : 2.7e6,
 15 : 6.5e6,
 25 : 14e6,

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22038
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I599df92d69eeb56a5d44327de08f004222cff073
Gerrit-Change-Number: 22038
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: iperf3: return 0 if iperf results can't be read back

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22037 )


Change subject: iperf3: return 0 if iperf results can't be read back
..

iperf3: return 0 if iperf results can't be read back

usually the calling code, i.e. the test, will use the result
of the iperf run to check against a threshold. for that to
work the return value can't be None.

Change-Id: I5a8fe32f0a2bb676dbb80dad500bc6c63c176485
---
M src/osmo_gsm_tester/obj/iperf3.py
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/37/22037/1

diff --git a/src/osmo_gsm_tester/obj/iperf3.py 
b/src/osmo_gsm_tester/obj/iperf3.py
index 4785f78..6571491 100644
--- a/src/osmo_gsm_tester/obj/iperf3.py
+++ b/src/osmo_gsm_tester/obj/iperf3.py
@@ -76,7 +76,7 @@
 return recv['bits_per_second']/1e6
 except Exception as e:
 print("Exception while using iperf3 results: %r" % (repr(result)))
-pass
+return 0

 class IPerf3Server(log.Origin):


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22037
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5a8fe32f0a2bb676dbb80dad500bc6c63c176485
Gerrit-Change-Number: 22037
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: srsue.conf.tmpl: disable TTI stats for ZMQ

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22036 )


Change subject: srsue.conf.tmpl: disable TTI stats for ZMQ
..

srsue.conf.tmpl: disable TTI stats for ZMQ

since ZMQ runs are not using wall clock anyway, measuring
TTI execution isn't useful, disable it therefore to avoid
misleading warnings.

Change-Id: I5c2cb0abcfce0ee67806f6611356f4d5d180541d
---
M src/osmo_gsm_tester/templates/srsue.conf.tmpl
1 file changed, 5 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/36/22036/1

diff --git a/src/osmo_gsm_tester/templates/srsue.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
index a57857b..a50f91e 100644
--- a/src/osmo_gsm_tester/templates/srsue.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
@@ -381,3 +381,8 @@
 metrics_csv_enable  = true
 #metrics_period_secs = 1
 metrics_csv_filename = ${ue.metrics_filename}
+
+% if ue.rf_dev_type == 'zmq':
+[stack]
+have_tti_time_stats = false
+% endif

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22036
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5c2cb0abcfce0ee67806f6611356f4d5d180541d
Gerrit-Change-Number: 22036
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: intraenb-intrafreq-ho.conf: add eNB to cell config

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22035 )


Change subject: intraenb-intrafreq-ho.conf: add eNB to cell config
..

intraenb-intrafreq-ho.conf: add eNB to cell config

make sure to have the right eNB that matches the
cells neighbor ECIDs, otherwise the eNB may trigger
a S1 HO if the eNB ID is set elsewhere

Change-Id: Id494ac1984846e78a6456bb6a17e72d63c9dacf5
---
M sysmocom/scenarios/mod-enb-cells-intraenb-intrafreq-ho.conf
1 file changed, 3 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/35/22035/1

diff --git a/sysmocom/scenarios/mod-enb-cells-intraenb-intrafreq-ho.conf 
b/sysmocom/scenarios/mod-enb-cells-intraenb-intrafreq-ho.conf
index 8b6f569..f4223a5 100644
--- a/sysmocom/scenarios/mod-enb-cells-intraenb-intrafreq-ho.conf
+++ b/sysmocom/scenarios/mod-enb-cells-intraenb-intrafreq-ho.conf
@@ -1,7 +1,9 @@
 # Cell config for Intra eNB HO, two cells on same frequencey (same EARFCN) on 
different RF ports (srsENB)
 modifiers:
   enb:
-  - cell_list:
+  - id: 0x19B
+enable_measurements: true
+cell_list:
 - cell_id: 0x01
   pci: 0x01
   tac: 0x0007

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22035
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Id494ac1984846e78a6456bb6a17e72d63c9dacf5
Gerrit-Change-Number: 22035
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: srsue.conf.tmpl: reorder conditions for setting RRC release

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22034 )


Change subject: srsue.conf.tmpl: reorder conditions for setting RRC release
..

srsue.conf.tmpl: reorder conditions for setting RRC release

the order of checks needs to go from high to low, i.e. the higher
release feature (e.g. qam256) needs to be checked and set first.

in theory it should also be possible to have a CA-capable UE
that does not support QAM256, but for srsUE we announce both anyway.

Change-Id: I2fa49f0cb5d80db412a811ceeb380359c8ad67a7
---
M src/osmo_gsm_tester/templates/srsue.conf.tmpl
1 file changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/34/22034/1

diff --git a/src/osmo_gsm_tester/templates/srsue.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
index 120b938..a57857b 100644
--- a/src/osmo_gsm_tester/templates/srsue.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
@@ -147,14 +147,14 @@
 # mbms_service_port: Port of the MBMS service
 #
 [rrc]
-% if int(ue.num_carriers) > 1:
-ue_category   = 7
-release   = 10
-% elif "dl_qam256" in ue.features or "ul_qam64" in ue.features:
+% if "dl_qam256" in ue.features or "ul_qam64" in ue.features:
 ue_category   = 8
 release   = 13
 ue_category_dl= 14
 ue_category_ul= 5
+% elif int(ue.num_carriers) > 1:
+ue_category   = 7
+release   = 10
 % else:
 #ue_category   = 4
 #release   = 8

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22034
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I2fa49f0cb5d80db412a811ceeb380359c8ad67a7
Gerrit-Change-Number: 22034
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: srsenb_drb.conf.tmpl: add QCI1 DRB config

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22032 )


Change subject: srsenb_drb.conf.tmpl: add QCI1 DRB config
..

srsenb_drb.conf.tmpl: add QCI1 DRB config

use default QCI1 config for real-time voice traffic

Change-Id: I43f408b1a3c172a5eb8327f03df49801e063e5cd
---
M src/osmo_gsm_tester/templates/srsenb_drb.conf.tmpl
1 file changed, 23 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/32/22032/1

diff --git a/src/osmo_gsm_tester/templates/srsenb_drb.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb_drb.conf.tmpl
index 5167dab..b10ef31 100644
--- a/src/osmo_gsm_tester/templates/srsenb_drb.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb_drb.conf.tmpl
@@ -1,7 +1,29 @@
 // All times are in ms. Use -1 for infinity, where available

 qci_config = (
-
+// UM - real time (RTP for VOIP)
+{
+  qci=1;
+  pdcp_config = {
+discard_timer = 100;
+pdcp_sn_size = 12;
+  }
+  rlc_config = {
+ul_um = {
+  sn_field_length = 5;
+};
+dl_um = {
+  sn_field_length = 5;
+  t_reordering= 45;
+};
+  };
+  logical_channel_config = {
+priority = 4;
+prioritized_bit_rate   = 256;
+bucket_size_duration  = 1000;
+log_chan_group = 1;
+  };
+},
 {
   qci=7;
   pdcp_config = {

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22032
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I43f408b1a3c172a5eb8327f03df49801e063e5cd
Gerrit-Change-Number: 22032
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: enb: include name label and run_addr in enb remote path

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22033 )


Change subject: enb: include name label and run_addr in enb remote path
..

enb: include name label and run_addr in enb remote path

this allows to quickly identify the enb host in the logs

Change-Id: I9c979de852a23041e43692e97ebcbfb389b52fdb
---
M src/osmo_gsm_tester/obj/enb.py
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/33/22033/1

diff --git a/src/osmo_gsm_tester/obj/enb.py b/src/osmo_gsm_tester/obj/enb.py
index cdf9505..3944459 100644
--- a/src/osmo_gsm_tester/obj/enb.py
+++ b/src/osmo_gsm_tester/obj/enb.py
@@ -89,7 +89,10 @@
 if self._gtp_bind_addr is None:
 self._gtp_bind_addr = self._run_node.run_addr()
 label = conf.get('label', None)
-self.set_name('%s_%s' % (name, label if label is not None else 
self._run_node.run_addr()))
+if label is not None:
+self.set_name('%s_%s_%s' % (name, label, 
self._run_node.run_addr()))
+else:
+self.set_name('%s_%s' % (name, self._run_node.run_addr()))
 self._txmode = 0
 self._id = None
 self._duplex = None

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22033
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I9c979de852a23041e43692e97ebcbfb389b52fdb
Gerrit-Change-Number: 22033
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: jenkins-build-srslte.sh: allow custom binaries

2021-01-07 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22031 )


Change subject: jenkins-build-srslte.sh: allow custom binaries
..

jenkins-build-srslte.sh: allow custom binaries

this allows to specify the environmental var
wanted_binaries_bin that will be included in the
trial package. if the variable isn't set, all three
main srsLTE binaries are included

Change-Id: I63f5799328435ce32b5bfd6dae6fd0b37e1e5397
---
M contrib/jenkins-build-srslte.sh
1 file changed, 5 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/31/22031/1

diff --git a/contrib/jenkins-build-srslte.sh b/contrib/jenkins-build-srslte.sh
index 80e4ffc..365c3e4 100755
--- a/contrib/jenkins-build-srslte.sh
+++ b/contrib/jenkins-build-srslte.sh
@@ -1,6 +1,10 @@
 #!/bin/sh
 set -e -x

+if [ -z "$trial_binaries" ]; then
+  trial_binaries="srsue srsenb srsepc"
+fi
+
 base="$PWD"
 name="srslte"
 git_url="${git_url:-https://github.com/srsLTE};
@@ -10,4 +14,4 @@
 #TODO: make sure libconfig, zeroMQ is installed
 build_repo $project_name $configure_opts

-create_bin_tgz "srsue srsenb srsepc"
+create_bin_tgz "$trial_binaries"

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/22031
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I63f5799328435ce32b5bfd6dae6fd0b37e1e5397
Gerrit-Change-Number: 22031
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Build failure of network:osmocom:nightly/simtrace2 in xUbuntu_20.10/x86_64

2021-01-07 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_20.10/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in 
xUbuntu_20.10/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly simtrace2

Last lines of build log:
[  184s] [COMPILING apps/dfu/main.c]
[  185s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  185s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  185s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  185s] Memory region Used Size  Region Size  %age Used
[  185s]  rom:   16580 B16 KB101.20%
[  185s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: warning: 
changing start of section .stack by 4 bytes
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  185s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 196 bytes
[  185s] collect2: error: ld returned 1 exit status
[  185s] %
[  185s] make[2]: *** [Makefile:234: flash] Error 1
[  185s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  185s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  185s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  185s] dh_auto_build: error: make -j1 returned exit code 2
[  185s] make: *** [debian/rules:16: build] Error 25
[  185s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  185s] ### VM INTERACTION START ###
[  188s] [  175.687654] sysrq: Power Off
[  188s] [  175.698840] reboot: Power down
[  188s] ### VM INTERACTION END ###
[  188s] 
[  188s] lamb24 failed "build simtrace2_0.7.0.69.aadd.dsc" at Thu Jan  7 
15:43:25 UTC 2021.
[  188s] 

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/simtrace2 in xUbuntu_20.04/x86_64

2021-01-07 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_20.04/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in 
xUbuntu_20.04/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly simtrace2

Last lines of build log:
[  118s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  119s] [COMPILING libboard/qmod/source/card_pres.c]
[  119s] [COMPILING libboard/qmod/source/wwan_led.c]
[  119s] [COMPILING libboard/qmod/source/i2c.c]
[  119s] [COMPILING libboard/qmod/source/board_qmod.c]
[  119s] [COMPILING apps/dfu/main.c]
[  119s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  119s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  119s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  120s] Memory region Used Size  Region Size  %age Used
[  120s]  rom:   16588 B16 KB101.25%
[  120s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  120s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 204 bytes
[  120s] collect2: error: ld returned 1 exit status
[  120s] %
[  120s] make[2]: *** [Makefile:234: flash] Error 1
[  120s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  120s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  120s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  120s] dh_auto_build: error: make -j1 returned exit code 2
[  120s] make: *** [debian/rules:16: build] Error 25
[  120s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  120s] ### VM INTERACTION START ###
[  123s] [  114.582452] sysrq: Power Off
[  123s] [  114.589445] reboot: Power down
[  123s] ### VM INTERACTION END ###
[  123s] 
[  123s] goat01 failed "build simtrace2_0.7.0.69.aadd.dsc" at Thu Jan  7 
15:33:53 UTC 2021.
[  123s] 

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Change in libosmocore[master]: WIP: Intoduce Packet Switch CGI

2021-01-07 Thread pespin
pespin has abandoned this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/21849 )

Change subject: WIP: Intoduce Packet Switch CGI
..


Abandoned
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/21849
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I87016d1c23f893cbb7dad2e6ff74566ad8a06b5c
Gerrit-Change-Number: 21849
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: abandon


Change in meta-telephony[laforge/nightly]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22030 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..

osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
---
M recipes-osmocom/osmo-iuh/osmo-iuh.inc
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved; Verified



diff --git a/recipes-osmocom/osmo-iuh/osmo-iuh.inc 
b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
index ab412ad..92b4633 100644
--- a/recipes-osmocom/osmo-iuh/osmo-iuh.inc
+++ b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
@@ -3,9 +3,13 @@
 LICENSE = "AGPLv3"
 LIC_FILES_CHKSUM = "file://COPYING;md5=73f1eb20517c55bf9493b7dd6e480788"

-DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c"
+DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c lksctp-tools"

-INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+INC_PR="r2.${META_TELEPHONY_OSMO_INC}"
+
+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "ac_cv_header_netinet_sctp_h=yes"

 inherit autotools pkgconfig systemd


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22030
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: laforge/nightly
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22030
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged


Change in meta-telephony[laforge/nightly]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22030 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..


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


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22030
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: laforge/nightly
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22030
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:32:39 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[laforge/nightly]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22030 )


Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..

osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
---
M recipes-osmocom/osmo-iuh/osmo-iuh.inc
1 file changed, 6 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/30/22030/1

diff --git a/recipes-osmocom/osmo-iuh/osmo-iuh.inc 
b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
index ab412ad..92b4633 100644
--- a/recipes-osmocom/osmo-iuh/osmo-iuh.inc
+++ b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
@@ -3,9 +3,13 @@
 LICENSE = "AGPLv3"
 LIC_FILES_CHKSUM = "file://COPYING;md5=73f1eb20517c55bf9493b7dd6e480788"

-DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c"
+DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c lksctp-tools"

-INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+INC_PR="r2.${META_TELEPHONY_OSMO_INC}"
+
+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "ac_cv_header_netinet_sctp_h=yes"

 inherit autotools pkgconfig systemd


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22030
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: laforge/nightly
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22030
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in libosmocore[master]: pkgconfig: link to mnl if available

2021-01-07 Thread Hoernchen
Hoernchen has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22026 )

Change subject: pkgconfig: link to mnl if available
..

pkgconfig: link to mnl if available

The dependency on mnl breaks builds of osmo-pcu et al if mnl is used for
libosmogb, but not linked in for lib users

Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
---
M libosmogb.pc.in
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, but someone else must approve
  Hoernchen: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/libosmogb.pc.in b/libosmogb.pc.in
index a163cc4..4eebe42 100644
--- a/libosmogb.pc.in
+++ b/libosmogb.pc.in
@@ -6,6 +6,6 @@
 Name: Osmocom GPRS Gb Library
 Description: Osmocom GPRS Gb Interface (NS/BSSGP) Library
 Version: @VERSION@
-Libs: -L${libdir} @TALLOC_LIBS@ -losmogb -losmovty -losmocore
+Libs: -L${libdir} @TALLOC_LIBS@ -losmogb -losmovty -losmocore @LIBMNL_LIBS@
 Cflags: -I${includedir}/ -fno-strict-aliasing


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
Gerrit-Change-Number: 22026
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in libosmocore[master]: pkgconfig: link to mnl if available

2021-01-07 Thread Hoernchen
Hoernchen has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22026 )

Change subject: pkgconfig: link to mnl if available
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
Gerrit-Change-Number: 22026
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:28:40 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: bssgp_rim: add encoder/decoder for NACC related RIM containers

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/21862 )

Change subject: bssgp_rim: add encoder/decoder for NACC related RIM containers
..


Patch Set 6:

(1 comment)

https://gerrit.osmocom.org/c/libosmocore/+/21862/6/include/osmocom/gprs/gprs_bssgp_rim.h
File include/osmocom/gprs/gprs_bssgp_rim.h:

https://gerrit.osmocom.org/c/libosmocore/+/21862/6/include/osmocom/gprs/gprs_bssgp_rim.h@7
PS6, Line 7:struct {
You could reuse "struct osmo_cell_global_id_ps" from 
https://gerrit.osmocom.org/c/libosmocore/+/22013 here.

In any case, you ruse this struct in at least 2 places, so better move its 
definition outside and reuse it.



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ibbc7fd67658e3040c12abb5706fe9d1f31894352
Gerrit-Change-Number: 21862
Gerrit-PatchSet: 6
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:19:46 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in libosmocore[master]: pkgconfig: link to mnl if available

2021-01-07 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22026 )

Change subject: pkgconfig: link to mnl if available
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
Gerrit-Change-Number: 22026
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:20:05 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: Intoduce Packet Switch CGI

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22013 )

Change subject: Intoduce Packet Switch CGI
..


Patch Set 2:

This change is ready for review.


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: If48f412c32e8e5a3e604a78d12b74787a4786374
Gerrit-Change-Number: 22013
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:18:05 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: CTRL: Introduce support to run osmocom CTRL server

2021-01-07 Thread pespin
Hello Jenkins Builder, laforge, fixeria,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012

to look at the new patch set (#2).

Change subject: CTRL: Introduce support to run osmocom CTRL server
..

CTRL: Introduce support to run osmocom CTRL server

Change-Id: I37db9962f51baf2c63bd58ec47ec89f773d7a255
---
M bsc-nat/BSCNAT_Tests.ttcn
M bsc/BSC_Tests.ttcn
M bts/BTS_Tests.ttcn
M hlr/HLR_Tests.ttcn
M library/Osmocom_CTRL_Adapter.ttcn
M library/Osmocom_CTRL_Functions.ttcn
M msc/BSC_ConnectionHandler.ttcn
M msc/MSC_Tests.ttcn
M sip/SIP_Tests.ttcn
9 files changed, 60 insertions(+), 22 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/12/22012/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I37db9962f51baf2c63bd58ec47ec89f773d7a255
Gerrit-Change-Number: 22012
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-ttcn3-hacks[master]: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field

2021-01-07 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011 )

Change subject: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field
..

CTRL: Fix tr_CtrlMsgGet containing invalid value in val field

"val" field is not used in GET commands and is hence set to omit by
TTCN3 decoder.

Change-Id: If1a273a2be71040eaea2189a0aeaf737adf848e5
---
M library/Osmocom_CTRL_Types.ttcn
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  fixeria: Looks good to me, approved



diff --git a/library/Osmocom_CTRL_Types.ttcn b/library/Osmocom_CTRL_Types.ttcn
index 52be956..36e6762 100644
--- a/library/Osmocom_CTRL_Types.ttcn
+++ b/library/Osmocom_CTRL_Types.ttcn
@@ -124,7 +124,7 @@
verb := "GET",
id := id,
variable := variable,
-   val := ?
+   val := omit
}
 }


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If1a273a2be71040eaea2189a0aeaf737adf848e5
Gerrit-Change-Number: 22011
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in libosmocore[master]: bssgp_bvc_fsm: Handle block request from application correctly

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22029 )

Change subject: bssgp_bvc_fsm: Handle block request from application correctly
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3e384b71d57e939efc1596ac1d92380ed5eb916d
Gerrit-Change-Number: 22029
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:02:18 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: pkgconfig: link to mnl if available

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22026 )

Change subject: pkgconfig: link to mnl if available
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
Gerrit-Change-Number: 22026
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:00:38 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: logging: Remove duplicate color for DLSMS

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22027 )

Change subject: logging: Remove duplicate color for DLSMS
..


Patch Set 1: Code-Review+1

Would maybe make sense remove the non-defined string.


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
Gerrit-Change-Number: 22027
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 14:01:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: ns2_frgre: Fix missing break statement for recv from IPV6

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22028 )

Change subject: ns2_frgre: Fix missing break statement for recv from IPV6
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I549277483fee55f390e0b5ae1dafec6f2d68eae7
Gerrit-Change-Number: 22028
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:59:32 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: bssgp_bvc_fsm: Handle block request from application correctly

2021-01-07 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22029 )


Change subject: bssgp_bvc_fsm: Handle block request from application correctly
..

bssgp_bvc_fsm: Handle block request from application correctly

This is no event from an incoming message so rx is NULL, and we can't send a
status PDU. Also blocking the signalling BVC is not allowed (unblocking it is 
already
forbidden).

Change-Id: I3e384b71d57e939efc1596ac1d92380ed5eb916d
Fixes: CID#215716
---
M src/gb/bssgp_bvc_fsm.c
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/29/22029/1

diff --git a/src/gb/bssgp_bvc_fsm.c b/src/gb/bssgp_bvc_fsm.c
index cc634a5..6de0028 100644
--- a/src/gb/bssgp_bvc_fsm.c
+++ b/src/gb/bssgp_bvc_fsm.c
@@ -456,7 +456,10 @@
case BSSGP_BVCFSM_E_REQ_BLOCK:
if (bfp->role_sgsn) {
LOGPFSML(fi, LOGL_ERROR, "SGSN may not initiate 
BVC-BLOCK\n");
-   _tx_status(fi, BSSGP_CAUSE_SEM_INCORR_PDU, rx);
+   break;
+   }
+   if (bfp->bvci == 0) {
+   LOGPFSML(fi, LOGL_ERROR, "BVCI 0 cannot be blocked\n");
break;
}
bfp->locally_blocked = true;

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3e384b71d57e939efc1596ac1d92380ed5eb916d
Gerrit-Change-Number: 22029
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-MessageType: newchange


Change in libosmocore[master]: logging: Remove duplicate color for DLSMS

2021-01-07 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22027 )


Change subject: logging: Remove duplicate color for DLSMS
..

logging: Remove duplicate color for DLSMS

Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
---
M src/logging.c
1 file changed, 0 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/27/22027/1

diff --git a/src/logging.c b/src/logging.c
index 01a00da..60dba64 100644
--- a/src/logging.c
+++ b/src/logging.c
@@ -177,7 +177,6 @@
.name = "DLSMS",
.description = "Layer3 Short Message Service (SMS)",
.enabled = 1, .loglevel = LOGL_NOTICE,
-   .color = OSMO_LOGCOLOR_BRIGHTWHITE,
.color = "\033[38;5;31m",
},
[INT2IDX(DLCTRL)] = {

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
Gerrit-Change-Number: 22027
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-MessageType: newchange


Change in libosmocore[master]: ns2_frgre: Fix missing break statement for recv from IPV6

2021-01-07 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22028 )


Change subject: ns2_frgre: Fix missing break statement for recv from IPV6
..

ns2_frgre: Fix missing break statement for recv from IPV6

Change-Id: I549277483fee55f390e0b5ae1dafec6f2d68eae7
Fixes: CID#215836
---
M src/gb/gprs_ns2_frgre.c
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/28/22028/1

diff --git a/src/gb/gprs_ns2_frgre.c b/src/gb/gprs_ns2_frgre.c
index 625d05c..86f0921 100644
--- a/src/gb/gprs_ns2_frgre.c
+++ b/src/gb/gprs_ns2_frgre.c
@@ -314,6 +314,7 @@
break;
case AF_INET6:
ip46hdr = sizeof(struct ip6_hdr);
+   break;
default:
*error = -EIO;
goto out_err;

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I549277483fee55f390e0b5ae1dafec6f2d68eae7
Gerrit-Change-Number: 22028
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-MessageType: newchange


Change in libosmocore[master]: pkgconfig: link to mnl if available

2021-01-07 Thread Hoernchen
Hoernchen has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22026 )


Change subject: pkgconfig: link to mnl if available
..

pkgconfig: link to mnl if available

The dependency on mnl breaks builds of osmo-pcu et al if mnl is used for
libosmogb, but not linked in for lib users

Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
---
M libosmogb.pc.in
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/26/22026/1

diff --git a/libosmogb.pc.in b/libosmogb.pc.in
index a163cc4..4eebe42 100644
--- a/libosmogb.pc.in
+++ b/libosmogb.pc.in
@@ -6,6 +6,6 @@
 Name: Osmocom GPRS Gb Library
 Description: Osmocom GPRS Gb Interface (NS/BSSGP) Library
 Version: @VERSION@
-Libs: -L${libdir} @TALLOC_LIBS@ -losmogb -losmovty -losmocore
+Libs: -L${libdir} @TALLOC_LIBS@ -losmogb -losmovty -losmocore @LIBMNL_LIBS@
 Cflags: -I${includedir}/ -fno-strict-aliasing


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib4df95d5c922f8edfa33e68645652fd30d321ff8
Gerrit-Change-Number: 22026
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-MessageType: newchange


Change in meta-telephony[201705]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22024 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22024
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22024
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:49:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: bts_features: s/Repeation/Repetition/ in osmo_bts_features_descs[]

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22021 )

Change subject: bts_features: s/Repeation/Repetition/ in 
osmo_bts_features_descs[]
..

bts_features: s/Repeation/Repetition/ in osmo_bts_features_descs[]

Change-Id: Ia8c77766842ce80eae969774959cbf46e86f97b9
---
M src/gsm/bts_features.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved



diff --git a/src/gsm/bts_features.c b/src/gsm/bts_features.c
index 93703aa..d8c27d8 100644
--- a/src/gsm/bts_features.c
+++ b/src/gsm/bts_features.c
@@ -41,7 +41,7 @@
{ BTS_FEAT_ETWS_PN, "ETWS Primary Notification via PCH" },
{ BTS_FEAT_PAGING_COORDINATION, "BSS Paging Coordination" },
{ BTS_FEAT_IPV6_NSVC,   "NSVC IPv6" },
-   { BTS_FEAT_ACCH_REP,"FACCH/SACCH Repeation" },
+   { BTS_FEAT_ACCH_REP,"FACCH/SACCH Repetition" },
{ 0, NULL }
 };


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia8c77766842ce80eae969774959cbf46e86f97b9
Gerrit-Change-Number: 22021
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in meta-telephony[201705]: update all of osmocom to "latest" tags

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22025 )

Change subject: update all of osmocom to "latest" tags
..

update all of osmocom to "latest" tags

We have to use some arcane construct to make the bitbake fetcher
actualyl fetch tags that are not on any branch (like those patch
releases that themselves branch off master, but which aren't on
master).  Otherwise we get errors like:

WARNING: osmo-sgsn-1.6.2+gitrAUTOINC+065500a347-r1.18.0 do_fetch: Failed to 
fetch URL git://git.osmocom.org/osmo-sgsn.git;protocol=git, attempting MIRRORS 
if available
ERROR: osmo-sgsn-1.6.2+gitrAUTOINC+065500a347-r1.18.0 do_fetch: Fetcher 
failure: Unable to find revision 065500a3471dd43b65dc87b95854c3015c551f0f in 
branch master even from upstream

Change-Id: I743173f8a3385686ffb11550aabc0fbc665cf043
---
M recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
M recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
M recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
M recipes-osmocom/libosmocore/libosmocore_git.bb
M recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
M recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
M recipes-osmocom/osmo-hlr/osmo-hlr_git.bb
M recipes-osmocom/osmo-iuh/osmo-iuh_git.bb
M recipes-osmocom/osmo-msc/osmo-msc_git.bb
M recipes-osmocom/osmo-sgsn/osmo-sgsn_git.bb
M recipes-osmocom/osmo-sip-connector/osmo-sip-connector_git.bb
M recipes-osmocom/osmo-sysmon/osmo-sysmon_git.bb
12 files changed, 30 insertions(+), 30 deletions(-)

Approvals:
  laforge: Looks good to me, approved; Verified



diff --git a/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb 
b/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
index 0582599..446a590 100644
--- a/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
+++ b/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "c313d44272f04b24dad38f4caea17180f9cbc509"
-SRC_URI = "git://git.osmocom.org/libosmo-abis.git;protocol=git"
-PV = "0.8.0+gitr${SRCPV}"
+SRCREV = "1.0.1"
+SRC_URI = 
"git://git.osmocom.org/libosmo-abis.git;protocol=git;tag=${SRCREV};nobranch=1"
+PV = "1.0.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb 
b/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
index c52c7e9..0f2856e 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "b8121e79ef53b30df535fb38987b3a48853ebbe9"
+SRCREV = "f3ccbecb6f802ccd98d154dc7ba7d552e298b2ea"
 SRC_URI = "git://git.osmocom.org/libosmo-netif.git;protocol=git"
-PV = "0.7.0+gitr${SRCPV}"
+PV = "1.0.0+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb 
b/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
index 31fa234..c78477c 100644
--- a/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
+++ b/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "5e4643f4bc76a1437621348550b5b2a603e2179b"
+SRCREV = "37880b696ee90dead7e3b1292b109cc041a44e52"
 SRC_URI = "git://git.osmocom.org/libosmo-sccp.git;protocol=git"
-PV = "1.2.0+gitr${SRCPV}"
+PV = "1.3.0+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmocore/libosmocore_git.bb 
b/recipes-osmocom/libosmocore/libosmocore_git.bb
index d5eab3a..1acf6ac 100644
--- a/recipes-osmocom/libosmocore/libosmocore_git.bb
+++ b/recipes-osmocom/libosmocore/libosmocore_git.bb
@@ -1,9 +1,9 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "2862f9082ed4f16383a134e1112610d76ad91bf0"
+SRCREV = "5681a3254c1727d4926ac6d36800fd63d87ad7a2"
 SRC_URI = "git://git.osmocom.org/libosmocore.git;protocol=git;nobranch=1"
-PV = "1.3.0+gitr${SRCPV}"
+PV = "1.4.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"

 PACKAGES =+ "libosmoctrl libosmocodec libosmogb libosmogsm libosmovty 
osmo-arfcn osmo-auc-gen osmo-config-merge"
diff --git a/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb 
b/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
index fdb389b..dce2ef1 100644
--- a/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
+++ b/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "a1362ce8c20b2fa60e3da67936ea69d406e23aff"
-SRC_URI = "git://git.osmocom.org/osmo-bsc.git;protocol=git"
-PV = "1.6.0+gitr${SRCPV}"
+SRCREV = "1.6.1"
+SRC_URI = 
"git://git.osmocom.org/osmo-bsc.git;protocol=git;tag=${SRCREV};nobranch=1"
+PV = "1.6.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb 
b/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
index b1b2298..a854384 100644
--- a/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
+++ b/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
@@ -1,9 +1,9 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "e71e0f2af84d8b26f4c09a8c859f86cf701b3357"
+SRCREV = "2154607fb06c9e4d1a286f40c756cdb4f792efdf"
 SRC_URI = 

Change in meta-telephony[201705]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22024 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..

osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
---
M recipes-osmocom/osmo-iuh/osmo-iuh.inc
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved; Verified



diff --git a/recipes-osmocom/osmo-iuh/osmo-iuh.inc 
b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
index ab412ad..92b4633 100644
--- a/recipes-osmocom/osmo-iuh/osmo-iuh.inc
+++ b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
@@ -3,9 +3,13 @@
 LICENSE = "AGPLv3"
 LIC_FILES_CHKSUM = "file://COPYING;md5=73f1eb20517c55bf9493b7dd6e480788"

-DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c"
+DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c lksctp-tools"

-INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+INC_PR="r2.${META_TELEPHONY_OSMO_INC}"
+
+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "ac_cv_header_netinet_sctp_h=yes"

 inherit autotools pkgconfig systemd


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22024
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22024
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in meta-telephony[201705]: libosmo-netif: Add explicit dependency lksctp-tools

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22022 )

Change subject: libosmo-netif: Add explicit dependency lksctp-tools
..

libosmo-netif: Add explicit dependency lksctp-tools

Since libosmo-netif.git d1b1ff77a6bc2f70366c5d2f5079c70f5eae52a0, use of
libsctp is enabled by default and explicitly required.

Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
---
M recipes-osmocom/libosmo-netif/libosmo-netif.inc
1 file changed, 9 insertions(+), 4 deletions(-)

Approvals:
  laforge: Looks good to me, approved; Verified



diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif.inc 
b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
index a353be6..0a51dfc 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif.inc
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
@@ -1,10 +1,15 @@
 DESCRIPTION = "Osmocom network interface library"
-DEPENDS = "libosmo-abis libtalloc lksctp-tools"
 HOMEPAGE = "http://git.osmocom.org;
 LICENSE = "GPLv2+"
-
-INC_PR="r0.${META_TELEPHONY_OSMO_INC}"
-
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"

+INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+
+DEPENDS = "libosmo-abis libtalloc"
+
+PACKAGECONFIG ??= "\
+libsctp \
+"
+PACKAGECONFIG[libsctp] = "--enable-libsctp,--disable-libsctp,lksctp-tools,"
+
 inherit autotools pkgconfig

--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22022
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
Gerrit-Change-Number: 22022
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in meta-telephony[201705]: libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22023 )

Change subject: libosmo-netif: workaround oe-core forcing fail of sctp.h 
autoconf check
..

libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
---
M recipes-osmocom/libosmo-netif/libosmo-netif.inc
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  laforge: Looks good to me, approved; Verified



diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif.inc 
b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
index 0a51dfc..7098c42 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif.inc
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
@@ -12,4 +12,8 @@
 "
 PACKAGECONFIG[libsctp] = "--enable-libsctp,--disable-libsctp,lksctp-tools,"

+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "${@bb.utils.contains('PACKAGECONFIG', 'libsctp', 
'ac_cv_header_netinet_sctp_h=yes', '',d)}"
+
 inherit autotools pkgconfig

--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22023
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
Gerrit-Change-Number: 22023
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in meta-telephony[201705]: update all of osmocom to "latest" tags

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22025 )

Change subject: update all of osmocom to "latest" tags
..


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


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22025
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: I743173f8a3385686ffb11550aabc0fbc665cf043
Gerrit-Change-Number: 22025
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:46:04 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[201705]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22024 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..


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


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22024
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22024
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:45:59 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[201705]: libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22023 )

Change subject: libosmo-netif: workaround oe-core forcing fail of sctp.h 
autoconf check
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22023
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
Gerrit-Change-Number: 22023
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:45:53 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[201705]: libosmo-netif: Add explicit dependency lksctp-tools

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22022 )

Change subject: libosmo-netif: Add explicit dependency lksctp-tools
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22022
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
Gerrit-Change-Number: 22022
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:45:50 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/21949 )

Change subject: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC
..


Patch Set 3: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
Gerrit-Change-Number: 21949
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:42:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: bts_features: s/Repeation/Repetition/ in osmo_bts_features_descs[]

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22021 )

Change subject: bts_features: s/Repeation/Repetition/ in 
osmo_bts_features_descs[]
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia8c77766842ce80eae969774959cbf46e86f97b9
Gerrit-Change-Number: 22021
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:42:31 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Build failure of network:osmocom:nightly/open5gs in Debian_10/i586

2021-01-07 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/open5gs/Debian_10/i586

Package network:osmocom:nightly/open5gs failed to build in Debian_10/i586

Check out the package for editing:
  osc checkout network:osmocom:nightly open5gs

Last lines of build log:
[  346s] [2541/2541] cc  -o tests/handover/handover 
'tests/handover/63da1a3@@handover@exe/abts-main.c.o' 
'tests/handover/63da1a3@@handover@exe/epc-x2-test.c.o' 
'tests/handover/63da1a3@@handover@exe/epc-s1-test.c.o' 
'tests/handover/63da1a3@@handover@exe/5gc-xn-test.c.o' -Wl,--no-undefined 
-Wl,--as-needed -g -O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. 
-fstack-protector-strong -Wformat -Werror=format-security -Wl,-z,relro 
-Wl,-z,now -Wl,--start-group tests/app/libtestapp.a tests/common/libtestcomon.a 
lib/core/libogscore.so.2.1.1 lib/app/libogsapp.so.2.1.1 
lib/dbi/libogsdbi.so.2.1.1 lib/crypt/libogscrypt.so.2.1.1 
lib/sctp/libogssctp.so.2.1.1 lib/gtp/libogsgtp.so.2.1.1 
lib/s1ap/libogss1ap.so.2.1.1 lib/asn1c/s1ap/libogsasn1c-s1ap.so.2.1.1 
lib/asn1c/common/libogsasn1c-common.so.2.1.1 
lib/asn1c/util/libogsasn1c-util.so.2.1.1 lib/ngap/libogsngap.so.2.1.1 
lib/asn1c/ngap/libogsasn1c-ngap.so.2.1.1 lib/nas/eps/libogsnas-eps.so.2.1.1 
lib/nas/common/libogsnas-common.so.2.1.1 lib/nas/5gs/libogsnas-5gs.so.2.1.1 
lib/diameter/rx/libogsdiameter-rx.so.2.1.1 
lib/diameter/common/libogsdiameter-common.so.2.1.1 
subprojects/freeDiameter/libfdcore/libfdcore.so.1.3.2 
subprojects/freeDiameter/libfdproto/libfdproto.so.1.3.2 
/usr/lib/i386-linux-gnu/libyaml.so 
/usr/lib/gcc/i686-linux-gnu/8/../../../i386-linux-gnu/libmongoc-1.0.so 
/usr/lib/gcc/i686-linux-gnu/8/../../../i386-linux-gnu/libbson-1.0.so -lsctp 
-lidn -ldl -lsctp -lgcrypt -lgnutls -lsctp -lidn -ldl -lsctp -lgcrypt -lgnutls 
-lsctp -lidn -ldl -lsctp -lgcrypt -lgnutls -Wl,--end-group -pthread 
'-Wl,-rpath,$ORIGIN/../app:$ORIGIN/../common:$ORIGIN/../../lib/core:$ORIGIN/../../lib/app:$ORIGIN/../../lib/dbi:$ORIGIN/../../lib/crypt:$ORIGIN/../../lib/sctp:$ORIGIN/../../lib/gtp:$ORIGIN/../../lib/s1ap:$ORIGIN/../../lib/asn1c/s1ap:$ORIGIN/../../lib/asn1c/common:$ORIGIN/../../lib/asn1c/util:$ORIGIN/../../lib/ngap:$ORIGIN/../../lib/asn1c/ngap:$ORIGIN/../../lib/nas/eps:$ORIGIN/../../lib/nas/common:$ORIGIN/../../lib/nas/5gs:$ORIGIN/../../lib/diameter/rx:$ORIGIN/../../lib/diameter/common:$ORIGIN/../../subprojects/freeDiameter/libfdcore:$ORIGIN/../../subprojects/freeDiameter/libfdproto'
 
-Wl,-rpath-link,/usr/src/packages/BUILD/obj-i686-linux-gnu/tests/app:/usr/src/packages/BUILD/obj-i686-linux-gnu/tests/common:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/core:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/app:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/dbi:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/crypt:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/sctp:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/gtp:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/s1ap:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/asn1c/s1ap:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/asn1c/common:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/asn1c/util:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/ngap:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/asn1c/ngap:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/nas/eps:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/nas/common:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/nas/5gs:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/diameter/rx:/usr/src/packages/BUILD/obj-i686-linux-gnu/lib/diameter/common:/usr/src/packages/BUILD/obj-i686-linux-gnu/subprojects/freeDiameter/libfdcore:/usr/src/packages/BUILD/obj-i686-linux-gnu/subprojects/freeDiameter/libfdproto
  
[  346s]debian/rules override_dh_auto_test
[  346s] make[1]: Entering directory '/usr/src/packages/BUILD'
[  346s] cd obj-i686-linux-gnu && meson test --suite unit
[  348s] ninja: Entering directory `/usr/src/packages/BUILD/obj-i686-linux-gnu'
[  348s] ninja: no work to do.
[  355s] 1/3 open5gs:unit / core OK   7.17 s 
[  355s] 2/3 open5gs:unit / cryptOK   0.15 s 
[  355s] 3/3 open5gs:unit / unit FAIL 0.07 s (exit 
status 1)
[  355s] 
[  355s] Ok:2
[  355s] Expected Fail: 0
[  355s] Fail:  1
[  355s] Unexpected Pass:   0
[  355s] Skipped:   0
[  355s] Timeout:   0
[  355s] 
[  355s] Full log written to 
/usr/src/packages/BUILD/obj-i686-linux-gnu/meson-logs/testlog.txt
[  355s] make[1]: *** [debian/rules:17: override_dh_auto_test] Error 1
[  355s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  355s] make: *** [debian/rules:11: binary] Error 2
[  355s] dpkg-buildpackage: error: debian/rules binary subprocess returned exit 
status 2
[  355s] ### VM INTERACTION START ###
[  358s] [  341.883899] sysrq: Power Off
[  358s] [  341.902447] reboot: Power down
[  358s] ### VM INTERACTION END ###
[  358s] 
[  358s] cloud111 failed "build 

Change in meta-telephony[201705]: libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22023 )

Change subject: libosmo-netif: workaround oe-core forcing fail of sctp.h 
autoconf check
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22023
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
Gerrit-Change-Number: 22023
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:40:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[201705]: libosmo-netif: Add explicit dependency lksctp-tools

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22022 )

Change subject: libosmo-netif: Add explicit dependency lksctp-tools
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22022
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
Gerrit-Change-Number: 22022
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:40:09 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in meta-telephony[201705]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22024 )

Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..


Patch Set 1:

Basically every user of netinet/sctp.h will need that workaround, that's why it 
would be better to actually fix the poky we use, or if we can somehow add 
CACHED_CONFIGUREVARS to all our recipes via some config?


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22024
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22024
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:39:02 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in meta-telephony[201705]: update all of osmocom to "latest" tags

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22025 )


Change subject: update all of osmocom to "latest" tags
..

update all of osmocom to "latest" tags

We have to use some arcane construct to make the bitbake fetcher
actualyl fetch tags that are not on any branch (like those patch
releases that themselves branch off master, but which aren't on
master).  Otherwise we get errors like:

WARNING: osmo-sgsn-1.6.2+gitrAUTOINC+065500a347-r1.18.0 do_fetch: Failed to 
fetch URL git://git.osmocom.org/osmo-sgsn.git;protocol=git, attempting MIRRORS 
if available
ERROR: osmo-sgsn-1.6.2+gitrAUTOINC+065500a347-r1.18.0 do_fetch: Fetcher 
failure: Unable to find revision 065500a3471dd43b65dc87b95854c3015c551f0f in 
branch master even from upstream

Change-Id: I743173f8a3385686ffb11550aabc0fbc665cf043
---
M recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
M recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
M recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
M recipes-osmocom/libosmocore/libosmocore_git.bb
M recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
M recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
M recipes-osmocom/osmo-hlr/osmo-hlr_git.bb
M recipes-osmocom/osmo-iuh/osmo-iuh_git.bb
M recipes-osmocom/osmo-msc/osmo-msc_git.bb
M recipes-osmocom/osmo-sgsn/osmo-sgsn_git.bb
M recipes-osmocom/osmo-sip-connector/osmo-sip-connector_git.bb
M recipes-osmocom/osmo-sysmon/osmo-sysmon_git.bb
12 files changed, 30 insertions(+), 30 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/25/22025/1

diff --git a/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb 
b/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
index 0582599..446a590 100644
--- a/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
+++ b/recipes-osmocom/libosmo-abis/libosmo-abis_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "c313d44272f04b24dad38f4caea17180f9cbc509"
-SRC_URI = "git://git.osmocom.org/libosmo-abis.git;protocol=git"
-PV = "0.8.0+gitr${SRCPV}"
+SRCREV = "1.0.1"
+SRC_URI = 
"git://git.osmocom.org/libosmo-abis.git;protocol=git;tag=${SRCREV};nobranch=1"
+PV = "1.0.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb 
b/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
index c52c7e9..0f2856e 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "b8121e79ef53b30df535fb38987b3a48853ebbe9"
+SRCREV = "f3ccbecb6f802ccd98d154dc7ba7d552e298b2ea"
 SRC_URI = "git://git.osmocom.org/libosmo-netif.git;protocol=git"
-PV = "0.7.0+gitr${SRCPV}"
+PV = "1.0.0+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb 
b/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
index 31fa234..c78477c 100644
--- a/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
+++ b/recipes-osmocom/libosmo-sccp/libosmo-sccp_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "5e4643f4bc76a1437621348550b5b2a603e2179b"
+SRCREV = "37880b696ee90dead7e3b1292b109cc041a44e52"
 SRC_URI = "git://git.osmocom.org/libosmo-sccp.git;protocol=git"
-PV = "1.2.0+gitr${SRCPV}"
+PV = "1.3.0+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/libosmocore/libosmocore_git.bb 
b/recipes-osmocom/libosmocore/libosmocore_git.bb
index d5eab3a..1acf6ac 100644
--- a/recipes-osmocom/libosmocore/libosmocore_git.bb
+++ b/recipes-osmocom/libosmocore/libosmocore_git.bb
@@ -1,9 +1,9 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "2862f9082ed4f16383a134e1112610d76ad91bf0"
+SRCREV = "5681a3254c1727d4926ac6d36800fd63d87ad7a2"
 SRC_URI = "git://git.osmocom.org/libosmocore.git;protocol=git;nobranch=1"
-PV = "1.3.0+gitr${SRCPV}"
+PV = "1.4.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"

 PACKAGES =+ "libosmoctrl libosmocodec libosmogb libosmogsm libosmovty 
osmo-arfcn osmo-auc-gen osmo-config-merge"
diff --git a/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb 
b/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
index fdb389b..dce2ef1 100644
--- a/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
+++ b/recipes-osmocom/osmo-bsc/osmo-bsc_git.bb
@@ -1,7 +1,7 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "a1362ce8c20b2fa60e3da67936ea69d406e23aff"
-SRC_URI = "git://git.osmocom.org/osmo-bsc.git;protocol=git"
-PV = "1.6.0+gitr${SRCPV}"
+SRCREV = "1.6.1"
+SRC_URI = 
"git://git.osmocom.org/osmo-bsc.git;protocol=git;tag=${SRCREV};nobranch=1"
+PV = "1.6.1+gitr${SRCPV}"
 PR = "${INC_PR}.0"
diff --git a/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb 
b/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
index b1b2298..a854384 100644
--- a/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
+++ b/recipes-osmocom/osmo-ggsn/osmo-ggsn_git.bb
@@ -1,9 +1,9 @@
 require ${PN}.inc

 S = "${WORKDIR}/git"
-SRCREV = "e71e0f2af84d8b26f4c09a8c859f86cf701b3357"
+SRCREV = 

Change in meta-telephony[201705]: libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
Hello pespin,

I'd like you to do a code review. Please visit

https://gerrit.osmocom.org/c/meta-telephony/+/22023

to review the following change.


Change subject: libosmo-netif: workaround oe-core forcing fail of sctp.h 
autoconf check
..

libosmo-netif: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
---
M recipes-osmocom/libosmo-netif/libosmo-netif.inc
1 file changed, 4 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/23/22023/1

diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif.inc 
b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
index 0a51dfc..7098c42 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif.inc
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
@@ -12,4 +12,8 @@
 "
 PACKAGECONFIG[libsctp] = "--enable-libsctp,--disable-libsctp,lksctp-tools,"

+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "${@bb.utils.contains('PACKAGECONFIG', 'libsctp', 
'ac_cv_header_netinet_sctp_h=yes', '',d)}"
+
 inherit autotools pkgconfig

--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22023
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ide91e96648881a0c6d415d727a14c2ef913fb180
Gerrit-Change-Number: 22023
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newchange


Change in meta-telephony[201705]: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/meta-telephony/+/22024 )


Change subject: osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf 
check
..

osmo-iuh: workaround oe-core forcing fail of sctp.h autoconf check

oe-core is currently (pyro and current master) forcing non-availability
of netinet/sctp.h header file through autoconf's export CONFIG_SITE
pointing to files in meta/site/, which contain:
ac_cv_header_netinet_sctp_h=${ac_cv_header_netinet_sctp_h=no}

As a result, netinet/sctp.h is not found despite DEPENDS="lksctp-tools"
and the file being available correctly under recipe-sysroot/ and finally
configure fails with AC_MSG_ERROR().

It was confirmed by OE devs that those lines should not be there, and a
patch has been submitted:
https://patchwork.openembedded.org/patch/168892/

Since it's not available in pyro nor latest release, let's workaround it
in our recipe to make sure it works with older versions of OE.

Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
---
M recipes-osmocom/osmo-iuh/osmo-iuh.inc
1 file changed, 6 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/24/22024/1

diff --git a/recipes-osmocom/osmo-iuh/osmo-iuh.inc 
b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
index ab412ad..92b4633 100644
--- a/recipes-osmocom/osmo-iuh/osmo-iuh.inc
+++ b/recipes-osmocom/osmo-iuh/osmo-iuh.inc
@@ -3,9 +3,13 @@
 LICENSE = "AGPLv3"
 LIC_FILES_CHKSUM = "file://COPYING;md5=73f1eb20517c55bf9493b7dd6e480788"

-DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c"
+DEPENDS = "libosmocore libosmo-netif libosmo-sccp libasn1c lksctp-tools"

-INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+INC_PR="r2.${META_TELEPHONY_OSMO_INC}"
+
+# Workaround to oe-core meta/site/* CONFIG_SITE files passed to autoconf 
forcing unavailability of netinet/sctp.h.
+# Patch fixing issue upstream: https://patchwork.openembedded.org/patch/168892/
+CACHED_CONFIGUREVARS += "ac_cv_header_netinet_sctp_h=yes"

 inherit autotools pkgconfig systemd


--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22024
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ic54c50300bcf2cd278db95550d14eef044caea05
Gerrit-Change-Number: 22024
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in meta-telephony[201705]: libosmo-netif: Add explicit dependency lksctp-tools

2021-01-07 Thread laforge
Hello pespin,

I'd like you to do a code review. Please visit

https://gerrit.osmocom.org/c/meta-telephony/+/22022

to review the following change.


Change subject: libosmo-netif: Add explicit dependency lksctp-tools
..

libosmo-netif: Add explicit dependency lksctp-tools

Since libosmo-netif.git d1b1ff77a6bc2f70366c5d2f5079c70f5eae52a0, use of
libsctp is enabled by default and explicitly required.

Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
---
M recipes-osmocom/libosmo-netif/libosmo-netif.inc
1 file changed, 9 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/22/22022/1

diff --git a/recipes-osmocom/libosmo-netif/libosmo-netif.inc 
b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
index a353be6..0a51dfc 100644
--- a/recipes-osmocom/libosmo-netif/libosmo-netif.inc
+++ b/recipes-osmocom/libosmo-netif/libosmo-netif.inc
@@ -1,10 +1,15 @@
 DESCRIPTION = "Osmocom network interface library"
-DEPENDS = "libosmo-abis libtalloc lksctp-tools"
 HOMEPAGE = "http://git.osmocom.org;
 LICENSE = "GPLv2+"
-
-INC_PR="r0.${META_TELEPHONY_OSMO_INC}"
-
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"

+INC_PR="r1.${META_TELEPHONY_OSMO_INC}"
+
+DEPENDS = "libosmo-abis libtalloc"
+
+PACKAGECONFIG ??= "\
+libsctp \
+"
+PACKAGECONFIG[libsctp] = "--enable-libsctp,--disable-libsctp,lksctp-tools,"
+
 inherit autotools pkgconfig

--
To view, visit https://gerrit.osmocom.org/c/meta-telephony/+/22022
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Gerrit-Change-Id: Ib53557f21cad86300738f43b247df1e17166c46f
Gerrit-Change-Number: 22022
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newchange


Change in osmo-ttcn3-hacks[master]: CTRL: Introduce support to run osmocom CTRL server

2021-01-07 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012 )

Change subject: CTRL: Introduce support to run osmocom CTRL server
..


Patch Set 1: Code-Review+1

(1 comment)

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012/1/library/Osmocom_CTRL_Adapter.ttcn
File library/Osmocom_CTRL_Adapter.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012/1/library/Osmocom_CTRL_Adapter.ttcn@62
PS1, Line 62:   v
> it might make sense to differentiate based on server/client in the ID. […]
Ack.



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I37db9962f51baf2c63bd58ec47ec89f773d7a255
Gerrit-Change-Number: 22012
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:30:20 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: laforge 
Gerrit-MessageType: comment


Change in osmo-bts[master]: vty: add a command to show GPRS related info

2021-01-07 Thread fixeria
fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/20484 )

Change subject: vty: add a command to show GPRS related info
..

vty: add a command to show GPRS related info

Here is a sample output:

  OsmoBTS# show bts 0 gprs
  BTS 0, RAC 0, NSEI 101, BVCI 2
Cell NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
NSE NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
NSVC0 (NSVCI 101) NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
  Address: r=127.0.0.1:23010<->l=0.0.0.0:55385
NSVC1 (NSVCI 0) NM state: Oper 'Disabled', Admin 'Locked', Avail 'Off line'

This command is useful for debugging NS connection problems.

Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
---
M src/common/vty.c
1 file changed, 70 insertions(+), 0 deletions(-)

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



diff --git a/src/common/vty.c b/src/common/vty.c
index 326e332..1c3b496 100644
--- a/src/common/vty.c
+++ b/src/common/vty.c
@@ -39,6 +39,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 


@@ -1104,6 +1105,73 @@
return CMD_SUCCESS;
 }

+static void gprs_dump_vty(struct vty *vty, const struct gsm_bts *bts)
+{
+   unsigned int i;
+
+   /* GPRS parameters received from the BSC */
+   vty_out(vty, "BTS %u, RAC %u, NSEI %u, BVCI %u%s",
+   bts->nr, bts->gprs.rac,
+   bts->gprs.nse.nsei,
+   bts->gprs.cell.bvci,
+   VTY_NEWLINE);
+
+   vty_out(vty, "  Cell NM state: ");
+   net_dump_nmstate(vty, >gprs.cell.mo.nm_state);
+   vty_out(vty, "  NSE NM state: ");
+   net_dump_nmstate(vty, >gprs.nse.mo.nm_state);
+
+   for (i = 0; i < ARRAY_SIZE(bts->gprs.nsvc); i++) {
+   const struct gsm_bts_gprs_nsvc *nsvc = >gprs.nsvc[i];
+
+   vty_out(vty, "  NSVC%u (NSVCI %u) NM state: ", i, nsvc->nsvci);
+   net_dump_nmstate(vty, >mo.nm_state);
+
+   if (nsvc->mo.nm_state.operational == NM_OPSTATE_ENABLED) {
+   struct osmo_sockaddr_str remote;
+   struct osmo_sockaddr_str local;
+
+   if (osmo_sockaddr_str_from_sockaddr(, 
>remote.u.sas) != 0)
+   remote = (struct osmo_sockaddr_str) { .ip = 
"" };
+   if (osmo_sockaddr_str_from_sockaddr(, 
>local.u.sas) != 0)
+   local = (struct osmo_sockaddr_str) { .ip = 
"" };
+
+   /* Work around for over-defensiveness of 
OSMO_SOCKADDR_STR_FMT_ARGS():
+*  error: the address of ‘remote’ will always evaluate 
as ‘true’
+*  error: the address of ‘local’ will always evaluate 
as ‘true’ */
+   const struct osmo_sockaddr_str *r = 
+   const struct osmo_sockaddr_str *l = 
+
+   /* Getting remote/local address info has never been so 
easy, right? */
+   vty_out(vty, "Address: r=" OSMO_SOCKADDR_STR_FMT
+  "<->l=" OSMO_SOCKADDR_STR_FMT 
"%s",
+   OSMO_SOCKADDR_STR_FMT_ARGS(r),
+   OSMO_SOCKADDR_STR_FMT_ARGS(l),
+   VTY_NEWLINE);
+   }
+   }
+}
+
+DEFUN(show_bts_gprs, show_bts_gprs_cmd,
+  "show bts <0-255> gprs",
+  SHOW_STR "Display information about a BTS\n"
+  BTS_NR_STR "GPRS/EGPRS configuration\n")
+{
+   const struct gsm_network *net = gsmnet_from_vty(vty);
+   const struct gsm_bts *bts;
+
+   bts = gsm_bts_num(net, atoi(argv[0]));
+   if (bts == NULL) {
+   vty_out(vty, "%% can't find BTS '%s'%s",
+   argv[0], VTY_NEWLINE);
+   return CMD_WARNING;
+   }
+
+   /* TODO: also print info about PCUIF connection */
+   gprs_dump_vty(vty, bts);
+   return CMD_SUCCESS;
+}
+
 DEFUN(test_send_failure_event_report, test_send_failure_event_report_cmd, 
"test send-failure-event-report <0-255>",
   "Various testing commands\n"
   "Send a test OML failure event report to the BSC\n" BTS_NR_STR)
@@ -2032,6 +2100,8 @@
install_element_ve(_ts_cmd);
install_element_ve(_lchan_cmd);
install_element_ve(_lchan_summary_cmd);
+   install_element_ve(_bts_gprs_cmd);
+
install_element_ve(_fltr_l1_sapi_cmd);
install_element_ve(_logging_fltr_l1_sapi_cmd);


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/20484
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder

Change in libosmocore[master]: bts_features: s/Repeation/Repetition/ in osmo_bts_features_descs[]

2021-01-07 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/22021 )


Change subject: bts_features: s/Repeation/Repetition/ in 
osmo_bts_features_descs[]
..

bts_features: s/Repeation/Repetition/ in osmo_bts_features_descs[]

Change-Id: Ia8c77766842ce80eae969774959cbf46e86f97b9
---
M src/gsm/bts_features.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/21/22021/1

diff --git a/src/gsm/bts_features.c b/src/gsm/bts_features.c
index 93703aa..d8c27d8 100644
--- a/src/gsm/bts_features.c
+++ b/src/gsm/bts_features.c
@@ -41,7 +41,7 @@
{ BTS_FEAT_ETWS_PN, "ETWS Primary Notification via PCH" },
{ BTS_FEAT_PAGING_COORDINATION, "BSS Paging Coordination" },
{ BTS_FEAT_IPV6_NSVC,   "NSVC IPv6" },
-   { BTS_FEAT_ACCH_REP,"FACCH/SACCH Repeation" },
+   { BTS_FEAT_ACCH_REP,"FACCH/SACCH Repetition" },
{ 0, NULL }
 };


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia8c77766842ce80eae969774959cbf46e86f97b9
Gerrit-Change-Number: 22021
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in libosmocore[master]: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

2021-01-07 Thread fixeria
Hello Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/libosmocore/+/21949

to look at the new patch set (#3).

Change subject: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC
..

gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

Both LAC and TAC take 2 octets and follow MCC/MNC fields on the wire.
We abuse gsm48_encode_ra() for encoding of MCC/MNC, but it can also
be abused to encode TAC in bssgp_create_rim_ri().  There is no need
to encode ''O and then override it with osmo_store16be().

Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
---
M src/gb/gprs_bssgp.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/49/21949/3
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/21949
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
Gerrit-Change-Number: 21949
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-CC: pespin 
Gerrit-MessageType: newpatchset


Change in osmo-bts[master]: vty: add a command to show GPRS related info

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/20484 )

Change subject: vty: add a command to show GPRS related info
..


Patch Set 4: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/20484
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:22:22 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/21949 )

Change subject: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC
..


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/c/libosmocore/+/21949/2/src/gb/gprs_bssgp.c
File src/gb/gprs_bssgp.c:

https://gerrit.osmocom.org/c/libosmocore/+/21949/2/src/gb/gprs_bssgp.c@405
PS2, Line 405:  .lac = ri->eutran.tai.tac, /* intentional */
so probably worth it adding "intentional, TAI uses same format as RAI"



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
Gerrit-Change-Number: 21949
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:21:17 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field

2021-01-07 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011 )

Change subject: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If1a273a2be71040eaea2189a0aeaf737adf848e5
Gerrit-Change-Number: 22011
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:17:48 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

2021-01-07 Thread fixeria
Hello Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/libosmocore/+/21949

to look at the new patch set (#2).

Change subject: gprs_bssgp: abuse gsm48_encode_ra() to encode TAC
..

gprs_bssgp: abuse gsm48_encode_ra() to encode TAC

Both LAC and TAC take 2 octets and follow MCC/MNC fields on the wire.
We abuse gsm48_encode_ra() for encoding of MCC/MNC, but it can also
be abused to encode TAC in bssgp_create_rim_ri().  There is no need
to encode ''O and then override it with osmo_store16be().

Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
---
M src/gb/gprs_bssgp.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/49/21949/2
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/21949
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I986552aa52cf38b1c5290d2e5cd3ff2d1c36a4e5
Gerrit-Change-Number: 21949
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-bts[master]: vty: add a command to show GPRS related info

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/20484 )

Change subject: vty: add a command to show GPRS related info
..


Patch Set 4: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/20484
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:14:52 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-trx[master]: manual: Fix typo OsmTRX -> OsmoTRX

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/22020 )

Change subject: manual: Fix typo OsmTRX -> OsmoTRX
..

manual: Fix typo OsmTRX -> OsmoTRX

Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
---
M doc/manuals/chapters/configuration.adoc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved
  laforge: Looks good to me, approved; Verified



diff --git a/doc/manuals/chapters/configuration.adoc 
b/doc/manuals/chapters/configuration.adoc
index a194537..e1323ef 100644
--- a/doc/manuals/chapters/configuration.adoc
+++ b/doc/manuals/chapters/configuration.adoc
@@ -1,4 +1,4 @@
-== Configuring OsmTRX
+== Configuring OsmoTRX

 OsmoTRX will read the configuration at startup time and configure the
 transceiver accordingly after validating the configuration.

--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/22020
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
Gerrit-Change-Number: 22020
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-MessageType: merged


Change in osmo-trx[master]: manual: Fix typo OsmTRX -> OsmoTRX

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/22020 )

Change subject: manual: Fix typo OsmTRX -> OsmoTRX
..


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


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/22020
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
Gerrit-Change-Number: 22020
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:13:41 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22017 )

Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..

Bump version: 0.6.0.24-da79 → 1.0.0

Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
---
M debian/changelog
1 file changed, 44 insertions(+), 0 deletions(-)

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



diff --git a/debian/changelog b/debian/changelog
index f59c49e..aae3463 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,47 @@
+osmo-iuh (1.0.0) unstable; urgency=medium
+
+  [ Martin Hauke ]
+  * tests: Missing return statement
+
+  [ Pau Espin Pedrol ]
+  * tests/hnb-test-ranap.c: Fix wrong printf format
+  * tests/test-ranap.c: Fix wrong printf format
+  * Fix compiler warning on signedness of char ptr using OCTET_STRING_fromBuf()
+  * hnbgw_vty.c: Drop unusued variables
+  * tests: Fix trailing whitespace
+  * Fix OneDefinitionRule Asan violation in tests and osmo-hnbgw
+  * Change default SCTP conn NULL->127.0.0.1 to localhost->localhost
+  * hnbgw: Mark SCTP conn as NODELAY
+
+  [ Neels Hofmeyr ]
+  * manual: add "Configuring Primary Links" and "Multiple instances"
+  * manual: link to new common cs7-config.adoc
+
+  [ Vadim Yanitskiy ]
+  * tests/test-helpers: fix some -Wpointer-sign warnings reported by GCC
+  * tests/test-helpers: fix endianness: do not print uint32_t as raw bytes
+  * tests/test-ranap: cosmetic: remove unused variable 'rc'
+  * debian/control: change maintainer to the Osmocom team / mailing list
+
+  [ Eric ]
+  * configure.ac: fix libtool issue  with clang and sanitizer
+
+  [ Philipp Maier ]
+  * hnbgw: do not use LOGL_DEBUG as default loglevel
+
+  [ Harald Welte ]
+  * Fix compilation with gcc-10
+  * Provide more context when logging "Error in ANY_fromType_aper"
+  * debian: Actually build manuals; add osmo-hnbgw-doc package
+  * configure.ac: Ensure netinet/sctp.h is present
+
+  [ Oliver Smith ]
+  * contrib: import RPM spec
+  * contrib: integrate RPM spec
+  * Makefile.am: EXTRA_DIST: debian, contrib/*.spec.in
+
+ -- Harald Welte   Thu, 07 Jan 2021 13:55:19 +0100
+
 osmo-iuh (0.6.0) unstable; urgency=medium
 
   [ Vadim Yanitskiy ]

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22017
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
Gerrit-Change-Number: 22017
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-trx[master]: manual: Fix typo OsmTRX -> OsmoTRX

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/22020 )

Change subject: manual: Fix typo OsmTRX -> OsmoTRX
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/22020
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
Gerrit-Change-Number: 22020
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:09:34 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-trx[master]: manual: Fix typo OsmTRX -> OsmoTRX

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/22020 )


Change subject: manual: Fix typo OsmTRX -> OsmoTRX
..

manual: Fix typo OsmTRX -> OsmoTRX

Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
---
M doc/manuals/chapters/configuration.adoc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/20/22020/1

diff --git a/doc/manuals/chapters/configuration.adoc 
b/doc/manuals/chapters/configuration.adoc
index a194537..e1323ef 100644
--- a/doc/manuals/chapters/configuration.adoc
+++ b/doc/manuals/chapters/configuration.adoc
@@ -1,4 +1,4 @@
-== Configuring OsmTRX
+== Configuring OsmoTRX

 OsmoTRX will read the configuration at startup time and configure the
 transceiver accordingly after validating the configuration.

--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/22020
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I4b3a76e41d4abbb08046a241ae9b7c079ce990ae
Gerrit-Change-Number: 22020
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in osmo-iuh[master]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22019 )

Change subject: configure.ac: Ensure netinet/sctp.h is present
..

configure.ac: Ensure netinet/sctp.h is present

we unconditionally depend on this header file, yet nothing
in configure.ac ensures its presence.  Building on a clean
Debian 10 with all existing dependencies installed passes
configure but fails at compilation time.

Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
---
M configure.ac
1 file changed, 2 insertions(+), 0 deletions(-)

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



diff --git a/configure.ac b/configure.ac
index 692db57..c186855 100644
--- a/configure.ac
+++ b/configure.ac
@@ -31,6 +31,8 @@
   [AS_CASE(["${host_os}"],
  [*linux*],[archive_cmds='$CC -shared $pic_flag $libobjs $deplibs 
$compiler_flags $wl-soname $wl$soname -o $lib'])])
 
+AC_CHECK_HEADERS(netinet/sctp.h,,AC_MSG_ERROR(netinet/sctp.h not found))
+
 dnl check for pkg-config (explained in detail in libosmocore/configure.ac)
 AC_PATH_PROG(PKG_CONFIG_INSTALLED, pkg-config, no)
 if test "x$PKG_CONFIG_INSTALLED" = "xno"; then

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22019
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22019
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-iuh[master]: debian: Actually build manuals; add osmo-hnbgw-doc package

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22018 )

Change subject: debian: Actually build manuals; add osmo-hnbgw-doc package
..

debian: Actually build manuals; add osmo-hnbgw-doc package

for some strange reason we didn't yet build the manual in the debian
package build.

Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
---
M TODO-RELEASE
M debian/control
M debian/rules
3 files changed, 16 insertions(+), 3 deletions(-)

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



diff --git a/TODO-RELEASE b/TODO-RELEASE
index dde4b72..d0852fc 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -7,4 +7,3 @@
 # If any interfaces have been added since the last public release: c:r:a + 1.
 # If any interfaces have been removed or changed since the last public 
release: c:r:0.
 #library   whatdescription / commit summary line
-manual needs common chapter cs7-config.adoc from 
osmo-gsm-manuals > 0.3.0
diff --git a/debian/control b/debian/control
index 416b9d7..d1805de 100644
--- a/debian/control
+++ b/debian/control
@@ -14,6 +14,7 @@
libosmocore-dev (>= 0.12.0),
libosmo-netif-dev (>= 0.3.0),
libosmo-sigtran-dev (>= 0.10.0),
+   osmo-gsm-manuals-dev (>= 1.0.0),
python3
 Standards-Version: 3.9.8
 Vcs-Git: git://git.osmocom.org/osmo-iuh.git
@@ -36,6 +37,14 @@
 Depends: osmo-hnbgw (= ${binary:Version}), ${misc:Depends}
 Description: osmocom Home Node B Gateway

+Package: osmo-hnbgw-doc
+Section: doc
+Architecture: all
+Depends: ${misc:Depends}
+Description: ${misc:Package} PDF documentation
+  Various manuals: user manual, VTY reference manual and/or
+  protocol/interface manuals.
+
 Package: libosmo-ranap3
 Section: libs
 Architecture: any
diff --git a/debian/rules b/debian/rules
index c3bdf50..e1f6124 100755
--- a/debian/rules
+++ b/debian/rules
@@ -8,6 +8,10 @@
 %:
dh $@ --with autoreconf

+CONFIGURE_FLAGS += --with-systemdsystemunitdir=/lib/systemd/system 
--enable-manuals
+override_dh_auto_configure:
+   dh_auto_configure -- $(CONFIGURE_FLAGS)
+
 # See 
https://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-dbg
 override_dh_strip:
dh_strip -plibosmo-ranap3 --dbg-package=libosmo-ranap-dbg
@@ -18,5 +22,6 @@
 override_dh_auto_test:
dh_auto_test || (find . -name testsuite.log -exec cat {} \; ; false)

-override_dh_auto_configure:
-   dh_auto_configure -- --with-systemdsystemunitdir=/lib/systemd/system
+# Don't create .pdf.gz files (barely saves space and they can't be opened 
directly by most pdf readers)
+override_dh_compress:
+   dh_compress -X.pdf

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22018
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
Gerrit-Change-Number: 22018
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-iuh[for-1.0]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22014 )

Change subject: configure.ac: Ensure netinet/sctp.h is present
..

configure.ac: Ensure netinet/sctp.h is present

we unconditionally depend on this header file, yet nothing
in configure.ac ensures its presence.  Building on a clean
Debian 10 with all existing dependencies installed passes
configure but fails at compilation time.

Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
---
M configure.ac
1 file changed, 2 insertions(+), 0 deletions(-)

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



diff --git a/configure.ac b/configure.ac
index 692db57..c186855 100644
--- a/configure.ac
+++ b/configure.ac
@@ -31,6 +31,8 @@
   [AS_CASE(["${host_os}"],
  [*linux*],[archive_cmds='$CC -shared $pic_flag $libobjs $deplibs 
$compiler_flags $wl-soname $wl$soname -o $lib'])])
 
+AC_CHECK_HEADERS(netinet/sctp.h,,AC_MSG_ERROR(netinet/sctp.h not found))
+
 dnl check for pkg-config (explained in detail in libosmocore/configure.ac)
 AC_PATH_PROG(PKG_CONFIG_INSTALLED, pkg-config, no)
 if test "x$PKG_CONFIG_INSTALLED" = "xno"; then

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22014
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22014
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-iuh[for-1.0]: debian: Actually build manuals; add osmo-hnbgw-doc package

2021-01-07 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22010 )

Change subject: debian: Actually build manuals; add osmo-hnbgw-doc package
..

debian: Actually build manuals; add osmo-hnbgw-doc package

for some strange reason we didn't yet build the manual in the debian
package build.

Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
---
M TODO-RELEASE
M debian/control
M debian/rules
3 files changed, 16 insertions(+), 3 deletions(-)

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



diff --git a/TODO-RELEASE b/TODO-RELEASE
index dde4b72..d0852fc 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -7,4 +7,3 @@
 # If any interfaces have been added since the last public release: c:r:a + 1.
 # If any interfaces have been removed or changed since the last public 
release: c:r:0.
 #library   whatdescription / commit summary line
-manual needs common chapter cs7-config.adoc from 
osmo-gsm-manuals > 0.3.0
diff --git a/debian/control b/debian/control
index 416b9d7..d1805de 100644
--- a/debian/control
+++ b/debian/control
@@ -14,6 +14,7 @@
libosmocore-dev (>= 0.12.0),
libosmo-netif-dev (>= 0.3.0),
libosmo-sigtran-dev (>= 0.10.0),
+   osmo-gsm-manuals-dev (>= 1.0.0),
python3
 Standards-Version: 3.9.8
 Vcs-Git: git://git.osmocom.org/osmo-iuh.git
@@ -36,6 +37,14 @@
 Depends: osmo-hnbgw (= ${binary:Version}), ${misc:Depends}
 Description: osmocom Home Node B Gateway

+Package: osmo-hnbgw-doc
+Section: doc
+Architecture: all
+Depends: ${misc:Depends}
+Description: ${misc:Package} PDF documentation
+  Various manuals: user manual, VTY reference manual and/or
+  protocol/interface manuals.
+
 Package: libosmo-ranap3
 Section: libs
 Architecture: any
diff --git a/debian/rules b/debian/rules
index c3bdf50..e1f6124 100755
--- a/debian/rules
+++ b/debian/rules
@@ -8,6 +8,10 @@
 %:
dh $@ --with autoreconf

+CONFIGURE_FLAGS += --with-systemdsystemunitdir=/lib/systemd/system 
--enable-manuals
+override_dh_auto_configure:
+   dh_auto_configure -- $(CONFIGURE_FLAGS)
+
 # See 
https://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-dbg
 override_dh_strip:
dh_strip -plibosmo-ranap3 --dbg-package=libosmo-ranap-dbg
@@ -18,5 +22,6 @@
 override_dh_auto_test:
dh_auto_test || (find . -name testsuite.log -exec cat {} \; ; false)

-override_dh_auto_configure:
-   dh_auto_configure -- --with-systemdsystemunitdir=/lib/systemd/system
+# Don't create .pdf.gz files (barely saves space and they can't be opened 
directly by most pdf readers)
+override_dh_compress:
+   dh_compress -X.pdf

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22010
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
Gerrit-Change-Number: 22010
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-iuh[master]: debian: Actually build manuals; add osmo-hnbgw-doc package

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22018 )

Change subject: debian: Actually build manuals; add osmo-hnbgw-doc package
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22018
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
Gerrit-Change-Number: 22018
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:03:52 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[master]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22019 )

Change subject: configure.ac: Ensure netinet/sctp.h is present
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22019
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22019
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:03:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22017 )

Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22017
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
Gerrit-Change-Number: 22017
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:03:28 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-bts[master]: vty: add a command to show GPRS related info

2021-01-07 Thread fixeria
Hello Jenkins Builder, pespin,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-bts/+/20484

to look at the new patch set (#4).

Change subject: vty: add a command to show GPRS related info
..

vty: add a command to show GPRS related info

Here is a sample output:

  OsmoBTS# show bts 0 gprs
  BTS 0, RAC 0, NSEI 101, BVCI 2
Cell NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
NSE NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
NSVC0 (NSVCI 101) NM state: Oper 'Enabled', Admin 'Unlocked', Avail 'OK'
  Address: r=127.0.0.1:23010<->l=0.0.0.0:55385
NSVC1 (NSVCI 0) NM state: Oper 'Disabled', Admin 'Locked', Avail 'Off line'

This command is useful for debugging NS connection problems.

Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
---
M src/common/vty.c
1 file changed, 70 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/84/20484/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/20484
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I149eea0b1c109020406eb67c9082c335a77aab06
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22017 )

Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22017
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
Gerrit-Change-Number: 22017
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:00:50 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[master]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22019 )

Change subject: configure.ac: Ensure netinet/sctp.h is present
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22019
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22019
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 13:00:20 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[master]: debian: Actually build manuals; add osmo-hnbgw-doc package

2021-01-07 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22018 )

Change subject: debian: Actually build manuals; add osmo-hnbgw-doc package
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22018
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
Gerrit-Change-Number: 22018
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 12:59:53 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: CTRL: Introduce support to run osmocom CTRL server

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012 )

Change subject: CTRL: Introduce support to run osmocom CTRL server
..


Patch Set 1: Code-Review+1

(1 comment)

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012/1/library/Osmocom_CTRL_Adapter.ttcn
File library/Osmocom_CTRL_Adapter.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012/1/library/Osmocom_CTRL_Adapter.ttcn@62
PS1, Line 62:   v
it might make sense to differentiate based on server/client in the ID.  
Othertwise you start both client and server "adapter" and their components have 
identical names.



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22012
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I37db9962f51baf2c63bd58ec47ec89f773d7a255
Gerrit-Change-Number: 22012
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 12:59:42 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011 )

Change subject: CTRL: Fix tr_CtrlMsgGet containing invalid value in val field
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22011
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If1a273a2be71040eaea2189a0aeaf737adf848e5
Gerrit-Change-Number: 22011
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 07 Jan 2021 12:58:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22017 )

Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..


Patch Set 1:

I've teste that this acutally builds against a current "latest" feed in a clean 
debian 10 container.


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22017
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
Gerrit-Change-Number: 22017
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 12:57:58 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-iuh[master]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22019 )


Change subject: configure.ac: Ensure netinet/sctp.h is present
..

configure.ac: Ensure netinet/sctp.h is present

we unconditionally depend on this header file, yet nothing
in configure.ac ensures its presence.  Building on a clean
Debian 10 with all existing dependencies installed passes
configure but fails at compilation time.

Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
---
M configure.ac
1 file changed, 2 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/19/22019/1

diff --git a/configure.ac b/configure.ac
index 692db57..c186855 100644
--- a/configure.ac
+++ b/configure.ac
@@ -31,6 +31,8 @@
   [AS_CASE(["${host_os}"],
  [*linux*],[archive_cmds='$CC -shared $pic_flag $libobjs $deplibs 
$compiler_flags $wl-soname $wl$soname -o $lib'])])

+AC_CHECK_HEADERS(netinet/sctp.h,,AC_MSG_ERROR(netinet/sctp.h not found))
+
 dnl check for pkg-config (explained in detail in libosmocore/configure.ac)
 AC_PATH_PROG(PKG_CONFIG_INSTALLED, pkg-config, no)
 if test "x$PKG_CONFIG_INSTALLED" = "xno"; then

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22019
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22019
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in osmo-iuh[master]: debian: Actually build manuals; add osmo-hnbgw-doc package

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22018 )


Change subject: debian: Actually build manuals; add osmo-hnbgw-doc package
..

debian: Actually build manuals; add osmo-hnbgw-doc package

for some strange reason we didn't yet build the manual in the debian
package build.

Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
---
M TODO-RELEASE
M debian/control
M debian/rules
3 files changed, 16 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/18/22018/1

diff --git a/TODO-RELEASE b/TODO-RELEASE
index dde4b72..d0852fc 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -7,4 +7,3 @@
 # If any interfaces have been added since the last public release: c:r:a + 1.
 # If any interfaces have been removed or changed since the last public 
release: c:r:0.
 #library   whatdescription / commit summary line
-manual needs common chapter cs7-config.adoc from 
osmo-gsm-manuals > 0.3.0
diff --git a/debian/control b/debian/control
index 416b9d7..d1805de 100644
--- a/debian/control
+++ b/debian/control
@@ -14,6 +14,7 @@
libosmocore-dev (>= 0.12.0),
libosmo-netif-dev (>= 0.3.0),
libosmo-sigtran-dev (>= 0.10.0),
+   osmo-gsm-manuals-dev (>= 1.0.0),
python3
 Standards-Version: 3.9.8
 Vcs-Git: git://git.osmocom.org/osmo-iuh.git
@@ -36,6 +37,14 @@
 Depends: osmo-hnbgw (= ${binary:Version}), ${misc:Depends}
 Description: osmocom Home Node B Gateway

+Package: osmo-hnbgw-doc
+Section: doc
+Architecture: all
+Depends: ${misc:Depends}
+Description: ${misc:Package} PDF documentation
+  Various manuals: user manual, VTY reference manual and/or
+  protocol/interface manuals.
+
 Package: libosmo-ranap3
 Section: libs
 Architecture: any
diff --git a/debian/rules b/debian/rules
index c3bdf50..e1f6124 100755
--- a/debian/rules
+++ b/debian/rules
@@ -8,6 +8,10 @@
 %:
dh $@ --with autoreconf

+CONFIGURE_FLAGS += --with-systemdsystemunitdir=/lib/systemd/system 
--enable-manuals
+override_dh_auto_configure:
+   dh_auto_configure -- $(CONFIGURE_FLAGS)
+
 # See 
https://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-dbg
 override_dh_strip:
dh_strip -plibosmo-ranap3 --dbg-package=libosmo-ranap-dbg
@@ -18,5 +22,6 @@
 override_dh_auto_test:
dh_auto_test || (find . -name testsuite.log -exec cat {} \; ; false)

-override_dh_auto_configure:
-   dh_auto_configure -- --with-systemdsystemunitdir=/lib/systemd/system
+# Don't create .pdf.gz files (barely saves space and they can't be opened 
directly by most pdf readers)
+override_dh_compress:
+   dh_compress -X.pdf

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22018
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: Id53956494144b46e78fc25a176a9b8917788d86e
Gerrit-Change-Number: 22018
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in osmo-iuh[for-1.0]: configure.ac: Ensure netinet/sctp.h is present

2021-01-07 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22014 )

Change subject: configure.ac: Ensure netinet/sctp.h is present
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22014
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I0a4407ef6ee512ad5ea8775d90acfe1c4fa67cd0
Gerrit-Change-Number: 22014
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 07 Jan 2021 12:55:46 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22017 )


Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..

Bump version: 0.6.0.24-da79 → 1.0.0

Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
---
M debian/changelog
1 file changed, 44 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/17/22017/1

diff --git a/debian/changelog b/debian/changelog
index f59c49e..aae3463 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,47 @@
+osmo-iuh (1.0.0) unstable; urgency=medium
+
+  [ Martin Hauke ]
+  * tests: Missing return statement
+
+  [ Pau Espin Pedrol ]
+  * tests/hnb-test-ranap.c: Fix wrong printf format
+  * tests/test-ranap.c: Fix wrong printf format
+  * Fix compiler warning on signedness of char ptr using OCTET_STRING_fromBuf()
+  * hnbgw_vty.c: Drop unusued variables
+  * tests: Fix trailing whitespace
+  * Fix OneDefinitionRule Asan violation in tests and osmo-hnbgw
+  * Change default SCTP conn NULL->127.0.0.1 to localhost->localhost
+  * hnbgw: Mark SCTP conn as NODELAY
+
+  [ Neels Hofmeyr ]
+  * manual: add "Configuring Primary Links" and "Multiple instances"
+  * manual: link to new common cs7-config.adoc
+
+  [ Vadim Yanitskiy ]
+  * tests/test-helpers: fix some -Wpointer-sign warnings reported by GCC
+  * tests/test-helpers: fix endianness: do not print uint32_t as raw bytes
+  * tests/test-ranap: cosmetic: remove unused variable 'rc'
+  * debian/control: change maintainer to the Osmocom team / mailing list
+
+  [ Eric ]
+  * configure.ac: fix libtool issue  with clang and sanitizer
+
+  [ Philipp Maier ]
+  * hnbgw: do not use LOGL_DEBUG as default loglevel
+
+  [ Harald Welte ]
+  * Fix compilation with gcc-10
+  * Provide more context when logging "Error in ANY_fromType_aper"
+  * debian: Actually build manuals; add osmo-hnbgw-doc package
+  * configure.ac: Ensure netinet/sctp.h is present
+
+  [ Oliver Smith ]
+  * contrib: import RPM spec
+  * contrib: integrate RPM spec
+  * Makefile.am: EXTRA_DIST: debian, contrib/*.spec.in
+
+ -- Harald Welte   Thu, 07 Jan 2021 13:55:19 +0100
+
 osmo-iuh (0.6.0) unstable; urgency=medium

   [ Vadim Yanitskiy ]

--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22017
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I4c26bf6c129d15833c4fc5f395465efe88826c44
Gerrit-Change-Number: 22017
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-MessageType: newchange


Change in osmo-iuh[for-1.0]: Bump version: 0.6.0.24-da79 → 1.0.0

2021-01-07 Thread laforge
laforge has abandoned this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/22016 )

Change subject: Bump version: 0.6.0.24-da79 → 1.0.0
..


Abandoned
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/22016
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: for-1.0
Gerrit-Change-Id: I17cb89a085689e15840f4c5812eb5ff75f961d33
Gerrit-Change-Number: 22016
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-CC: Jenkins Builder
Gerrit-MessageType: abandon


<    1   2   3   >