Attention is currently required from: jolly, pespin. fixeria has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008?usp=email ) Change subject: Add templates for S1AP Handover procedure ...................................................................... Patch Set 1: (11 comments) File library/s1ap/S1AP_Templates.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/9dd59a67_f798… : PS1, Line 2293: procedureCode := S1AP_Constants.id_HandoverPreparation, It may look confusing, but yes, it's correct. See 3GPP TS 36.413, section 8.4.1 "Handover Preparation". https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/1d9ab67d_fd76… : PS1, Line 2337: procedureCode := S1AP_Constants.id_HandoverPreparation, It may look confusing, but yes, it's correct. See 3GPP TS 36.413, section 8.4.1 "Handover Preparation". https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/e3709bce_b93e… : PS1, Line 2370: } Optional IEs can be added later whenever they're needed. For testing the S1 HANDOVER procedures, it's enough to have mandatory IEs only. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/8ac1fa85_cdbf… : PS1, Line 2374: ts_E_RABSubjecttoDataForwardingList(template (value) E_RABDataForwardingItem it) := { `E-RABSubjecttoDataForwardingItem` does not exist in the ASN.1 definition, so it's not possible to define templates for a single list item. This is how it's defined: ``` E-RABSubjecttoDataForwardingList ::= E-RAB-IE-ContainerList { {E-RABDataForwardingItemIEs} } ``` Same applies to the `E_RABModifyListBearerModConf` templates above. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/e1e49616_bddf… : PS1, Line 2423: successfulOutcome := { Yes, because this PDU is sent in response to the `HANDOVER REQUIRED`. * `HANDOVER REQUIRED` - `initiatingMessage`, * `HANDOVER COMMAND` - `successfulOutcome`. See 3GPP TS 36.413, section 8.4.1 "Handover Preparation". https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/19585d4b_fe0a… : PS1, Line 2424: procedureCode := S1AP_Constants.id_HandoverPreparation, See above (yes). https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/20931929_6431… : PS1, Line 2450: } We definitely want `E-RABs to Release List` IE (optional) here. Not a merge blocker, can be added later in a separate patch. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/5d45d5b8_b6e6… : PS1, Line 2462: procedureCode := S1AP_Constants.id_HandoverPreparation, See above. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/40b75a58_5b55… : PS1, Line 2560: procedureCode := S1AP_Constants.id_HandoverResourceAllocation, Yes. See 3GPP TS 36.413, section 8.4.2 "Handover Resource Allocation". https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/a0577d5d_8cc9… : PS1, Line 2638: *, I see no optional IEs between the `E-RABs To Be Setup List` and `Source to Target Transparent Container`, so a wildcard is not needed here. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008/comment/e3051405_106d… : PS1, Line 2732: }, { Likewise, would be nice to have optional `E-RABs Failed to Setup List` IE here. Not a merge blocker, can be added later in a separate patch. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41008?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: Ib8b8aee0665a4f6644287b802f5e941bd82128f6 Gerrit-Change-Number: 41008 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Sun, 07 Sep 2025 10:23:31 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>