Attention is currently required from: lynxis lazus.
pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40650?usp=email )
Change subject: GSUP Templates: fix empty destination_name in GSUP ISD/UL
......................................................................
Patch Set 3: Code-Review-1
(1 comment)
File library/GSUP_Templates.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40650/comment/a4a8791b_b346… :
PS2, Line 421: template GSUP_PDU ts_GSUP_UL_RES(hexstring imsi, template octetstring destination_name := omit) :=
> Done
You submitted a new version, marked as DONE but I see no change here yet?
Please try to go through all pending things before resubmitting, otherwise I need to check N times instead of 1.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40650?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: Iab4d1f6211e4f6351ad56246aa1d934c5fc34912
Gerrit-Change-Number: 40650
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:21:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: daniel, lynxis lazus.
pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40649?usp=email )
Change subject: SGSN: Iu: use correct service request type
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40649?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: Ia47edd8ca916cf377da875583a3c4eb6ff5f1f52
Gerrit-Change-Number: 40649
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:19:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: lynxis lazus.
pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/40641?usp=email )
Change subject: gtp: SGSN Ctx: prevent a stack reference to be in **ie
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Can't we change gtpie_encaps3() to expect a const pointer instead? or array of const pointers or whatever.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/40641?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Ideca8beb21c6cce7104721b4d80854448baf6c4e
Gerrit-Change-Number: 40641
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: daniel <dwillmann(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:19:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: pespin.
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40651?usp=email )
Change subject: BSSGP_ConnHdlr: f_service_request: allow to define if a Service Accept is expect
......................................................................
Patch Set 4:
(1 comment)
File sgsn/BSSGP_ConnHdlr.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40651/comment/67fb5a2f_cddc… :
PS2, Line 760: boolean expect_service_acc := true,
> so you are adding a boolean arg which is never used?
Oh. Sorry, this was a rebase accident.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40651?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: I5c190db28263c530a8eea188bf570a58eaffdd4d
Gerrit-Change-Number: 40651
Gerrit-PatchSet: 4
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:13:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40651?usp=email )
Change subject: BSSGP_ConnHdlr: f_service_request: allow to define if a Service Accept is expect
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4:
Oh. Sorry, this was a rebase accident.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40651?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: I5c190db28263c530a8eea188bf570a58eaffdd4d
Gerrit-Change-Number: 40651
Gerrit-PatchSet: 4
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:13:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: daniel.
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40649?usp=email )
Change subject: SGSN: Iu: use correct service request type
......................................................................
Patch Set 3:
(1 comment)
File sgsn/SGSN_Tests_Iu.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40649/comment/24bc008f_51f4… :
PS3, Line 213: SERVICE_TYPE_Signalling
> Why not use the default value and continue to omit if it's Signalling?
to make it more clear.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40649?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: Ia47edd8ca916cf377da875583a3c4eb6ff5f1f52
Gerrit-Change-Number: 40649
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 14 Jul 2025 16:12:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: daniel <dwillmann(a)sysmocom.de>