Attention is currently required from: fixeria, laforge, lynxis lazus, osmith, pespin.

osmith uploaded patch set #3 to this change.

View Change

The following approvals got outdated and were removed: Code-Review+1 by fixeria, Code-Review+1 by pespin, Verified+1 by Jenkins Builder

GSUP: PDP Context ID: start at 1

In OsmoHLR, PDP Contexts can have IDs <1-10>. 0 is used internally to
mark a profile slot as disabled:
https://gitea.osmocom.org/cellular-infrastructure/osmo-hlr/src/commit/d4b4548589f929d7c888ad213bd601ab2a421d8a/src/hlr_vty.c#L201

HLR Test TC_gsup_ul_subscriber_data in osmo-ttcn3-hacks also makes this
assumption:
https://gitea.osmocom.org/ttcn3/osmo-ttcn3-hacks/src/commit/85276b94c474bb649a0c4893b760c7b76f1f342e/hlr/HLR_Tests.ttcn#L1165-L1172

However the new GSUP implementation in PyHSS started its IDs at 0. I've
suggested to start it at 1 here:
https://github.com/nickvsnetworking/pyhss/pull/277/files/4b6da744743877806810b6eade937d2360eb0fbf#r2545727324

Change-Id: I9c39002cb7f7dd81ae0f1f0f9d80df1aaf399f48
---
M common/chapters/gsup.adoc
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-manuals refs/changes/71/41471/3

To view, visit change 41471. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I9c39002cb7f7dd81ae0f1f0f9d80df1aaf399f48
Gerrit-Change-Number: 41471
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-Attention: osmith <osmith@sysmocom.de>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: pespin <pespin@sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis@fe80.eu>