July 2026 - gerrit-log - lists.osmocom.org

[XS] Change in onomondo-ipa[master]: V1.2: Remove error code from ListEimResult
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43064?usp=email ) Change subject: V1.2: Remove error code from ListEimResult ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43064?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I8acb868970a589e70c1688427994854f9393a01b Gerrit-Change-Number: 43064 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 09:39:18 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add eimTransactionId to EuiccPackageErrorUnsigned
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43063?usp=email ) Change subject: V1.2: Add eimTransactionId to EuiccPackageErrorUnsigned ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43063?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: Idfd596c56ef8cfe745f96cbaa6b8a5537927a919 Gerrit-Change-Number: 43063 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 09:29:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[L] Change in onomondo-ipa[master]: V1.2: Add AuthenticateResponseOk and EuiccSigned1
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43061?usp=email ) Change subject: V1.2: Add AuthenticateResponseOk and EuiccSigned1 ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43061?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I501247132de30618a2b1ace44c7c3772ab42eb09 Gerrit-Change-Number: 43061 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 09:16:16 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add function to set Device Capabilities
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email ) Change subject: V1.2: Add function to set Device Capabilities ...................................................................... Patch Set 1: Code-Review+1 (1 comment) File src/ipa/main.c: https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/04720e8d_74ac1a90… : PS1, Line 264: case 'D': I wonder if it would make sense to accept a string like "00001111". A one byte hex number would also be ok. Somewhat unrelated, but helpful when fixed/improved: One thing that would be really helpful would be a converter that converts from ASCII hex digits to binary hex. This would solve a very annoying problem: The eIM generates an eIM configuration and outputs it as ASCII hex digits on the REST API. To use this output we have to convert it to binary using external tools to get the sample_eim_cfg.ber file you find in contrib. When we have a conveter, we could just use a text file with the ASCII hex string or even switch to a commandline parameter. After all, you could then use the converter here to set cfg.device_capabilities from a 2-hex-digit string (If you think this would make sense). -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I866787b8cda9a87a9f2e0db4965fbb554d339a6c Gerrit-Change-Number: 43060 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 09:11:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add new causes to initiateAuthenticationErrorEsipa
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43059?usp=email ) Change subject: V1.2: Add new causes to initiateAuthenticationErrorEsipa ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43059?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I0a9acaba7b706f68bc89bae3dff7d2d0e380e797 Gerrit-Change-Number: 43059 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 08:49:19 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-ipa[master]: V1.2: Change field name in InitiateAuthenticationOkEsipa
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43058?usp=email ) Change subject: V1.2: Change field name in InitiateAuthenticationOkEsipa ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43058?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I68d59b44bd8b5dd23ae070d42dab9727c0da5272 Gerrit-Change-Number: 43058 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 08:47:30 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in onomondo-ipa[master]: V1.2: Add StateChangeCause to GetEimPackageRequest
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43057?usp=email ) Change subject: V1.2: Add StateChangeCause to GetEimPackageRequest ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43057?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I1ae561f6709a9eb490fa7a53420c332ff20bd058 Gerrit-Change-Number: 43057 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 08:46:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add eimTransactionId to InitiateAuthenticationRequestEsipa
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43056?usp=email ) Change subject: V1.2: Add eimTransactionId to InitiateAuthenticationRequestEsipa ...................................................................... Patch Set 1: (3 comments) File src/ipa/libipa/esipa_init_auth.h: https://gerrit.osmocom.org/c/onomondo-ipa/+/43056/comment/e80c9040_f2430c2c… : PS1, Line 19: TransactionId_t *eim_transaction_id; As it seems there is a tab instead of a space between type and member name. The member name is consistent, however in other code locations we use struct OCTET_STRING to declare the type. (here the name of the struct meber is consistent) File src/ipa/libipa/proc_cmn_mtl_auth.h: https://gerrit.osmocom.org/c/onomondo-ipa/+/43056/comment/35c58fc0_6b9ed194… : PS1, Line 19: TransactionId_t *eim_transaction_id; maybe declare this as struct OCTET_STRING *eim_transaction_id? It could also make sense to apply some refactoring, so that we consistently use TransactionId_t throughout the entire codebase. What we should not do is to mix the two type names, even when they are the same under to hood. We may also check if it would work to declare the eim_transaction_id as const n the pars and req structs. File src/ipa/libipa/proc_indirect_prfle_dwnld.h: https://gerrit.osmocom.org/c/onomondo-ipa/+/43056/comment/d7702c23_4a653703… : PS1, Line 18: TransactionId_t *eim_transaction_id; (see the other comments) -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43056?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I1c8ef6fc0a3f7ab5735e401597fc8c6b2021fc09 Gerrit-Change-Number: 43056 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 08:45:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[L] Change in onomondo-ipa[master]: V1.2: Change fields in ProvideEimPackage(Result)
by dexter 23 Jul '26

23 Jul '26

Attention is currently required from: jolly. dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43055?usp=email ) Change subject: V1.2: Change fields in ProvideEimPackage(Result) ...................................................................... Patch Set 1: (1 comment) File src/ipa/libipa/esipa_prvde_eim_pkg_rslt.h: https://gerrit.osmocom.org/c/onomondo-ipa/+/43055/comment/826ae0e4_d15ed434… : PS1, Line 20: struct OCTET_STRING *eim_trans_id; I think its better to call it eim_transaction_id, because in other interfaces we have called it "transaction_id" already. We should have a consistent naming scheme. (see esipa_cancel_session.h for an example) -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43055?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I24baab4f9351aec27c58403b94974cba4b274c34 Gerrit-Change-Number: 43055 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Comment-Date: Thu, 23 Jul 2026 07:54:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in ...osmo_dia2gsup[master]: Set Item-Number in UTRAN and GERAN vectors.
by fixeria 22 Jul '26

22 Jul '26

Attention is currently required from: Q Misell. fixeria has posted comments on this change by Q Misell. ( https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/43092?usp=email ) Change subject: Set Item-Number in UTRAN and GERAN vectors. ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo_dia2gsup/+/43092?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_dia2gsup Gerrit-Branch: master Gerrit-Change-Id: I75da04b806f23452c9b4cfaad0446dceb32a0369 Gerrit-Change-Number: 43092 Gerrit-PatchSet: 2 Gerrit-Owner: Q Misell <q(a)magicalcodewit.ch> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: Q Misell <q(a)magicalcodewit.ch> Gerrit-Comment-Date: Wed, 22 Jul 2026 17:38:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log July 2026