Attention is currently required from: neels.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817 )
Change subject: add library/PFCP_*, deps/PFCP
......................................................................
Patch Set 1:
(5 comments)
File library/Osmocom_VTY_Functions.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817/comment/3b9498ad_d441…
PS1, Line 247: type record of charstring StrList;
You may be probably reusing this for something no VTY related? then maybe move it to another module actually aimed at providing types?
File library/PFCP_CodecPort_CtrlFunct.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817/comment/98d16bbb_a6a2…
PS1, Line 7: inout PFCP_PT portRef,
did you copy this? indentation looks weird.
File library/PFCP_Templates.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817/comment/1368050c_8d56…
PS1, Line 18: type enumerated e_PFCP_Cause {
You could move some of these into PFCP_Types.ttcn
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817/comment/79d5a3d0_ec6f…
PS1, Line 418: function ts_PFCP_Session_Est_Req(charstring node_id, OCT8 cp_seid, Create_PDR_list create_pdr, Create_FAR_list create_far) return template PDU_PFCP {
Please improve the scope of template types.
ts_* -> "template (value)"
tr_* -> "template (present)"
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817/comment/5a13e485_f106…
PS1, Line 460: function tr_PFCP_Session_Est_Resp(template OCT8 hdr_seid := ?) return template PDU_PFCP {
template (present) OCT8 hdr_seid
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/28817
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: I0723b931b3f755ea291bffa2f27c34ba446c2f2f
Gerrit-Change-Number: 28817
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 27 Jul 2022 14:48:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: neels.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/28815 )
Change subject: ranap_rab_ass_req_encode(): return msgb
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
Patchset:
PS1:
It would even probably make more sense to allocate the msgb at the caller and pass it to the encode function, in case the caller wants to prepend something (like RUA) before it.
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/28815
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I85e715326e1d8f4f301f82f78da109f1a7a92f30
Gerrit-Change-Number: 28815
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 27 Jul 2022 14:07:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/28812 )
Change subject: pfcp_endpoint: fix final PFCP retrans resp_cb
......................................................................
pfcp_endpoint: fix final PFCP retrans resp_cb
After the final retransmission of a sent request, still keep the message
in the queue for its expiry period, so that a later response is matched
to the request.
The osmo_pfcp_msg.resp_cb() depends on the sent message to remain in the
queue until it times out. That was not the case in an earlier stage of
libosmo-pfcp development.
I noticed this during ttcn3 testing, where osmo-hnbgw continuously
resends PFCP Association Setup Requests, and fails to associate if ttcn3
happens to respond to the final retransmission of a request.
Related: SYS#5599
Change-Id: Iaca396891921f7057015ce6e1e4528b955757809
---
M src/libosmo-pfcp/pfcp_endpoint.c
1 file changed, 10 insertions(+), 9 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-pfcp refs/changes/12/28812/1
diff --git a/src/libosmo-pfcp/pfcp_endpoint.c b/src/libosmo-pfcp/pfcp_endpoint.c
index 3002591..454f102 100644
--- a/src/libosmo-pfcp/pfcp_endpoint.c
+++ b/src/libosmo-pfcp/pfcp_endpoint.c
@@ -181,18 +181,19 @@
struct osmo_pfcp_msg *m = qe->m;
int rc;
- /* re-transmit */
- if (qe->n1_remaining)
- qe->n1_remaining--;
- OSMO_LOG_PFCP_MSG(m, LOGL_INFO, "re-sending (%u attempts remaining)\n", qe->n1_remaining);
-
- rc = osmo_pfcp_endpoint_tx_data_no_logging(endpoint, m);
- /* If encoding failed, it cannot ever succeed. Drop the queue entry. */
- if (rc)
- return false;
/* if no more attempts remaining, drop from queue */
if (!qe->n1_remaining)
return false;
+
+ /* re-transmit */
+ qe->n1_remaining--;
+ OSMO_LOG_PFCP_MSG(m, LOGL_INFO, "re-sending (%u attempts remaining after this)\n", qe->n1_remaining);
+
+ rc = osmo_pfcp_endpoint_tx_data_no_logging(endpoint, m);
+ /* If encoding failed, it cannot ever succeed. Drop the queue entry. (Error logging already taken care of in
+ * osmo_pfcp_endpoint_tx_data_no_logging().) */
+ if (rc)
+ return false;
/* re-schedule timer, keep in queue */
osmo_timer_schedule(&qe->t1, t1_ms/1000, (t1_ms % 1000) * 1000);
return true;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/28812
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: Iaca396891921f7057015ce6e1e4528b955757809
Gerrit-Change-Number: 28812
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bsc/+/28811
to look at the new patch set (#2).
Change subject: tests/ctrl: Avoid creating logfile
......................................................................
tests/ctrl: Avoid creating logfile
It's of no use for the test. Furthemore, it was being created outside
the build direcory, being left there.
Change-Id: Iaeee14a01badb8439bc8893ba8be06b13e4318f3
---
M tests/ctrl/osmo-bsc-neigh-test.cfg
1 file changed, 0 insertions(+), 9 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/11/28811/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/28811
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Iaeee14a01badb8439bc8893ba8be06b13e4318f3
Gerrit-Change-Number: 28811
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
pespin has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/28775 )
Change subject: ttcn3-cbc: Set mode server in osmo-cbc.cfg
......................................................................
ttcn3-cbc: Set mode server in osmo-cbc.cfg
TCP/SCTP client support was added recently in osmo-cbc. Since then, the
connection mode (client|server|disabled) needs to be specified in the
config. Let's update osmo-cbc.git to provide the correct info in here.
Requires: osmo-cbc.git Change-Id I3ec54b615b41b56f7a9c64298e3fcaac37f4b60e
Change-Id: I2e3a806a8371011cdb4c3b1f194e132d3896374e
---
M ttcn3-cbc-test/osmo-cbc.cfg
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/ttcn3-cbc-test/osmo-cbc.cfg b/ttcn3-cbc-test/osmo-cbc.cfg
index 0e131c0..c2c1b7b 100644
--- a/ttcn3-cbc-test/osmo-cbc.cfg
+++ b/ttcn3-cbc-test/osmo-cbc.cfg
@@ -32,8 +32,10 @@
local-ip 172.18.27.20
local-port 29168
peer cbsp ttcn3-bsc
+ mode server
remote-port 9999
remote-ip 172.18.27.100
peer sbcap ttcn3-mme
+ mode server
remote-port 9998
remote-ip 172.18.27.100
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/28775
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I2e3a806a8371011cdb4c3b1f194e132d3896374e
Gerrit-Change-Number: 28775
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/28803 )
Change subject: sbcap: Improve logging on non-implemented PWS Restart/Failure Ind
......................................................................
sbcap: Improve logging on non-implemented PWS Restart/Failure Ind
Change-Id: I0085ac40a7676d9c3626a9da38d02d3b1b2cf6b4
---
M src/sbcap_link_fsm.c
1 file changed, 4 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
fixeria: Looks good to me, approved
diff --git a/src/sbcap_link_fsm.c b/src/sbcap_link_fsm.c
index 39dd281..3135f1f 100644
--- a/src/sbcap_link_fsm.c
+++ b/src/sbcap_link_fsm.c
@@ -80,7 +80,7 @@
//pdu = data;
/* TODO: delete any CBS state we have for this peer */
/* TODO: re-send messages we have matching the scope of the peer */
- LOGPSBCAPC(link, LOGL_NOTICE, "RESTART but re-sending not implemented yet\n");
+ LOGPSBCAPC(link, LOGL_NOTICE, "Rx PWS Restart Indication not implemented yet\n");
break;
default:
OSMO_ASSERT(0);
@@ -274,10 +274,12 @@
return osmo_fsm_inst_dispatch(link->fi, SBcAP_LINK_E_RX_RESTART, pdu);
case SBcAP_ProcedureId_Error_Indication:
return cbc_sbcap_link_rx_error_ind(link, pdu);
+ case SBcAP_ProcedureId_PWS_Failure_Indication:
+ LOGPSBCAPC(link, LOGL_NOTICE, "Rx PWS Failure Indication not implemented yet\n");
+ return 0;
case SBcAP_ProcedureId_Stop_Warning_Indication:
case SBcAP_ProcedureId_Write_Replace_Warning_Indication:
break; /* Handle msg id below */
- case SBcAP_ProcedureId_PWS_Failure_Indication:
default:
LOGPSBCAPC(link, LOGL_ERROR, "SBcAP initiatingMessage procedure=%ld not implemented?\n",
pdu->choice.initiatingMessage.procedureCode);
4 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28803
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I0085ac40a7676d9c3626a9da38d02d3b1b2cf6b4
Gerrit-Change-Number: 28803
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
Attention is currently required from: pespin.
Hello Jenkins Builder, laforge, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28803
to look at the new patch set (#5).
Change subject: sbcap: Improve logging on non-implemented PWS Restart/Failure Ind
......................................................................
sbcap: Improve logging on non-implemented PWS Restart/Failure Ind
Change-Id: I0085ac40a7676d9c3626a9da38d02d3b1b2cf6b4
---
M src/sbcap_link_fsm.c
1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/03/28803/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28803
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I0085ac40a7676d9c3626a9da38d02d3b1b2cf6b4
Gerrit-Change-Number: 28803
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
Hello Jenkins Builder, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28805
to look at the new patch set (#3).
Change subject: Move sbcap_as_find_ie() to libsbcap
......................................................................
Move sbcap_as_find_ie() to libsbcap
This function is generic and can be (will be) used in more places.
Change-Id: Ib0c7fa12a8f6faa4271e4132ca5a5c656e717400
---
M include/osmocom/sbcap/sbcap_common.h
M src/sbcap/sbcap_common.c
M src/sbcap_link_fsm.c
3 files changed, 16 insertions(+), 14 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/05/28805/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28805
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: Ib0c7fa12a8f6faa4271e4132ca5a5c656e717400
Gerrit-Change-Number: 28805
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
Hello Jenkins Builder, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28806
to look at the new patch set (#2).
Change subject: sbcap: Fix typo in comment
......................................................................
sbcap: Fix typo in comment
Change-Id: I6d519cb80a90e2a7e80e4608c713996f94043f21
---
M src/sbcap_link_fsm.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/06/28806/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28806
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I6d519cb80a90e2a7e80e4608c713996f94043f21
Gerrit-Change-Number: 28806
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
Hello Jenkins Builder, laforge, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-cbc/+/28804
to look at the new patch set (#5).
Change subject: sbcap: Tx Error ind if Rx ProcedureCode is unknown
......................................................................
sbcap: Tx Error ind if Rx ProcedureCode is unknown
Change-Id: I3ee760b98d87f179917acbc369f56d72d0db571d
---
M src/sbcap_link_fsm.c
1 file changed, 9 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/04/28804/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28804
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I3ee760b98d87f179917acbc369f56d72d0db571d
Gerrit-Change-Number: 28804
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: laforge.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/28802 )
Change subject: sbcap: Improve handling of rx Error Indication
......................................................................
Patch Set 5:
(1 comment)
Patchset:
PS5:
> I would suggest (next time) to put a one-line description on what that improvement is. […]
Indeed. Right now I only logging, but I also implemented all the parsing of the message iterating it, so now using it for whatever in the future should also be easier.
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/28802
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I63871a57f219a98f8191e6ee4787fc33edb21528
Gerrit-Change-Number: 28802
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Wed, 27 Jul 2022 11:41:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/28807 )
Change subject: trxcon: make l1sched_start_ciphering() accept const *key
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Patchset:
PS1:
IT would be more like pass, it already accepted const pointer before :P
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/28807
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I042903d779b6fbae68951a94788a458bf5226046
Gerrit-Change-Number: 28807
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 27 Jul 2022 11:13:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
fixeria has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/osmocom-bb/+/28810 )
Change subject: trxcon: assert *ts is not NULL in l1sched_set_lchans
......................................................................
trxcon: assert *ts is not NULL in l1sched_set_lchans
We cannot use ts->sched as logging context because ts is NULL.
Change-Id: I53b40b31f20fc9aefdfcac1406c465d226e5fde0
Related: CID#275337
---
M src/host/trxcon/src/sched_trx.c
1 file changed, 1 insertion(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/10/28810/2
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/28810
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I53b40b31f20fc9aefdfcac1406c465d226e5fde0
Gerrit-Change-Number: 28810
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-MessageType: newpatchset
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/o…
Package network:osmocom:nightly/osmo-hnbgw failed to build in Raspbian_11/armv7l
Check out the package for editing:
osc checkout network:osmocom:nightly osmo-hnbgw
Last lines of build log:
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2odt to provide /usr/bin/rst2odt (rst2odt) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2odt_prepstyles to provide /usr/bin/rst2odt_prepstyles (rst2odt_prepstyles) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2pseudoxml to provide /usr/bin/rst2pseudoxml (rst2pseudoxml) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2s5 to provide /usr/bin/rst2s5 (rst2s5) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2xetex to provide /usr/bin/rst2xetex (rst2xetex) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rst2xml to provide /usr/bin/rst2xml (rst2xml) in auto mode
[ 342s] update-alternatives: using /usr/share/docutils/scripts/python3/rstpep2html to provide /usr/bin/rstpep2html (rstpep2html) in auto mode
[ 343s] [458/491] installing python3-reportlab-3.5.59-2
[ 346s] [459/491] installing g++-4:10.2.1-1+rpi1
[ 346s] update-alternatives: using /usr/bin/g++ to provide /usr/bin/c++ (c++) in auto mode
[ 346s] [460/491] installing libosmo-netif-dev-1.2.0.4.dcae.202207270052
[ 346s] [461/491] installing mount-2.36.1-8+deb11u1
[ 347s] [462/491] installing texlive-latex-extra-2020.20210202-3
[ 360s] [463/491] installing debhelper-13.3.4
[ 360s] [464/491] installing bash-5.1-2+deb11u1
[ 361s] dpkg-deb (subprocess): decompressing archive '.init_b_cache/bash.deb' (size=1295464) member 'data.tar': lzma error: compressed data is corrupt
[ 361s] dpkg-deb: error: <decompress> subprocess returned error exit status 2
[ 361s] dpkg: error processing archive .init_b_cache/bash.deb (--install):
[ 361s] cannot copy extracted data for './bin/bash' to '/bin/bash.dpkg-new': unexpected end of file or stream
[ 361s] Errors were encountered while processing:
[ 361s] .init_b_cache/bash.deb
[ 361s] exit ...
[ 361s] ### VM INTERACTION START ###
[ 364s] [ 336.630022] sysrq: SysRq : Power Off
[ 364s] [ 336.631715] reboot: Power down
[ 364s] ### VM INTERACTION END ###
[ 364s]
[ 364s] armbuild01 failed "build osmo-hnbgw_1.3.0.1.7eb8.202207270052.dsc" at Wed Jul 27 06:11:14 UTC 2022.
[ 364s]
--
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)
Attention is currently required from: neels.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/28784 )
Change subject: sdp_msg: s/sdp_audio_codec_/sdp_audio_codecs_
......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-msc/+/28784/comment/6b7c1ab1_bf7373c6
PS3, Line 9: act on a struct sdp_audio_codecs
But they all return a struct sdp_audio_codec (singular), so I guess it's fine?
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/28784
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Id87eb350c1f17f8dbf776909824bfa06634c1d04
Gerrit-Change-Number: 28784
Gerrit-PatchSet: 3
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 26 Jul 2022 20:56:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: neels, pespin, dexter.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/28782 )
Change subject: in sdp logging: add payload type number like 'AMR#111'
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
> Good idea! Maybe it makes sense to print something like "AMR-PT=111" so that one knows that the numb […]
Ack, I like this approach more than 'AMR#111'.
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/28782
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Icbb4e89ce2947bf787c3ee14e3e115d406e43de2
Gerrit-Change-Number: 28782
Gerrit-PatchSet: 3
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 26 Jul 2022 20:53:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment