lists.osmocom.org
Sign In Sign Up
Manage this list Sign In Sign Up

Keyboard Shortcuts

Thread View

  • j: Next unread message
  • k: Previous unread message
  • j a: Jump to all threads
  • j l: Jump to MailingList overview

gerrit-log

Thread Start a new thread
Download
Threads by month
  • ----- 2026 -----
  • October
  • September
  • August
  • July
  • June
  • May
  • April
  • March
  • February
  • January
  • ----- 2025 -----
  • December
  • November
  • October
  • September
  • August
  • July
  • June
  • May
  • April
  • March
  • February
  • January
  • ----- 2024 -----
  • December
  • November
  • October
  • September
  • August
  • July
  • June
  • May
  • April
  • March
  • February
  • January
  • ----- 2023 -----
  • December
  • November
  • October
  • September
  • August
  • July
  • June
  • May
  • April
  • March
  • February
  • January
  • ----- 2022 -----
  • December
  • November
  • October
  • September
  • August
  • July
  • June
  • May
  • April
  • March
  • February
  • January
gerrit-log@lists.osmocom.org

August 2026

  • 1 participants
  • 2269 discussions
[S] Change in libosmo-sigtran[master]: asp: vty: Explicitly test timer lm value ranges
by pespin 03 Aug '26

03 Aug '26
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43157?usp=email ) Change subject: asp: vty: Explicitly test timer lm value ranges ...................................................................... asp: vty: Explicitly test timer lm value ranges Validate the value to be set provided by user over VTY is actually within the range of the specific tdef being set. Mark all of them explicitly as min_val=1 in its tdef, since they are not expected to be set to 0 (see also gen_asp_timer_lm_cmd_strs() defining a VTY range <1..999999>). Change-Id: I9b2c426a1eaceb6e7c5fbe35872ada324fef5733 --- M src/ss7_asp_vty.c M src/xua_default_lm_fsm.c M tests/vty/osmo_stp_test.vty M tests/vty/ss7_asp_test.vty 4 files changed, 34 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/57/43157/1 diff --git a/src/ss7_asp_vty.c b/src/ss7_asp_vty.c index 21d19d6..5136105 100644 --- a/src/ss7_asp_vty.c +++ b/src/ss7_asp_vty.c @@ -891,13 +891,27 @@ { struct osmo_ss7_asp *asp = vty->index; enum ss7_asp_lm_timer timer = get_string_value(ss7_asp_lm_timer_names, argv[0]); + int rc; + unsigned long new_val = atoi(argv[1]); if (timer <= 0 || timer >= SS7_ASP_LM_TIMERS_LEN) { vty_out(vty, "%% Invalid timer: %s%s", argv[0], VTY_NEWLINE); return CMD_WARNING; } - osmo_tdef_set(asp->cfg.T_defs_lm, timer, atoi(argv[1]), OSMO_TDEF_S); + rc = osmo_tdef_set(asp->cfg.T_defs_lm, timer, new_val, OSMO_TDEF_S); + if (rc < 0) { + struct osmo_tdef *t = osmo_tdef_get_entry(asp->cfg.T_defs_lm, timer); + if (!t) + return CMD_WARNING; + if (!osmo_tdef_val_in_range(t, new_val)) { + char range_str[64]; + osmo_tdef_range_str_buf(range_str, sizeof(range_str), t); + vty_out(vty, "%% Timer %s value %lu is out of range %s%s", + argv[0], new_val, range_str, VTY_NEWLINE); + } + return CMD_WARNING; + } return CMD_SUCCESS; } diff --git a/src/xua_default_lm_fsm.c b/src/xua_default_lm_fsm.c index 73c4fdc..db84e6e 100644 --- a/src/xua_default_lm_fsm.c +++ b/src/xua_default_lm_fsm.c @@ -102,13 +102,17 @@ const struct osmo_tdef ss7_asp_lm_timer_defaults[SS7_ASP_LM_TIMERS_LEN] = { { .T = SS7_ASP_LM_T_WAIT_ASP_UP, .default_val = 20, .unit = OSMO_TDEF_S, - .desc = "Restart ASP after timeout waiting for ASP UP (SG role) / ASP UP ACK (ASP role) (s)" }, + .desc = "Restart ASP after timeout waiting for ASP UP (SG role) / ASP UP ACK (ASP role) (s)", + .min_val = 1 }, { .T = SS7_ASP_LM_T_WAIT_NOTIFY, .default_val = 2, .unit = OSMO_TDEF_S, - .desc = "Restart ASP after timeout waiting for NOTIFY (s)" }, + .desc = "Restart ASP after timeout waiting for NOTIFY (s)", + .min_val = 1 }, { .T = SS7_ASP_LM_T_WAIT_NOTIY_RKM, .default_val = 20, .unit = OSMO_TDEF_S, - .desc = "Restart ASP after timeout waiting for NOTIFY after RKM registration (s)" }, + .desc = "Restart ASP after timeout waiting for NOTIFY after RKM registration (s)", + .min_val = 1 }, { .T = SS7_ASP_LM_T_WAIT_RK_REG_RESP, .default_val = 10, .unit = OSMO_TDEF_S, - .desc = "Restart ASP after timeout waiting for RK_REG_RESP (s)" }, + .desc = "Restart ASP after timeout waiting for RK_REG_RESP (s)", + .min_val = 1 }, {} }; diff --git a/tests/vty/osmo_stp_test.vty b/tests/vty/osmo_stp_test.vty index 2dbd5b2..d52ae5f 100644 --- a/tests/vty/osmo_stp_test.vty +++ b/tests/vty/osmo_stp_test.vty @@ -415,6 +415,11 @@ ... !timer xua beat 30 ... +OsmoSTP(config-cs7-asp)# timer lm ? + wait_asp_up Restart ASP after timeout waiting for ASP UP (SG role) / ASP UP ACK (ASP role) (s) (default: 20) + wait_notify Restart ASP after timeout waiting for NOTIFY (s) (default: 2) + wait_notify_rkm Restart ASP after timeout waiting for NOTIFY after RKM registration (s) (default: 20) + wait_rk_reg_resp Restart ASP after timeout waiting for RK_REG_RESP (s) (default: 10) OsmoSTP(config-cs7-asp)# exit % NOTE: Skipping automatic restart of ASP since an explicit '[no] shutdown' command was entered diff --git a/tests/vty/ss7_asp_test.vty b/tests/vty/ss7_asp_test.vty index f3193b6..5ea965b 100644 --- a/tests/vty/ss7_asp_test.vty +++ b/tests/vty/ss7_asp_test.vty @@ -432,6 +432,12 @@ ... !timer xua beat 30 ... +ss7_asp_vty_test(config-cs7-asp)# timer lm ? + wait_asp_up Restart ASP after timeout waiting for ASP UP (SG role) / ASP UP ACK (ASP role) (s) (default: 20) + wait_notify Restart ASP after timeout waiting for NOTIFY (s) (default: 2) + wait_notify_rkm Restart ASP after timeout waiting for NOTIFY after RKM registration (s) (default: 20) + wait_rk_reg_resp Restart ASP after timeout waiting for RK_REG_RESP (s) (default: 10) + ss7_asp_vty_test(config-cs7-asp)# exit % NOTE: Make sure to use '[no] shutdown' command in 'asp' node in order to restart the ASP for new configs to be applied. -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43157?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I9b2c426a1eaceb6e7c5fbe35872ada324fef5733 Gerrit-Change-Number: 43157 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>
1 0
0 0
[M] Change in libosmo-sigtran[master]: asp: Support setting timer xua beat 0 through VTY
by pespin 03 Aug '26

03 Aug '26
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43156?usp=email ) Change subject: asp: Support setting timer xua beat 0 through VTY ...................................................................... asp: Support setting timer xua beat 0 through VTY Previous VTY configured range for the set of tdefs in timer xua didn't allow to pass a 0 to it, which "timer xua beat 0" should be able to accept in order to disable heartbeat procedure on the ASP. In order to fix it: * Change VTY range to minimal accepted value to 0 for the whole tdef set. * Make sure the VTY command validates and rejects user trying to set a value outside the accepted range of the specific timer. * Set .min_val=1 in SS7_ASP_XUA_T_ACK, so it can't accept value 0. According to spec, T(ack) is always expected to be set, even if the resulting timeout may be handled by LM instead of simply retransmitting the packet. * Write vty tests to validate the restrictions are followed properly in the VTY. Related: SYS#8156 Change-Id: Iadd7c6751ab09640fd2df214afa2ec87274a3210 --- M src/ss7_asp.c M src/ss7_asp_vty.c M tests/vty/osmo_stp_test.vty M tests/vty/ss7_asp_test.vty 4 files changed, 101 insertions(+), 7 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/56/43156/1 diff --git a/src/ss7_asp.c b/src/ss7_asp.c index c47ee7c..5a47248 100644 --- a/src/ss7_asp.c +++ b/src/ss7_asp.c @@ -121,9 +121,10 @@ const struct osmo_tdef ss7_asp_xua_timer_defaults[SS7_ASP_XUA_TIMERS_LEN] = { { .T = SS7_ASP_XUA_T_ACK, .default_val = SS7_ASP_XUA_DEFAULT_T_ACK_SEC, .unit = OSMO_TDEF_S, - .desc = "Resend ASP Up/Down/Active/Inactive after timeout waiting for ASP Up/Down/Active/Inactive ACK (ASP role)(s)" }, + .desc = "T(ack): Resend ASP Up/Down/Active/Inactive after timeout waiting for ASP Up/Down/Active/Inactive ACK (ASP role) (s)", + .min_val = 1 }, { .T = SS7_ASP_XUA_T_BEAT, .default_val = SS7_ASP_XUA_DEFAULT_T_BEAT_SEC, .unit = OSMO_TDEF_S, - .desc = "Heartbeat Timer (0 = disabled) (s)" }, + .desc = "T(beat): Heartbeat Timer (0 = disabled) (s)" }, {} }; diff --git a/src/ss7_asp_vty.c b/src/ss7_asp_vty.c index c38b5b8..21d19d6 100644 --- a/src/ss7_asp_vty.c +++ b/src/ss7_asp_vty.c @@ -814,20 +814,34 @@ return CMD_SUCCESS; } -/* timer xua <name> <1-999999> +/* timer xua <name> <0-999999> * (cmdstr and doc are dynamically generated from ss7_asp_xua_timer_names.) */ DEFUN_ATTR(asp_timer_xua, asp_timer_xua_cmd, NULL, NULL, CMD_ATTR_IMMEDIATE) { struct osmo_ss7_asp *asp = vty->index; enum ss7_asp_xua_timer timer = get_string_value(ss7_asp_xua_timer_names, argv[0]); + int rc; + unsigned long new_val = atoi(argv[1]); if (timer <= 0 || timer >= SS7_ASP_XUA_TIMERS_LEN) { vty_out(vty, "%% Invalid timer: %s%s", argv[0], VTY_NEWLINE); return CMD_WARNING; } - osmo_tdef_set(asp->cfg.T_defs_xua, timer, atoi(argv[1]), OSMO_TDEF_S); + rc = osmo_tdef_set(asp->cfg.T_defs_xua, timer, new_val, OSMO_TDEF_S); + if (rc < 0) { + struct osmo_tdef *t = osmo_tdef_get_entry(asp->cfg.T_defs_xua, timer); + if (!t) + return CMD_WARNING; + if (!osmo_tdef_val_in_range(t, new_val)) { + char range_str[64]; + osmo_tdef_range_str_buf(range_str, sizeof(range_str), t); + vty_out(vty, "%% Timer %s value %lu is out of range %s%s", + argv[0], new_val, range_str, VTY_NEWLINE); + } + return CMD_WARNING; + } return CMD_SUCCESS; } @@ -861,7 +875,7 @@ def->default_val); } - osmo_talloc_asprintf(tall_vty_ctx, cmd_str, ") <1-999999>"); + osmo_talloc_asprintf(tall_vty_ctx, cmd_str, ") <0-999999>"); osmo_talloc_asprintf(tall_vty_ctx, doc_str, "Timer value, in seconds\n"); diff --git a/tests/vty/osmo_stp_test.vty b/tests/vty/osmo_stp_test.vty index 1b452c8..2dbd5b2 100644 --- a/tests/vty/osmo_stp_test.vty +++ b/tests/vty/osmo_stp_test.vty @@ -302,7 +302,7 @@ no destination-audit asp-active quirk (no_notify|daud_in_asp|snm_inactive) no quirk (no_notify|daud_in_asp|snm_inactive) - timer xua (ack|beat) <1-999999> + timer xua (ack|beat) <0-999999> timer lm (wait_asp_up|wait_notify|wait_notify_rkm|wait_rk_reg_resp) <1-999999> block no block @@ -386,6 +386,36 @@ ... end OsmoSTP(config-cs7-asp)# no shutdown + +OsmoSTP(config-cs7-asp)# ### Test ASP node timers +OsmoSTP(config-cs7-asp)# timer ? + xua Configure ASP default xua timer values + lm Configure ASP default lm timer values +OsmoSTP(config-cs7-asp)# timer xua ? + ack T(ack): Resend ASP Up/Down/Active/Inactive after timeout waiting for ASP Up/Down/Active/Inactive ACK (ASP role) (s) (default: 2) + beat T(beat): Heartbeat Timer (0 = disabled) (s) (default: 30) +OsmoSTP(config-cs7-asp)# timer xua beat 0 +OsmoSTP(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... + timer xua beat 0 +... +OsmoSTP(config-cs7-asp)# timer xua beat 5 +OsmoSTP(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... + timer xua beat 5 +... +OsmoSTP(config-cs7-asp)# timer xua beat 30 +OsmoSTP(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... !timer xua beat 30 +... + + OsmoSTP(config-cs7-asp)# exit % NOTE: Skipping automatic restart of ASP since an explicit '[no] shutdown' command was entered OsmoSTP(config-cs7)# as my-ass m3ua diff --git a/tests/vty/ss7_asp_test.vty b/tests/vty/ss7_asp_test.vty index e93e037..f3193b6 100644 --- a/tests/vty/ss7_asp_test.vty +++ b/tests/vty/ss7_asp_test.vty @@ -298,7 +298,7 @@ no destination-audit asp-active quirk (no_notify|daud_in_asp|snm_inactive) no quirk (no_notify|daud_in_asp|snm_inactive) - timer xua (ack|beat) <1-999999> + timer xua (ack|beat) <0-999999> timer lm (wait_asp_up|wait_notify|wait_notify_rkm|wait_rk_reg_resp) <1-999999> block no block @@ -383,6 +383,55 @@ remote-ip 127.0.0.201 ... end + +ss7_asp_vty_test(config-cs7-asp)# ### Test ASP node timers +ss7_asp_vty_test(config-cs7-asp)# timer ? + xua Configure ASP default xua timer values + lm Configure ASP default lm timer values +ss7_asp_vty_test(config-cs7-asp)# timer xua ? + ack T(ack): Resend ASP Up/Down/Active/Inactive after timeout waiting for ASP Up/Down/Active/Inactive ACK (ASP role) (s) (default: 2) + beat T(beat): Heartbeat Timer (0 = disabled) (s) (default: 30) +ss7_asp_vty_test(config-cs7-asp)# timer xua ack 1 +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... + timer xua ack 1 +... +ss7_asp_vty_test(config-cs7-asp)# timer xua ack 2 +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... !timer xua ack 1 +... +ss7_asp_vty_test(config-cs7-asp)# timer xua ack 0 +% Timer ack value 0 is out of range [1 .. inf] +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... !timer xua ack 0 +... +ss7_asp_vty_test(config-cs7-asp)# timer xua beat 0 +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... + timer xua beat 0 +... +ss7_asp_vty_test(config-cs7-asp)# timer xua beat 5 +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... + timer xua beat 5 +... +ss7_asp_vty_test(config-cs7-asp)# timer xua beat 30 +ss7_asp_vty_test(config-cs7-asp)# show running-config +... + asp my-asp 12345 54321 m3ua +... !timer xua beat 30 +... + ss7_asp_vty_test(config-cs7-asp)# exit % NOTE: Make sure to use '[no] shutdown' command in 'asp' node in order to restart the ASP for new configs to be applied. -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43156?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Iadd7c6751ab09640fd2df214afa2ec87274a3210 Gerrit-Change-Number: 43156 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>
1 0
0 0
[M] Change in osmo-ttcn3-hacks[master]: stp: tcap: Avoid need of N_NOTICE.req
by fixeria 03 Aug '26

03 Aug '26
Attention is currently required from: daniel, laforge, lynxis lazus, pespin. fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?usp=email ) Change subject: stp: tcap: Avoid need of N_NOTICE.req ...................................................................... Patch Set 1: (1 comment) File stp/STP_Tests_TCAP.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152/comment/1de11925_d88d… : PS1, Line 227: SCCP_PAR_Reason_For_Return reasonForReturn unused -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?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: Ia1b7f487124584ab1720cb89bca5999cbb12392d Gerrit-Change-Number: 43152 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 03 Aug 2026 12:18:01 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
1 0
0 0
[M] Change in osmo-ttcn3-hacks[master]: stp: tcap: Avoid need of N_NOTICE.req
by osmith 03 Aug '26

03 Aug '26
Attention is currently required from: daniel, fixeria, laforge, lynxis lazus, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?usp=email ) Change subject: stp: tcap: Avoid need of N_NOTICE.req ...................................................................... Patch Set 1: (1 comment) File stp/STP_Tests_TCAP.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152/comment/00f0c447_d105… : PS1, Line 212: private function f_asp_disconnect_mtp3_port(integer asp_idx := 0) runs on TCAP_CT > Not really needed for the existing test, but since most of the complexity with all this is actually […] Acknowledged -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?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: Ia1b7f487124584ab1720cb89bca5999cbb12392d Gerrit-Change-Number: 43152 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:41:17 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
1 0
0 0
[M] Change in osmo-ttcn3-hacks[master]: stp: tcap: Avoid need of N_NOTICE.req
by pespin 03 Aug '26

03 Aug '26
Attention is currently required from: daniel, fixeria, laforge, lynxis lazus, osmith. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?usp=email ) Change subject: stp: tcap: Avoid need of N_NOTICE.req ...................................................................... Patch Set 1: (1 comment) File stp/STP_Tests_TCAP.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152/comment/6fd5f0f1_b9e4… : PS1, Line 212: private function f_asp_disconnect_mtp3_port(integer asp_idx := 0) runs on TCAP_CT > do you need to call this somewhere? Not really needed for the existing test, but since most of the complexity with all this is actually figureing out where/how to hook stuff in the big set of layers, I think it's a good idea to leave it here in case one needs to write more tests needing to send UDTS and then disconnecting to let the SCCP layer continue without the hook. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?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: Ia1b7f487124584ab1720cb89bca5999cbb12392d Gerrit-Change-Number: 43152 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:40:47 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
1 0
0 0
[XS] Change in osmo-ttcn3-hacks[master]: Revert "deps: use a fork of titan.ProtocolEmulations.SCCP"
by osmith 03 Aug '26

03 Aug '26
Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43155?usp=email ) Change subject: Revert "deps: use a fork of titan.ProtocolEmulations.SCCP" ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43155?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: I4eb81e0ffc6500340199c4a23a7aabfee925c33b Gerrit-Change-Number: 43155 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:29:32 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes
1 0
0 0
[XS] Change in osmo-ttcn3-hacks[master]: Revert "stp: add ASP_SCCP_N_NOTICE_req support and f_asp_tx_notice()"
by osmith 03 Aug '26

03 Aug '26
Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43154?usp=email ) Change subject: Revert "stp: add ASP_SCCP_N_NOTICE_req support and f_asp_tx_notice()" ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43154?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: Iecce8999b72b363a7aa3f583beeb937e79fe215f Gerrit-Change-Number: 43154 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:29:16 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes
1 0
0 0
[S] Change in osmo-ttcn3-hacks[master]: Revert "stp: Add TCAP_N_NOTICE_req to send a UDTS with TCAP payload"
by osmith 03 Aug '26

03 Aug '26
Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43153?usp=email ) Change subject: Revert "stp: Add TCAP_N_NOTICE_req to send a UDTS with TCAP payload" ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43153?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: I4e96d9f0ceaafb2b5883b0e1f2d8d9f6065fd415 Gerrit-Change-Number: 43153 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:29:06 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes
1 0
0 0
[M] Change in osmo-ttcn3-hacks[master]: stp: tcap: Avoid need of N_NOTICE.req
by osmith 03 Aug '26

03 Aug '26
Attention is currently required from: daniel, fixeria, laforge, lynxis lazus, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?usp=email ) Change subject: stp: tcap: Avoid need of N_NOTICE.req ...................................................................... Patch Set 1: Code-Review+1 (1 comment) File stp/STP_Tests_TCAP.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152/comment/bd65f8bc_6e19… : PS1, Line 212: private function f_asp_disconnect_mtp3_port(integer asp_idx := 0) runs on TCAP_CT do you need to call this somewhere? -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43152?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: Ia1b7f487124584ab1720cb89bca5999cbb12392d Gerrit-Change-Number: 43152 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 03 Aug 2026 11:28:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes
1 0
0 0
[XS] Change in osmo-ttcn3-hacks[master]: Revert "deps: use a fork of titan.ProtocolEmulations.SCCP"
by pespin 03 Aug '26

03 Aug '26
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43155?usp=email ) Change subject: Revert "deps: use a fork of titan.ProtocolEmulations.SCCP" ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: @vyanitskiy@sysmocom.de, once this is merged you/we can get rid of the fixeria/udts branch in our titan.ProtocoEmulations.SCCP fork. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43155?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: I4eb81e0ffc6500340199c4a23a7aabfee925c33b Gerrit-Change-Number: 43155 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Comment-Date: Mon, 03 Aug 2026 09:44:18 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
1 0
0 0
  • ← Newer
  • 1
  • ...
  • 221
  • 222
  • 223
  • 224
  • 225
  • 226
  • 227
  • Older →

HyperKitty Powered by HyperKitty version 1.3.12.