Attention is currently required from: pespin.
Hello Jenkins Builder, fixeria, osmith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/libosmo-sigtran/+/40696?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: xua_asp_fsm: Move helper function to ss7_asp internal API
......................................................................
xua_asp_fsm: Move helper function to ss7_asp internal API
This function will be used in other files in follow-up commit.
Change-Id: I3eb57c3fb690e7486b52b2c433b25930d16599e5
---
M src/ss7_asp.c
M src/ss7_asp.h
M src/xua_asp_fsm.c
3 files changed, 30 insertions(+), 28 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/96/40696/3
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40696?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I3eb57c3fb690e7486b52b2c433b25930d16599e5
Gerrit-Change-Number: 40696
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Attention is currently required from: osmith.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40684?usp=email )
Change subject: xua: Introduce internal APIs to transmit DAUD
......................................................................
Patch Set 2:
(1 comment)
File src/m3ua.c:
https://gerrit.osmocom.org/c/libosmo-sigtran/+/40684/comment/f8daa8c3_b7f78… :
PS2, Line 988: /* encoding is exactly identical to DUNA */
> I'd drop that comment here, given that it isn't calling m3ua_encode_duna, xua->hdr. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40684?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Iab6126f1a9eac1a66bb8ec362b87e20a5eb795ff
Gerrit-Change-Number: 40684
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 16 Jul 2025 11:47:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40701?usp=email )
Change subject: rebar.{config,lock}: use a fork of exometer_report_statsd
......................................................................
rebar.{config,lock}: use a fork of exometer_report_statsd
As it turns out, the statsd reporter plugin that we have been using
so far has a flaw: the way it reports counters differs from what
daemons like statsd or collectd expect. The problem is that this
plugin simply reports current values, while the delta is expected.
I'll try to get the upstream fixed, but for now let's switch to a fork.
Change-Id: Ic5c80dd66b2659efc6bf148b91316b7e45dfaf7b
Related: SYS#7065, SYS#7307
---
M rebar.config
M rebar.lock
2 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-s1gw refs/changes/01/40701/1
diff --git a/rebar.config b/rebar.config
index 0100f17..591ec61 100644
--- a/rebar.config
+++ b/rebar.config
@@ -12,7 +12,7 @@
{exometer_core,
{git, "https://github.com/Feuerlabs/exometer_core.git", {branch, "master"}}},
{exometer_report_statsd,
- {git, "https://github.com/esl/exometer_report_statsd.git", {branch, "master"}}}
+ {git, "https://gitea.osmocom.org/vyanitskiy/exometer_report_statsd.git", {branch, "master"}}}
]}.
%% test deps
diff --git a/rebar.lock b/rebar.lock
index feb8d54..da069fe 100644
--- a/rebar.lock
+++ b/rebar.lock
@@ -5,8 +5,8 @@
{ref,"fc744c45bbe5de2ab0ee550e2667f93f7abbb618"}},
0},
{<<"exometer_report_statsd">>,
- {git,"https://github.com/esl/exometer_report_statsd.git",
- {ref,"f1c369becb6e57871f1c7b0e491f6c3a302a65ee"}},
+ {git,"https://gitea.osmocom.org/vyanitskiy/exometer_report_statsd.git",
+ {ref,"158a9708ec3fce8c74648b5da1c8fac69ac5d05a"}},
0},
{<<"hut">>,{pkg,<<"hut">>,<<"1.3.0">>},1},
{<<"logger_color_formatter">>,
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40701?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ic5c80dd66b2659efc6bf148b91316b7e45dfaf7b
Gerrit-Change-Number: 40701
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: pespin.
fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40696?usp=email )
Change subject: xua_asp_fsm: Move helper function to ss7_asp internal API
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40696?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I3eb57c3fb690e7486b52b2c433b25930d16599e5
Gerrit-Change-Number: 40696
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 16 Jul 2025 07:57:33 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes