August 2025 - gerrit-log - lists.osmocom.org

[XS] Change in ...logger_gsmtap[master]: logger_gsmtap.app.src: add vim modeline and syntax hint

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40801?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: logger_gsmtap.app.src: add vim modeline and syntax hint ...................................................................... logger_gsmtap.app.src: add vim modeline and syntax hint Change-Id: Id8cd8e1ab19ed2209ff1d867a7026b7663920013 --- M src/logger_gsmtap.app.src 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/logger_gsmtap.app.src b/src/logger_gsmtap.app.src index 0307fb0..8eb1a26 100644 --- a/src/logger_gsmtap.app.src +++ b/src/logger_gsmtap.app.src @@ -1,3 +1,5 @@ +%% -*- erlang -*- + {application, logger_gsmtap, [ {description, "GSMTAP handler for Erlang's Logger module"}, {vsn, semver}, @@ -9,3 +11,5 @@ {licenses, ["Mozilla Public License 2.0"]}, {links, [{"Gitea", "https://gitea.osmocom.org/erlang/logger_gsmtap"}]} ]}. + +%% vim:set ts=4 sw=4 et: -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40801?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: Id8cd8e1ab19ed2209ff1d867a7026b7663920013 Gerrit-Change-Number: 40801 Gerrit-PatchSet: 4 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: Fix warning: behaviour logger_handler undefined

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email ) Change subject: Fix warning: behaviour logger_handler undefined ...................................................................... Fix warning: behaviour logger_handler undefined Erlang/OTP versions below 27.0.0 have no 'logger_handler' behaviour, so let's declare it conditionally using the ?OTP_RELEASE macro. Change-Id: I11f5723a1b2fa46decee2a3117202d9882b859a7 --- M src/logger_gsmtap_h.erl 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/logger_gsmtap_h.erl b/src/logger_gsmtap_h.erl index c3cba56..99da8d6 100644 --- a/src/logger_gsmtap_h.erl +++ b/src/logger_gsmtap_h.erl @@ -15,7 +15,10 @@ GSMTAP logging handler. """. +%% Erlang/OTP versions below 27.0.0 have no 'logger_handler' behaviour +-if(?OTP_RELEASE >= 27). -behaviour(logger_handler). +-endif. %% logger_handler callbacks -export([adding_handler/1, -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I11f5723a1b2fa46decee2a3117202d9882b859a7 Gerrit-Change-Number: 40808 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: Fix warning: behaviour logger_handler undefined

by osmith

Attention is currently required from: fixeria. osmith has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email ) Change subject: Fix warning: behaviour logger_handler undefined ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I11f5723a1b2fa46decee2a3117202d9882b859a7 Gerrit-Change-Number: 40808 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 01 Aug 2025 12:48:22 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: TEST

by fixeria

fixeria has abandoned this change. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email ) Change subject: TEST ...................................................................... Abandoned -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: abandon Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I172c97e1998ca491a468c5f4b8c643b73592092c Gerrit-Change-Number: 40807 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: Fix warning: behaviour logger_handler undefined

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email ) Change subject: Fix warning: behaviour logger_handler undefined ...................................................................... Fix warning: behaviour logger_handler undefined Erlang/OTP versions below 27.0.0 have no 'logger_handler' behaviour, so let's declare it conditionally using the ?OTP_RELEASE macro. Change-Id: I11f5723a1b2fa46decee2a3117202d9882b859a7 --- M src/logger_gsmtap_h.erl 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/erlang/logger_gsmtap refs/changes/08/40808/1 diff --git a/src/logger_gsmtap_h.erl b/src/logger_gsmtap_h.erl index c3cba56..99da8d6 100644 --- a/src/logger_gsmtap_h.erl +++ b/src/logger_gsmtap_h.erl @@ -15,7 +15,10 @@ GSMTAP logging handler. """. +%% Erlang/OTP versions below 27.0.0 have no 'logger_handler' behaviour +-if(?OTP_RELEASE >= 27). -behaviour(logger_handler). +-endif. %% logger_handler callbacks -export([adding_handler/1, -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40808?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I11f5723a1b2fa46decee2a3117202d9882b859a7 Gerrit-Change-Number: 40808 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>

1 month

1
0
0 0

[S] Change in osmo-sgsn[master]: routing area: on alloc, check if RA already exists

by lynxis lazus

Attention is currently required from: fixeria, pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/40691?usp=email ) Change subject: routing area: on alloc, check if RA already exists ...................................................................... Patch Set 2: (1 comment) File src/sgsn/gprs_routing_area.c: https://gerrit.osmocom.org/c/osmo-sgsn/+/40691/comment/f6a780bb_3a729493?us… : PS2, Line 88: LOGL_ERROR > Please, leave this alloc function lone and use a find_or_create() API instead which uses this alloc […] It should only happen in rare cases, when the PCU + hnodeb/RNC is misconfigured. Ack at pespin. -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/40691?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I6955a09d9d8f3ba4dc3a14d7ed5cd9798d929c61 Gerrit-Change-Number: 40691 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 01 Aug 2025 12:45:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 month

1
0
0 0

[M] Change in osmo-sgsn[master]: routing area: introduce ran_type on the RA

by lynxis lazus

Attention is currently required from: fixeria, pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/40688?usp=email ) Change subject: routing area: introduce ran_type on the RA ...................................................................... Patch Set 2: (7 comments) Commit Message: https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/c7d40e7a_520e232c?us… : PS2, Line 9: unique > uniquely Done File include/osmocom/sgsn/gprs_routing_area.h: https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/7502a69a_eee4c86d?us… : PS2, Line 34: * and for routing, the SGSN must know to which PCU a given cell is connected. > "... and, for routing, the SGSN". […] Done https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/aee3fb1d_509a88bd?us… : PS2, Line 35: 2x PCU serves > Agree with fixeria, writing like does clarifies it. Done https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/e7dd3a47_c2ecd07c?us… : PS2, Line 38: individual > individually Done https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/2f5237c0_f8cfa01c?us… : PS2, Line 39: has > have Done https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/6145ce6d_a2b53cfe?us… : PS2, Line 47: which are alive > yeah, known_cells, active_cells, served_cells, whatever. […] Done File src/sgsn/gprs_routing_area.c: https://gerrit.osmocom.org/c/osmo-sgsn/+/40688/comment/2da1ab82_62cd4071?us… : PS2, Line 201: valid for GERAN > I agree, but this commit can't fix all that at the same time. Most of the previous api is related to geran. But will it do in a following commit. -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/40688?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I59c35f1a4912ff11574bb31e4fe424816993548c Gerrit-Change-Number: 40688 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Fri, 01 Aug 2025 12:42:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: TEST

by fixeria

fixeria has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email ) Change subject: TEST ...................................................................... TEST Change-Id: I172c97e1998ca491a468c5f4b8c643b73592092c --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/erlang/logger_gsmtap refs/changes/07/40807/2 -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I172c97e1998ca491a468c5f4b8c643b73592092c Gerrit-Change-Number: 40807 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in ...logger_gsmtap[master]: TEST

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email ) Change subject: TEST ...................................................................... TEST Change-Id: I172c97e1998ca491a468c5f4b8c643b73592092c --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/erlang/logger_gsmtap refs/changes/07/40807/1 diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index d58398d..30fc494 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -3,3 +3,5 @@ rebar3 clean rebar3 compile rebar3 dialyzer + +pwd -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/40807?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I172c97e1998ca491a468c5f4b8c643b73592092c Gerrit-Change-Number: 40807 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in osmo-ci[master]: jobs/gerrit-verifications: add logger_gsmtap

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/40804?usp=email ) Change subject: jobs/gerrit-verifications: add logger_gsmtap ...................................................................... jobs/gerrit-verifications: add logger_gsmtap Change-Id: I4ffae01156caa467ec23f6b89185b8d65d4691a3 Related: logger_gsmtap.git e06a0b9b3506e8ab1a7cfeeae72e32b2d6186989 --- M jobs/gerrit-verifications.yml 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml index fa32dc1..3d70e47 100644 --- a/jobs/gerrit-verifications.yml +++ b/jobs/gerrit-verifications.yml @@ -239,6 +239,12 @@ pipeline_binpkgs: "debian:12" cmd: '{docker_run} {docker_img_erlang} {timeout_cmd} /build/contrib/jenkins.sh' + - logger_gsmtap: + repos_url: '{gerrit_url}/erlang/{repos}' + gerrit_project: 'erlang/{repos}' + pipeline_binpkgs: "" + cmd: '{docker_run} {docker_img_erlang} {timeout_cmd} /build/contrib/jenkins.sh' + - osmo-e1-hardware: a2_name: JOB_TYPE a2: !!python/tuple ["firmware", "gateware", "manuals", "software"] -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/40804?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4ffae01156caa467ec23f6b89185b8d65d4691a3 Gerrit-Change-Number: 40804 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 month

1
0
0 0

2025

2024

2023

2022

gerrit-log August 2025