February 2024 - gerrit-log - lists.osmocom.org

[M] Change in pysim[master]: saip.personalization: Add support for SCP80/81/02/03 keys

by laforge

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/36015?usp=email ) Change subject: saip.personalization: Add support for SCP80/81/02/03 keys ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/36015?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ibe4806366f1cce8edb09d52613b1dd56250fa5ae Gerrit-Change-Number: 36015 Gerrit-PatchSet: 5 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Thu, 22 Feb 2024 10:23:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[M] Change in pysim[master]: saip.personalization: include encode/decode of value; add validation ...

by laforge

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/36014?usp=email ) Change subject: saip.personalization: include encode/decode of value; add validation method ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/36014?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ia9fa39c25817448afb191061acd4be894300eeef Gerrit-Change-Number: 36014 Gerrit-PatchSet: 5 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Thu, 22 Feb 2024 10:23:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[S] Change in libgtpnl[master]: tools/gtp-tunnel: pass rc of gtp_del_tunnel

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36033?usp=email ) Change subject: tools/gtp-tunnel: pass rc of gtp_del_tunnel ...................................................................... tools/gtp-tunnel: pass rc of gtp_del_tunnel Don't exit with 0 if gtp_del_tunnel fails. Change-Id: I8046a57a5831e6bdee2ac37415380ce391c79c22 --- M tools/gtp-tunnel.c 1 file changed, 14 insertions(+), 2 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/tools/gtp-tunnel.c b/tools/gtp-tunnel.c index e8db171..2a5540a 100644 --- a/tools/gtp-tunnel.c +++ b/tools/gtp-tunnel.c @@ -131,6 +131,7 @@ { struct gtp_tunnel *t; uint32_t gtp_ifidx; + int ret; if (argc != 6) { printf("%s del <gtp device> <version> <tid|i_tei> <family>\n", @@ -171,10 +172,10 @@ return EXIT_FAILURE; } - gtp_del_tunnel(genl_id, nl, t); + ret = gtp_del_tunnel(genl_id, nl, t); gtp_tunnel_free(t); - return 0; + return ret; } struct gtp_pdp { -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36033?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: I8046a57a5831e6bdee2ac37415380ce391c79c22 Gerrit-Change-Number: 36033 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 year, 4 months

1
0
0 0

[M] Change in libgtpnl[master]: tests/qemu: add 05_ms_ip46_sgsn_ip4.sh

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36034?usp=email ) Change subject: tests/qemu: add 05_ms_ip46_sgsn_ip4.sh ...................................................................... tests/qemu: add 05_ms_ip46_sgsn_ip4.sh Change-Id: I1e306bc4534f14138249cd4034f54f77d940b261 --- M tests/qemu/00_test_functions.sh A tests/qemu/05_ms_ip46_sgsn_ip4.sh M tests/qemu/initrd-init.sh 3 files changed, 79 insertions(+), 0 deletions(-) Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/tests/qemu/00_test_functions.sh b/tests/qemu/00_test_functions.sh index 5839363..7be8dbf 100644 --- a/tests/qemu/00_test_functions.sh +++ b/tests/qemu/00_test_functions.sh @@ -50,16 +50,52 @@ ggsn_side gtp-tunnel list } +# Add a second tunnel to test MS with IPv4v6 +tunnel_start_2() { + test -n "$MS2_PROTO" + test -n "$MS2" + test -n "$MS2_PREFLEN" + test -n "$WEBSERVER2" + + # SGSN side: add second IP to lo (MS) + ip addr add "$MS2"/"$MS2_PREFLEN" dev lo + + # SGSN side: prepare second gtp-tunnel + gtp-tunnel add gtp_sgsn v1 200 100 "$MS2" "$GGSN" + ip route add "$WEBSERVER2"/"$MS2_PREFLEN" dev gtp_sgsn + + # GGSN side: add second IP to lo (WEBSERVER) + ggsn_side ip addr add "$WEBSERVER2"/"$MS2_PREFLEN" dev lo + + # GGSN side: prepare second gtp-tunnel + ggsn_side gtp-tunnel add gtp_ggsn v1 100 200 "$MS2" "$SGSN" + ggsn_side ip route add "$MS2"/"$MS2_PREFLEN" dev gtp_ggsn + + # List tunnels from both sides + gtp-tunnel list + ggsn_side gtp-tunnel list +} + tunnel_ping() { ip addr show ping -c 1 "$WEBSERVER" ggsn_side ping -c 1 "$MS" + + if [ -n "$MS2" ]; then + ping -c 1 "$WEBSERVER2" + ggsn_side ping -c 1 "$MS2" + fi } tunnel_stop() { killall gtp-link ip addr del "$MS"/"$MS_PREFLEN" dev lo + + if [ -n "$MS2" ]; then + ip addr del "$MS2"/"$MS2_PREFLEN" dev lo + fi + ip link set veth_sgsn down if [ "$SGSN_GGSN_PROTO" == "ip" ]; then # FIXME: doesn't work with ip6 @@ -69,8 +105,19 @@ ip link del veth_sgsn ip route del "$WEBSERVER"/"$MS_PREFLEN" dev gtp_sgsn gtp-tunnel delete gtp_sgsn v1 200 "$MS_PROTO" + + if [ -n "$MS2" ]; then + ip route del "$WEBSERVER2"/"$MS2_PREFLEN" dev gtp_sgsn + gtp-tunnel delete gtp_sgsn v1 200 "$MS2_PROTO" + fi + gtp-link del gtp_sgsn ggsn_side gtp-tunnel delete gtp_ggsn v1 100 "$MS_PROTO" + + if [ -n "$MS2" ]; then + ggsn_side gtp-tunnel delete gtp_ggsn v1 100 "$MS2_PROTO" + fi + ggsn_side gtp-link del gtp_ggsn ip netns del ggsn_side } diff --git a/tests/qemu/05_ms_ip46_sgsn_ip4.sh b/tests/qemu/05_ms_ip46_sgsn_ip4.sh new file mode 100644 index 0000000..0bd48bb --- /dev/null +++ b/tests/qemu/05_ms_ip46_sgsn_ip4.sh @@ -0,0 +1,22 @@ +#!/bin/sh -ex +. /tests/00_test_functions.sh + +MS_PROTO="ip" +MS="172.99.0.1" +MS_PREFLEN="32" +SGSN_GGSN_PROTO="ip" +SGSN="172.0.0.1" +SGSN_PREFLEN="24" +GGSN="172.0.0.2" +WEBSERVER="172.99.0.2" + +tunnel_start + +MS2_PROTO="ip6" +MS2="fd00::" +MS2_PREFLEN="64" +WEBSERVER2="fe00::2" + +tunnel_start_2 +tunnel_ping +tunnel_stop diff --git a/tests/qemu/initrd-init.sh b/tests/qemu/initrd-init.sh index ba20592..5dacb86 100755 --- a/tests/qemu/initrd-init.sh +++ b/tests/qemu/initrd-init.sh @@ -29,6 +29,7 @@ run_test 02_ms_ip4_sgsn_ip6.sh run_test 03_ms_ip6_sgsn_ip4.sh run_test 04_ms_ip6_sgsn_ip6.sh +run_test 05_ms_ip46_sgsn_ip4.sh # Success (run-qemu.sh checks for this line) echo "QEMU_TEST_SUCCESSFUL" -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36034?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: I1e306bc4534f14138249cd4034f54f77d940b261 Gerrit-Change-Number: 36034 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 year, 4 months

1
0
0 0

[S] Change in libgtpnl[master]: tests/qemu: pass MS_PROTO to gtp-tunnel delete

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36032?usp=email ) Change subject: tests/qemu: pass MS_PROTO to gtp-tunnel delete ...................................................................... tests/qemu: pass MS_PROTO to gtp-tunnel delete The MS protocol needs to be passed to gtp-tunnel delete, instead of the SGSN_GGSN_PROTO. Fix for: + gtp-tunnel add gtp_sgsn v1 200 100 172.99.0.1 fd00::2 … + gtp-tunnel delete gtp_sgsn v1 200 ip6 genl_socket_talk: No such file or directory Change-Id: Ib6b011520fe41855b87ed0e2f6f8a6af3d0b2400 --- M tests/qemu/00_test_functions.sh M tests/qemu/01_ms_ip4_sgsn_ip4.sh M tests/qemu/02_ms_ip4_sgsn_ip6.sh M tests/qemu/03_ms_ip6_sgsn_ip4.sh M tests/qemu/04_ms_ip6_sgsn_ip6.sh 5 files changed, 25 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/tests/qemu/00_test_functions.sh b/tests/qemu/00_test_functions.sh index c4813ab..5839363 100644 --- a/tests/qemu/00_test_functions.sh +++ b/tests/qemu/00_test_functions.sh @@ -8,6 +8,7 @@ # MS - SGSN -gtp- GGSN - WEBSERVER tunnel_start() { + test -n "$MS_PROTO" test -n "$MS" test -n "$MS_PREFLEN" test -n "$SGSN_GGSN_PROTO" @@ -67,9 +68,9 @@ ip link del veth_sgsn ip route del "$WEBSERVER"/"$MS_PREFLEN" dev gtp_sgsn - gtp-tunnel delete gtp_sgsn v1 200 "$SGSN_GGSN_PROTO" + gtp-tunnel delete gtp_sgsn v1 200 "$MS_PROTO" gtp-link del gtp_sgsn - ggsn_side gtp-tunnel delete gtp_ggsn v1 100 "$SGSN_GGSN_PROTO" + ggsn_side gtp-tunnel delete gtp_ggsn v1 100 "$MS_PROTO" ggsn_side gtp-link del gtp_ggsn ip netns del ggsn_side } diff --git a/tests/qemu/01_ms_ip4_sgsn_ip4.sh b/tests/qemu/01_ms_ip4_sgsn_ip4.sh index 42e2842..4789483 100644 --- a/tests/qemu/01_ms_ip4_sgsn_ip4.sh +++ b/tests/qemu/01_ms_ip4_sgsn_ip4.sh @@ -1,6 +1,7 @@ #!/bin/sh -ex . /tests/00_test_functions.sh +MS_PROTO="ip" MS="172.99.0.1" MS_PREFLEN="32" SGSN_GGSN_PROTO="ip" diff --git a/tests/qemu/02_ms_ip4_sgsn_ip6.sh b/tests/qemu/02_ms_ip4_sgsn_ip6.sh index b5858ab..2a489cd 100644 --- a/tests/qemu/02_ms_ip4_sgsn_ip6.sh +++ b/tests/qemu/02_ms_ip4_sgsn_ip6.sh @@ -1,6 +1,7 @@ #!/bin/sh -ex . /tests/00_test_functions.sh +MS_PROTO="ip" MS="172.99.0.1" MS_PREFLEN="32" SGSN_GGSN_PROTO="ip6" diff --git a/tests/qemu/03_ms_ip6_sgsn_ip4.sh b/tests/qemu/03_ms_ip6_sgsn_ip4.sh index 12793b0..f873faf 100644 --- a/tests/qemu/03_ms_ip6_sgsn_ip4.sh +++ b/tests/qemu/03_ms_ip6_sgsn_ip4.sh @@ -1,6 +1,7 @@ #!/bin/sh -ex . /tests/00_test_functions.sh +MS_PROTO="ip6" MS="fd00::" MS_PREFLEN="64" SGSN_GGSN_PROTO="ip" diff --git a/tests/qemu/04_ms_ip6_sgsn_ip6.sh b/tests/qemu/04_ms_ip6_sgsn_ip6.sh index 6c2f13f..483a9d7 100644 --- a/tests/qemu/04_ms_ip6_sgsn_ip6.sh +++ b/tests/qemu/04_ms_ip6_sgsn_ip6.sh @@ -1,6 +1,7 @@ #!/bin/sh -ex . /tests/00_test_functions.sh +MS_PROTO="ip6" MS="fc00::" MS_PREFLEN="64" SGSN_GGSN_PROTO="ip6" -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36032?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: Ib6b011520fe41855b87ed0e2f6f8a6af3d0b2400 Gerrit-Change-Number: 36032 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 year, 4 months

1
0
0 0

[M] Change in libgtpnl[master]: tests/qemu: add 05_ms_ip46_sgsn_ip4.sh

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36034?usp=email ) Change subject: tests/qemu: add 05_ms_ip46_sgsn_ip4.sh ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36034?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: I1e306bc4534f14138249cd4034f54f77d940b261 Gerrit-Change-Number: 36034 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 22 Feb 2024 10:23:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[S] Change in libgtpnl[master]: tools/gtp-tunnel: pass rc of gtp_del_tunnel

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36033?usp=email ) Change subject: tools/gtp-tunnel: pass rc of gtp_del_tunnel ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36033?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: I8046a57a5831e6bdee2ac37415380ce391c79c22 Gerrit-Change-Number: 36033 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 22 Feb 2024 10:22:53 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[S] Change in libgtpnl[master]: tests/qemu: pass MS_PROTO to gtp-tunnel delete

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/36032?usp=email ) Change subject: tests/qemu: pass MS_PROTO to gtp-tunnel delete ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libgtpnl/+/36032?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libgtpnl Gerrit-Branch: master Gerrit-Change-Id: Ib6b011520fe41855b87ed0e2f6f8a6af3d0b2400 Gerrit-Change-Number: 36032 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Thu, 22 Feb 2024 10:22:42 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[S] Change in docker-playground[master]: ttcn3-epdg: Give some time for osmo-epdg to start

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36043?usp=email ) Change subject: ttcn3-epdg: Give some time for osmo-epdg to start ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36043?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I14e39db6548409b203e15fb562e0b44a90bd44cd Gerrit-Change-Number: 36043 Gerrit-PatchSet: 1 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-Comment-Date: Thu, 22 Feb 2024 10:22:15 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

[S] Change in docker-playground[master]: ttcn3-epdg: Create tun with iface name gtp0

by laforge

Attention is currently required from: pespin. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36042?usp=email ) Change subject: ttcn3-epdg: Create tun with iface name gtp0 ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36042?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I066c22a381ab85762c8ef247a5eb248f7adf6c71 Gerrit-Change-Number: 36042 Gerrit-PatchSet: 1 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-Comment-Date: Thu, 22 Feb 2024 10:21:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 4 months

1
0
0 0

2025

2024

2023

2022

gerrit-log February 2024