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

[M] Change in onomondo-eim[master]: restop.py, tryme_*.sh: improve log output and handling
by dexter 15 Jul '26

15 Jul '26

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email ) Change subject: restop.py, tryme_*.sh: improve log output and handling ...................................................................... restop.py, tryme_*.sh: improve log output and handling The tryme scripts and the restop.py script produce lots of distracting and difficult to read log output. Let's renovate the scripts a bit so that the useful information is displayed in a readable way. Change-Id: I7fe40e7ed50354114e80cd96a5b40f3e579220e9 Related SYS#8100 Change-Id: Id257e7a541fc92386cecee5af3dad2077b62709c --- M contrib/restop.py M contrib/tryme_addEim.sh M contrib/tryme_configureImmediateEnable.sh M contrib/tryme_delete.sh M contrib/tryme_deleteEim.sh M contrib/tryme_disable.sh M contrib/tryme_download.sh M contrib/tryme_enable.sh M contrib/tryme_euiccDataRequest.sh M contrib/tryme_euicc_get_state.sh M contrib/tryme_euicc_set_state.sh M contrib/tryme_getRAT.sh M contrib/tryme_listEim.sh M contrib/tryme_listProfileInfo.sh M contrib/tryme_lookup.sh M contrib/tryme_updateEim.sh 16 files changed, 38 insertions(+), 39 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/14/43014/1 diff --git a/contrib/restop.py b/contrib/restop.py index 47c1953..9b43c89 100755 --- a/contrib/restop.py +++ b/contrib/restop.py @@ -8,8 +8,8 @@ import argparse import json import requests -import erlang import pprint +import textwrap DOWNLOAD_DEFAULT='{ "eidValue" : "89882119900000000000000000000005", "order" : {"activationCode" : "1$testsmdpplus1.example.com$OPxLD-UVRuC-jysPI-YkOwT"}}' PSMO_DEFAULT='{ "eidValue" : "89882119900000000000000000000005", "order" : [{"psmo" : "enable", "iccid" : "98001032547698103285", "rollback" : false }]}' @@ -37,53 +37,66 @@ def main(argv): parser = argparse.ArgumentParser(prog='restop', description='utility to operate on the REST API of onomondo_eim') parser.add_argument("-s", "--host", default="127.0.0.1:8080") - parser.add_argument("-f", "--facility", default="download") - parser.add_argument("-c", "--create", action='store_true', default=False) - parser.add_argument("-l", "--lookup", action='store_true', default=False) - parser.add_argument("-d", "--delete", action='store_true', default=False) - parser.add_argument("-t", "--list", action='store_true', default=False) - parser.add_argument("-r", "--resource-id") - parser.add_argument("-j", "--json") + parser.add_argument("-f", "--facility", help="REST API facility", default="download") + parser.add_argument("-c", "--create", help="creata a new REST resource", action='store_true') + parser.add_argument("-l", "--lookup", help="lookup an existing REST resource", action='store_true') + parser.add_argument("-d", "--delete", help="delete a no longer needed REST resource", action='store_true') + parser.add_argument("-t", "--list", help="list all currently existing REST resources", action='store_true') + parser.add_argument("-r", "--resource-id", help="REST resource identifier") + parser.add_argument("-j", "--json", help="JSON input") + parser.add_argument("-e", "--erlang-debuginfo", help="decode and display erlang debug information", action='store_true') args = parser.parse_args() + pp = pprint.PrettyPrinter() if args.create: print("create on: " + str(args.host), file=sys.stderr) print(" facility: " + str(args.facility), file=sys.stderr) if args.json: - args_json = str(args.json) + args_json = json.loads(str(args.json)) else: if args.facility == "download": - args_json = DOWNLOAD_DEFAULT + args_json = json.loads(DOWNLOAD_DEFAULT) elif args.facility == "psmo": - args_json = PSMO_DEFAULT - print(" json: " + args_json, file=sys.stderr) - print("result:", file=sys.stderr) - print(rest_create(args.host, args.facility, json.loads(args_json))) + args_json = json.loads(PSMO_DEFAULT) + args_json_pretty = pp.pformat(args_json) + print(" json: " + textwrap.indent(args_json_pretty, " " * 7)[7:], file=sys.stderr) + resource_url = rest_create(args.host, args.facility, args_json) + print(" resourceId: " + resource_url.rsplit('/')[-1], file=sys.stderr) + print(resource_url) + elif args.lookup: print("lookup on: " + str(args.host), file=sys.stderr) print(" facility: " + str(args.facility), file=sys.stderr) print(" resourceId: " + str(args.resource_id), file=sys.stderr) - result = rest_lookup(args.host, args.facility, args.resource_id) - print(json.dumps(result)) - if 'debuginfo' in result: - debuginfo_hexstr=result['debuginfo'] + result_json = rest_lookup(args.host, args.facility, args.resource_id) + result_json_pretty = pp.pformat(result_json) + print(" json: " + textwrap.indent(result_json_pretty, " " * 7)[7:], file=sys.stderr) + if args.erlang_debuginfo and 'debuginfo' in result_json: + import erlang + debuginfo_hexstr=result_json['debuginfo'] debuginfo_bytes=h2b(debuginfo_hexstr) debuginfo_term=erlang.binary_to_term(debuginfo_bytes) - pp = pprint.PrettyPrinter(depth=4) debuginfo_pretty = pp.pformat(debuginfo_term) - print("decoded debuginfo: " + str(debuginfo_pretty), file=sys.stderr) + print(" debuginfo: " + textwrap.indent(debuginfo_pretty, " " * 12)[12:], file=sys.stderr) + print(json.dumps(result_json)) + elif args.delete: print("delete on: " + str(args.host), file=sys.stderr) print(" facility: " + str(args.facility), file=sys.stderr) print(" resourceId: " + str(args.resource_id), file=sys.stderr) - print("result:", file=sys.stderr) - print(json.dumps(rest_delete(args.host, args.facility, args.resource_id))) + result_json = rest_delete(args.host, args.facility, args.resource_id) + result_json_pretty = pp.pformat(result_json) + print(" json: " + textwrap.indent(result_json_pretty, " " * 7)[7:], file=sys.stderr) + print(json.dumps(result_json)) + elif args.list: print("list on: " + str(args.host), file=sys.stderr) print(" facility: " + str(args.facility), file=sys.stderr) - print("result:", file=sys.stderr) - print(json.dumps(rest_list(args.host, args.facility)), file=sys.stderr) + result_json = rest_list(args.host, args.facility) + result_json_pretty = pp.pformat(result_json) + print(" json: " + textwrap.indent(result_json_pretty, " " * 7)[7:], file=sys.stderr) + print(json.dumps(result_json)) if __name__ == "__main__": main(sys.argv[1:]) diff --git a/contrib/tryme_addEim.sh b/contrib/tryme_addEim.sh index a504010..9bdee03 100755 --- a/contrib/tryme_addEim.sh +++ b/contrib/tryme_addEim.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "eco" : [ { "addEim" : { "eimConfigurationData" : "3079800465494D32810E3132372E302E302E313A383030308301018401FFA55BA059301306072A8648CE3D020106082A8648CE3D03010703420004FE584A6F450459574AECA195D0299737F74C89BA2D36DF9286EC25D973037A0FBA70D14DF3E1F7D0A305E57B95B731C4DE218D2D7F9F22113ED5D18C2E3DDF1C" } } ] } }' RC=`./restop.py -c -f eco -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_configureImmediateEnable.sh b/contrib/tryme_configureImmediateEnable.sh index 9cf4fde..1d8233c 100755 --- a/contrib/tryme_configureImmediateEnable.sh +++ b/contrib/tryme_configureImmediateEnable.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo": [ { "configureImmediateEnable" : { "immediateEnableFlag" : true, "defaultSmdpAddress" : "testsmdpplus1.example.com" } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_delete.sh b/contrib/tryme_delete.sh index bf1a9b2..b220838 100755 --- a/contrib/tryme_delete.sh +++ b/contrib/tryme_delete.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo" : [ { "delete" : { "iccid" : "'$ICCID'" } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_deleteEim.sh b/contrib/tryme_deleteEim.sh index 8faf657..439e6fe 100755 --- a/contrib/tryme_deleteEim.sh +++ b/contrib/tryme_deleteEim.sh @@ -3,7 +3,6 @@ EIM_ID="eIM2" JSON='{ "eidValue" : "'$EID'", "order" : { "eco" : [ { "deleteEim" : { "eimId" : "'$EIM_ID'" } } ] } }' RC=`./restop.py -c -f eco -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_disable.sh b/contrib/tryme_disable.sh index a45af94..f0d5de4 100755 --- a/contrib/tryme_disable.sh +++ b/contrib/tryme_disable.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo" : [ { "disable" : { "iccid" : "'$ICCID'" } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_download.sh b/contrib/tryme_download.sh index 10725a2..09e6eb2 100755 --- a/contrib/tryme_download.sh +++ b/contrib/tryme_download.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "download" : {"activationCode" : "'$AC'" } } }' RC=`./restop.py -c -f download -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_enable.sh b/contrib/tryme_enable.sh index ac08706..30a8c7f 100755 --- a/contrib/tryme_enable.sh +++ b/contrib/tryme_enable.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo": [ { "enable" : { "iccid" : "'$ICCID'", "rollback" : false } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_euiccDataRequest.sh b/contrib/tryme_euiccDataRequest.sh index ba70a69..20d2fad 100755 --- a/contrib/tryme_euiccDataRequest.sh +++ b/contrib/tryme_euiccDataRequest.sh @@ -2,7 +2,6 @@ . ./tryme.cfg JSON='{ "eidValue" : "'$EID'", "order" : { "edr" : { "tagList" : "81BF20BF228384A5A6A8A9A0" } } }' RC=`./restop.py -c -f edr -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_euicc_get_state.sh b/contrib/tryme_euicc_get_state.sh index dc96cf0..e7d8e7b 100755 --- a/contrib/tryme_euicc_get_state.sh +++ b/contrib/tryme_euicc_get_state.sh @@ -6,7 +6,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "euicc" : { "get" : [ '$STATES' ] } } }' RC=`./restop.py -c -f euicc -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_euicc_set_state.sh b/contrib/tryme_euicc_set_state.sh index 04d8c51..6911007 100755 --- a/contrib/tryme_euicc_set_state.sh +++ b/contrib/tryme_euicc_set_state.sh @@ -11,7 +11,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "euicc" : { "set" : '$STATES' } } }' RC=`./restop.py -c -f euicc -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_getRAT.sh b/contrib/tryme_getRAT.sh index fa61d51..e1a0160 100755 --- a/contrib/tryme_getRAT.sh +++ b/contrib/tryme_getRAT.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo" : [ { "getRAT" : { } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_listEim.sh b/contrib/tryme_listEim.sh index 0d86787..47eb9e7 100755 --- a/contrib/tryme_listEim.sh +++ b/contrib/tryme_listEim.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "eco" : [ { "listEim" : { } } ] } }' RC=`./restop.py -c -f eco -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_listProfileInfo.sh b/contrib/tryme_listProfileInfo.sh index 7d64b98..47209f7 100755 --- a/contrib/tryme_listProfileInfo.sh +++ b/contrib/tryme_listProfileInfo.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "psmo" : [ { "listProfileInfo" : { } } ] } }' RC=`./restop.py -c -f psmo -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` diff --git a/contrib/tryme_lookup.sh b/contrib/tryme_lookup.sh index df8e129..5e49bb6 100755 --- a/contrib/tryme_lookup.sh +++ b/contrib/tryme_lookup.sh @@ -3,7 +3,7 @@ if [[ $# == 2 ]]; then sleep 5 while true; do - ./restop.py -l -f $1 -r $2 + ./restop.py -l -f $1 -r $2 > /dev/null sleep 5 done else diff --git a/contrib/tryme_updateEim.sh b/contrib/tryme_updateEim.sh index 5d08398..faf34aa 100755 --- a/contrib/tryme_updateEim.sh +++ b/contrib/tryme_updateEim.sh @@ -3,7 +3,6 @@ JSON='{ "eidValue" : "'$EID'", "order" : { "eco" : [ { "updateEim" : { "eimConfigurationData" : "3017800465494D32810F3132372E302E302E34323A39303030" } } ] } }' RC=`./restop.py -c -f eco -j "$JSON"` -echo $RC echo "---------------------------------------8<---------------------------------------" RESOURCE_ID=`echo $RC | cut -d '/' -f 6` -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Id257e7a541fc92386cecee5af3dad2077b62709c Gerrit-Change-Number: 43014 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[XS] Change in onomondo-eim[master]: mnesia_db_rest: add comment on why we create an euicc entry early
by dexter 15 Jul '26

15 Jul '26

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42991?usp=email to look at the new patch set (#3). Change subject: mnesia_db_rest: add comment on why we create an euicc entry early ...................................................................... mnesia_db_rest: add comment on why we create an euicc entry early Even though it is technically not necessary, we ensure that an entry is present in the euicc table on each REST API call. Let's add a comment to explain why we do it anyway. Change-Id: Ifb9340655c5f36e074a1a92c0e234ef62c79b9d8 Related: SYS#8100 --- M src/mnesia_db_rest.erl 1 file changed, 6 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/91/42991/3 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42991?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ifb9340655c5f36e074a1a92c0e234ef62c79b9d8 Gerrit-Change-Number: 42991 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 0

[S] Change in onomondo-eim[master]: mnesia_db_euicc: rename timer_setparam to timer_rest
by dexter 15 Jul '26

15 Jul '26

Attention is currently required from: laforge. Hello Jenkins Builder, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42960?usp=email to look at the new patch set (#4). Change subject: mnesia_db_euicc: rename timer_setparam to timer_rest ...................................................................... mnesia_db_euicc: rename timer_setparam to timer_rest The function timer_setparam is processing REST API orders in regular intervals. Naming the timer "timer_rest" makes more sense. Change-Id: Icfa203477e044ce2cc312d4b6e2693a97191c61d Related: SYS#8100 --- M src/mnesia_db.erl M src/mnesia_db_euicc.erl 2 files changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/60/42960/4 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42960?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Icfa203477e044ce2cc312d4b6e2693a97191c61d Gerrit-Change-Number: 42960 Gerrit-PatchSet: 4 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org>

1 0

[XS] Change in onomondo-eim[master]: contrib/tryme_euiccDataRequest: fixup example tag string
by dexter 15 Jul '26

15 Jul '26

Attention is currently required from: jolly. Hello Jenkins Builder, jolly, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42992?usp=email to look at the new patch set (#3). Change subject: contrib/tryme_euiccDataRequest: fixup example tag string ...................................................................... contrib/tryme_euiccDataRequest: fixup example tag string The eUICC data request takes a string with TLV tags as input. The example in this example still uses tags from SGP.32 V.1.0.1, let's update the string so that it is compliant with SGP.32 V.1.2. (see also SGP.32, section 2.11.1.2) Change-Id: I5be04c954330638d3e7d78e8f5f280bebb707efa Related: SYS:8100 --- M contrib/tryme_euiccDataRequest.sh 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/92/42992/3 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42992?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I5be04c954330638d3e7d78e8f5f280bebb707efa Gerrit-Change-Number: 42992 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: jolly <andreas(a)eversberg.eu> Gerrit-Attention: jolly <andreas(a)eversberg.eu>

1 0

[L] Change in onomondo-eim[master]: mnesia_db_euicc: renovate handling of euicc table (mnesia, REST)
by dexter 15 Jul '26

15 Jul '26

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: mnesia_db_euicc: renovate handling of euicc table (mnesia, REST) ...................................................................... mnesia_db_euicc: renovate handling of euicc table (mnesia, REST) The REST API that allows users to set certain eUICC parameters (we call them "states" from now on) only allows to set eUICC states, but doesn't allow to read them back. With this patch, we renovate the handling of the mnesia euicc table from the ground up so that REST API users have read and write access to the states stored in the mnesia euicc table. Change-Id: I74c602555b194a28d6eb9dd67ec4c6a8865fcb77 Related: SYS#8100 --- M contrib/rest_api_resource_schema.json M contrib/rest_api_response_schema.json A contrib/tryme_euicc_get_state.sh A contrib/tryme_euicc_set_state.sh D contrib/tryme_set_euicc_param.sh M doc/database.md M doc/examples.md M doc/rest_api.md M src/crypto_utils.erl M src/esipa_asn1_handler_utils.erl M src/mnesia_db.erl M src/mnesia_db_euicc.erl 12 files changed, 315 insertions(+), 175 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/93/42993/3 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I74c602555b194a28d6eb9dd67ec4c6a8865fcb77 Gerrit-Change-Number: 42993 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 0

[XS] Change in onomondo-eim[master]: doc/build.md: fix typo
by dexter 15 Jul '26

15 Jul '26

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: doc/build.md: fix typo ...................................................................... doc/build.md: fix typo Change-Id: Ie6a066f3289fbdde40f55bd9f38a3dd90102e7e6 Related: SYS#8100 --- M doc/build.md 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/94/42994/3 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ie6a066f3289fbdde40f55bd9f38a3dd90102e7e6 Gerrit-Change-Number: 42994 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 0

[S] Change in osmo-ttcn3-hacks[master]: SGP32Definitions_Templates: rename tr/ts_getEimPackageResponse_eimPkg...
by dexter 15 Jul '26

15 Jul '26

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43011?usp=email ) Change subject: SGP32Definitions_Templates: rename tr/ts_getEimPackageResponse_eimPkgErrUndef ...................................................................... SGP32Definitions_Templates: rename tr/ts_getEimPackageResponse_eimPkgErrUndef The template name suffix _eimPkgErrUndef is a bit unflexible. Let's change it to eimPkgErr and add a parameter erroCode with a default value. This allows us to use the template with any eimPackageError. Change-Id: I81b7c3b5ebc8c95d8ea8f1a6030b884ed2efcff3 Related: SYS#8100 --- M ipad/IPAd_Tests.ttcn M library/euicc/SGP32Definitions_Templates.ttcn 2 files changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/43011/1 diff --git a/ipad/IPAd_Tests.ttcn b/ipad/IPAd_Tests.ttcn index cf7ff5e..c999cfb 100644 --- a/ipad/IPAd_Tests.ttcn +++ b/ipad/IPAd_Tests.ttcn @@ -656,7 +656,7 @@ f_http_register(); /* IPAd requests a package, we respond with an eimPackageError code 127 (undefined error) */ - f_esipa_transceive(valueof(ts_getEimPackageResponse_eimPkgErrUndef), tr_getEimPackageRequest); + f_esipa_transceive(valueof(ts_getEimPackageResponse_eimPkgErr(127)), tr_getEimPackageRequest); /* Wait some time until the the last HTTP response is actually delivered */ f_sleep(2.0); diff --git a/library/euicc/SGP32Definitions_Templates.ttcn b/library/euicc/SGP32Definitions_Templates.ttcn index c83dc8e..13110ab 100644 --- a/library/euicc/SGP32Definitions_Templates.ttcn +++ b/library/euicc/SGP32Definitions_Templates.ttcn @@ -333,15 +333,15 @@ /* GSMA SGP.32, section 6.3.2.6 */ template (present) EsipaMessageFromEimToIpa -tr_getEimPackageResponse_eimPkgErrUndef := { +tr_getEimPackageResponse_eimPkgErr(template (present) integer errorCode := ?) := { getEimPackageResponse := { - eimPackageError := ? + eimPackageError := errorCode } } template (value) EsipaMessageFromEimToIpa -ts_getEimPackageResponse_eimPkgErrUndef := { +ts_getEimPackageResponse_eimPkgErr(template (value) integer errorCode := 127) := { getEimPackageResponse := { - eimPackageError := 127 + eimPackageError := errorCode } } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43011?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I81b7c3b5ebc8c95d8ea8f1a6030b884ed2efcff3 Gerrit-Change-Number: 43011 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[M] Change in osmo-ttcn3-hacks[master]: eIM_Tests: add testcase to test stateChangeCause/stateChangeCause
by dexter 15 Jul '26

15 Jul '26

dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43012?usp=email ) Change subject: eIM_Tests: add testcase to test stateChangeCause/stateChangeCause ...................................................................... eIM_Tests: add testcase to test stateChangeCause/stateChangeCause The IPAd may pass a stateChangeCause with the getEimPackageRequest. The stateChangeCause is then recorded by the eIM and can be queried via the REST API. This patch adds a testcase to verify that functionality. Related: SYS#8100 Change-Id: I6befc43f2dfd2178e352b9ab0a24a04d6e116fbb --- M eim/REST_Types_JSON.ttcn M eim/eIM_Tests.ttcn M library/euicc/SGP32Definitions.asn M library/euicc/SGP32Definitions_Templates.ttcn 4 files changed, 94 insertions(+), 11 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/12/43012/1 diff --git a/eim/REST_Types_JSON.ttcn b/eim/REST_Types_JSON.ttcn index b82bea0..0a34939 100644 --- a/eim/REST_Types_JSON.ttcn +++ b/eim/REST_Types_JSON.ttcn @@ -74,14 +74,24 @@ octetstring tagList }; -type union JSON_REST_Euicc { + +type union JSON_REST_Euicc_State { integer counterValue, boolean consumerEuicc, integer associationToken, octetstring signAlgo, - octetstring signPubKey + octetstring signPubKey, + record of charstring stateChangeCauseList }; +type union JSON_REST_Euicc { + record of JSON_REST_Euicc_State set_, + record of charstring get +} with { + variant(set_) "JSON:name as set"; +}; + + type record JSON_REST_Resource { charstring eidValue, union { @@ -89,7 +99,7 @@ record of JSON_REST_Psmo psmo, record of JSON_REST_Eco eco, JSON_REST_Edr edr, - record of JSON_REST_Euicc euicc + JSON_REST_Euicc euicc } order }; @@ -160,7 +170,7 @@ charstring finalResult, charstring iccid optional } profileInstallationResult, - charstring euiccUpdateResult, + record of JSON_REST_Euicc_State euiccStateResult, record { charstring edrResult, record { @@ -178,11 +188,13 @@ } euiccDataResult } +type record of JSON_REST_Result JSON_REST_Results; + type record JSON_REST_Response_long { charstring status, charstring timestamp, JSON_REST_Resource resource, - record of JSON_REST_Result outcome, + JSON_REST_Results outcome, charstring debuginfo }; @@ -219,12 +231,12 @@ } -template (present) JSON_REST_Response tr_JSON_REST_success := { +template (present) JSON_REST_Response tr_JSON_REST_success(template JSON_REST_Results outcome := ?) := { long := { status := "done", timestamp := ?, resource := ?, - outcome := ?, + outcome := outcome, debuginfo := ? } } @@ -317,6 +329,16 @@ order := { edr := { tagList := tagList } } } +template (value) JSON_REST_Resource ts_JSON_REST_euicc_set_order(octetstring eidValue, JSON_REST_Euicc_State setValue) := { + eidValue := oct2str(eidValue), + order := { euicc := { set_ := { setValue } } } +} + +template (value) JSON_REST_Resource ts_JSON_REST_euicc_get_order(octetstring eidValue, charstring stateName) := { + eidValue := oct2str(eidValue), + order := { euicc := { get := { stateName } } } +} + } with { encode "JSON"; } diff --git a/eim/eIM_Tests.ttcn b/eim/eIM_Tests.ttcn index 5ed9176..8728082 100644 --- a/eim/eIM_Tests.ttcn +++ b/eim/eIM_Tests.ttcn @@ -297,7 +297,7 @@ } } - if (not istemplatekind(expected_rest_res, "omit") and not match(rest_res, tr_JSON_REST_success)) { + if (not istemplatekind(expected_rest_res, "omit") and not match(rest_res, expected_rest_res)) { setverdict(fail, "unexpected REST lookup result from eIM"); } @@ -892,6 +892,59 @@ setverdict(pass); } +/* (see also above) Test enable PSMO */ +private function f_TC_proc_notifyStateChange(charstring id) runs on eIM_ConnHdlr { + var charstring resource_id; + var HTTP_Adapter_Params http_adapter_pars; + var template EsipaMessageFromEimToIpa esipa_res; + var JSON_REST_Euicc_State euiccState; + var JSON_REST_Result outcome; + + f_http_register(); + + f_init_pipe(); + http_adapter_pars := { + http_host := mp_esipa_ip, + http_port := mp_esipa_port, + use_ssl := not mp_esipa_disable_ssl + }; + f_http_init(http_adapter_pars); + + /* Ensure that the stateChangeCauseList at the eIM is empty, for that we set the stateChangeCauseList to an + * empty list. */ + euiccState.stateChangeCauseList := {}; + outcome.euiccStateResult := { euiccState }; + resource_id := f_rest_create_order("euicc", ts_JSON_REST_euicc_set_order(eID, euiccState)); + f_sleep(1.5); + f_rest_lookup_resource(resource_id, "euicc", tr_JSON_REST_success({outcome})); + f_rest_delete_resource(resource_id, "euicc"); + + /* Check the eIM for new eIM packages and set notifyStateChange/stateChangeCause. Since we haven't issued any + * orders we expect an eimPackageError (noEimPackageAvailable) */ + esipa_res := f_esipa_transceive(valueof(ts_getEimPackageRequest(eID, notifyStateChange := NULL, stateChangeCause := 6)), + tr_getEimPackageResponse_eimPkgErr(1)); + + /* After the getEimPackageRequest, we expect to find the stateChangeCause in the stateChangeCauseList */ + euiccState.stateChangeCauseList := { "deviceChange" }; + outcome.euiccStateResult := { euiccState }; + resource_id := f_rest_create_order("euicc", ts_JSON_REST_euicc_get_order(eID, "stateChangeCauseList")); + f_sleep(1.5); + f_rest_lookup_resource(resource_id, "euicc", tr_JSON_REST_success({outcome})); + f_rest_delete_resource(resource_id, "euicc"); + + f_sleep(2.0); + setverdict(pass); +} +testcase TC_proc_notifyStateChange() runs on MTC_CT { + var charstring id := testcasename(); + var eIM_ConnHdlrPars pars := f_init_pars(); + var eIM_ConnHdlr vc_conn; + f_init(id); + vc_conn := f_start_handler(refers(f_TC_proc_notifyStateChange), pars); + vc_conn.done; + setverdict(pass); +} + control { execute ( TC_proc_indirect_prfle_dwnld() ); execute ( TC_proc_euicc_pkg_dwnld_exec_enable_psmo() ); @@ -910,6 +963,7 @@ execute ( TC_proc_euicc_pkg_dwnld_exec_listEim_eco() ); execute ( TC_proc_euicc_data_req() ); execute ( TC_proc_euicc_data_req_err() ); + execute ( TC_proc_notifyStateChange() ); } } diff --git a/library/euicc/SGP32Definitions.asn b/library/euicc/SGP32Definitions.asn index 69c3bf3..93b31b2 100644 --- a/library/euicc/SGP32Definitions.asn +++ b/library/euicc/SGP32Definitions.asn @@ -8,6 +8,11 @@ IMPORTS Certificate, SubjectPublicKeyInfo FROM PKIX1Explicit88 {iso(1) identified-organization(3) dod(6) internet(1) security(5) mechanisms(5) pkix(7) id-mod(0) id-pkix1-explicit(18)} SubjectKeyIdentifier FROM PKIX1Implicit88 {iso(1) identified-organization(3) dod(6) internet(1) security(5) mechanisms(5) pkix(7) id-mod(0) id-pkix1-implicit(19)} CancelSessionReason, ServerSigned1, RspCapability, Iccid, TransactionId, ProfileInfoListRequest, RulesAuthorisationTable, EUICCInfo1, DeviceInfo, VersionType, UICCCapability, PprIds, CertificationDataObject, Octet1, Octet16, Octet32, PrepareDownloadResponseOk, PrepareDownloadResponseError, AuthenticateResponseError, CtxParams1, ProfileInstallationResultData, OtherSignedNotification, EuiccSignPIR, ErrorResult, NotificationMetadata, CancelSessionResponseOk, SmdpSigned2, BoundProfilePackage, NotificationEvent, VendorSpecificExtension, DpProprietaryData, OperatorId, NotificationConfigurationInformation, ProfileClass, IconType, ProfileState, OctetTo16 FROM RSPDefinitions {joint-iso-itu-t(2) international-organizations(23) gsma(146) rsp(1) asn1modules(1) sgp22v2(2)}; + +-- workaround for TTCN3, define a usable "AsnNull" type since "NULL" cannot be used as a type when defining template +-- parameters. +AsnNull ::= NULL + EuiccPackageRequest ::= [81] SEQUENCE { -- Tag ' BF51', #SupportedForPsmoV1.0.0# euiccPackageSigned EuiccPackageSigned, eimSignature [APPLICATION 55] OCTET STRING -- Tag '5F37' diff --git a/library/euicc/SGP32Definitions_Templates.ttcn b/library/euicc/SGP32Definitions_Templates.ttcn index 13110ab..bdc5fba 100644 --- a/library/euicc/SGP32Definitions_Templates.ttcn +++ b/library/euicc/SGP32Definitions_Templates.ttcn @@ -272,11 +272,13 @@ } } template (value) EsipaMessageFromIpaToEim -ts_getEimPackageRequest(template (value) octetstring eidValue) := { +ts_getEimPackageRequest(template (value) octetstring eidValue, + template (omit) AsnNull notifyStateChange := omit, + template (omit) integer stateChangeCause := omit) := { getEimPackageRequest := { eidValue := eidValue, - notifyStateChange := omit, - stateChangeCause := omit, + notifyStateChange := notifyStateChange, + stateChangeCause := stateChangeCause, rPLMN := omit } } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43012?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I6befc43f2dfd2178e352b9ab0a24a04d6e116fbb Gerrit-Change-Number: 43012 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de>

1 0

[M] Change in osmo-hlr[master]: systemd: run db-upgrade script in ExecStartPre
by fixeria 15 Jul '26

15 Jul '26

Attention is currently required from: dexter, jolly, osmith. fixeria has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-hlr/+/43010?usp=email ) Change subject: systemd: run db-upgrade script in ExecStartPre ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/43010?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: If13f4c8b49397e6630956e92a7825843c7d55ebd Gerrit-Change-Number: 43010 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: jolly <andreas(a)eversberg.eu> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Wed, 15 Jul 2026 09:57:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in osmo-hlr[master]: systemd: run db-upgrade script in ExecStartPre
by osmith 15 Jul '26

15 Jul '26

Attention is currently required from: dexter, fixeria, jolly. Hello Jenkins Builder, dexter, fixeria, jolly, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-hlr/+/43010?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: systemd: run db-upgrade script in ExecStartPre ...................................................................... systemd: run db-upgrade script in ExecStartPre We have seen in CI jobs that calling the DB upgrade script from debian/postinst does not work reliably. When doing a fresh install of osmo-hlr, it usually works as expected: Setting up osmo-hlr (1.9.2) ... osmo-hlr-post-upgrade: nothing to do (no existing database) But sometimes: Setting up osmo-hlr (1.9.2) ... osmo-hlr-post-upgrade: database upgrade is required osmo-hlr-post-upgrade: stopping osmo-hlr service osmo-hlr-post-upgrade: creating backup: /var/lib/osmocom/hlr.db.20260706135658.bak osmo-hlr-post-upgrade: performing database upgrade <0001> db.c:600 using database: /var/lib/osmocom/hlr.db <0001> db.c:103 (283) recovered 4 frames from WAL file /var/lib/osmocom/hlr.db-wal <0001> db.c:679 Database '/var/lib/osmocom/hlr.db' has HLR DB schema version 0 <0001> db.c:103 (1) duplicate column name: last_lu_seen in "ALTER TABLE subscriber ADD COLUMN last_lu_seen TIMESTAMP default NULL" <0001> db.c:248 Unable to prepare SQL statement 'ALTER TABLE subscriber ADD COLUMN last_lu_seen TIMESTAMP default NULL' <0001> db.c:318 Unable to update HLR database schema to version 1 <0001> db.c:685 Failed to upgrade HLR DB schema to version 1: (rc=1) duplicate column name: last_lu_seen <0000> hlr_db_tool.c:509 Error opening database It seems that osmo-hlr can get started before the osmo-hlr-post-upgrade script runs, then starts to create the database with the current schema, and gets stopped before it is done, leading to the failure above. Rename the script to osmo-hlr-db-upgrade.sh, remove logic for stopping and starting the service and let systemd call it in ExecStartPre. That way we are sure that it runs *before* osmo-hlr starts up. Related: SYS#8125 Change-Id: If13f4c8b49397e6630956e92a7825843c7d55ebd --- M contrib/Makefile.am R contrib/osmo-hlr-db-upgrade.sh M contrib/systemd/osmo-hlr.service M debian/osmo-hlr.install M debian/postinst 5 files changed, 8 insertions(+), 43 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/10/43010/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/43010?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: If13f4c8b49397e6630956e92a7825843c7d55ebd Gerrit-Change-Number: 43010 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: jolly <andreas(a)eversberg.eu> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de>

1 0

2026

2025

2024

2023

2022

gerrit-log July 2026