Attention is currently required from: fixeria.
pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email )
Change subject: enft_kpi: make enb_set_addr/1 non-blocking
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iacd380f66fc0249a6a645293abd5aac9cdba34d5
Gerrit-Change-Number: 40438
Gerrit-PatchSet: 3
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: Thu, 05 Jun 2025 17:41:27 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email )
Change subject: enft_kpi: make enb_set_addr/1 non-blocking
......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438/comment/974f451c_7e97… :
PS2, Line 9: This function is called each time a new E-REB is established, while
> E-RAB?
Acknowledged
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iacd380f66fc0249a6a645293abd5aac9cdba34d5
Gerrit-Change-Number: 40438
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Jun 2025 17:39:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Attention is currently required from: fixeria, pespin.
Hello Jenkins Builder, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Code-Review+1 by pespin
Change subject: enft_kpi: make enb_set_addr/1 non-blocking
......................................................................
enft_kpi: make enb_set_addr/1 non-blocking
This function is called each time a new E-RAB is established, while
the result, be it success or failure, does not matter to s1ap_proxy.
Use gen_server:cast/2 instead of gen_server:call/2 to avoid blocking.
Change-Id: Iacd380f66fc0249a6a645293abd5aac9cdba34d5
Related: SYS#7307
---
M src/enft_kpi.erl
1 file changed, 29 insertions(+), 30 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-s1gw refs/changes/38/40438/3
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40438?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iacd380f66fc0249a6a645293abd5aac9cdba34d5
Gerrit-Change-Number: 40438
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
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-ttcn3-hacks/+/40440?usp=email
to look at the new patch set (#6).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
5gc: Implement NAS integrity check of DL messages
Change-Id: I118081af10f260513734550854c3a1751e32cbb4
---
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/regen_makefile.sh
M deps/Makefile
M library/LTE_CryptoFunctions.ttcn
M library/NGAP_Emulation.ttcn
M library/NG_CryptoFunctionDefs.cc
M library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
M library/ng_crypto/key_derivation.c
M library/ng_crypto/key_derivation.h
11 files changed, 541 insertions(+), 38 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/40440/6
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 6
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
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-ttcn3-hacks/+/40440?usp=email
to look at the new patch set (#5).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
5gc: Implement NAS integrity check of DL messages
Change-Id: I118081af10f260513734550854c3a1751e32cbb4
---
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/regen_makefile.sh
M deps/Makefile
M library/LTE_CryptoFunctions.ttcn
M library/NGAP_Emulation.ttcn
M library/NG_CryptoFunctionDefs.cc
M library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
M library/ng_crypto/key_derivation.c
M library/ng_crypto/key_derivation.h
11 files changed, 540 insertions(+), 37 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/40440/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
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-ttcn3-hacks/+/40440?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
5gc: Implement NAS integrity check of DL messages
Change-Id: I118081af10f260513734550854c3a1751e32cbb4
---
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/regen_makefile.sh
M library/LTE_CryptoFunctions.ttcn
M library/NGAP_Emulation.ttcn
M library/NG_CryptoFunctionDefs.cc
M library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
M library/ng_crypto/key_derivation.c
M library/ng_crypto/key_derivation.h
10 files changed, 503 insertions(+), 31 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/40440/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
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-ttcn3-hacks/+/40440?usp=email
to look at the new patch set (#3).
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
5gc: Implement NAS integrity check of DL messages
Change-Id: I118081af10f260513734550854c3a1751e32cbb4
---
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/open5gs/open5gs-amf.yaml
M 5gc/regen_makefile.sh
M library/LTE_CryptoFunctions.ttcn
M library/NGAP_Emulation.ttcn
M library/NG_CryptoFunctionDefs.cc
M library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
M library/ng_crypto/key_derivation.c
M library/ng_crypto/key_derivation.h
11 files changed, 499 insertions(+), 29 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/40440/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Attention is currently required from: pespin.
Jenkins Builder has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email )
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
Patch Set 2:
(1 comment)
File library/ng_crypto/key_derivation.c:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440/comment/a4603bc8_ebcb… :
PS2, Line 156: }
adding a line without newline at end of file
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Jun 2025 16:06:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
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-ttcn3-hacks/+/40440?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: 5gc: Implement NAS integrity check of DL messages
......................................................................
5gc: Implement NAS integrity check of DL messages
Change-Id: I118081af10f260513734550854c3a1751e32cbb4
---
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/open5gs/open5gs-amf.yaml
M 5gc/regen_makefile.sh
M library/LTE_CryptoFunctions.ttcn
M library/NGAP_Emulation.ttcn
M library/NG_CryptoFunctionDefs.cc
M library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
M library/ng_crypto/key_derivation.c
M library/ng_crypto/key_derivation.h
11 files changed, 499 insertions(+), 29 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/40440/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40440?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I118081af10f260513734550854c3a1751e32cbb4
Gerrit-Change-Number: 40440
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>