Attention is currently required from: neels, pespin, daniel, lynxis lazus. keith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/16808 )
Change subject: Add vty `reject-cause` to set the reject cause ...................................................................... Patch Set 11: (1 comment) Patchset: PS11: > and I'm not against keeping the functionality clean, and I'm all for > questioning the need for patche […] This also would not be the first time that a feature that is entirely (even it it were so) investigation/testing based was included in mainstream osmo I wonder what does a commercial HLR look like, I mean there are quite some LU reject causes. osmo-hlr uses ONE of them, hard coded. Maybe it's one of those things in the spec where options kept getting added, but really there's only one or two in use. Probably not the HLRs decision anyway? When would an operator decide between #12 and #15 for example? For me it's quite hard to see form the spec what the intention is as to have the MS behaviour differs between them. The difference between "forbidden location areas for regional provision of service" and "forbidden location areas for roaming" seems subtle, as does the difference between "shall perform a cell selection" and "shall search for a suitable cell in another location area" - given that the LAI was added to the forbidden list. Maybe #15 is never intended for use when a SIM attempts to connect to it's HOME network. I guess that kind of thing could be automatic in the code; Is the IMSI "ours"? -> then #12 else #15 -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/16808 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Icea39020c23fbbea9e92847df76af8986fdbf48a Gerrit-Change-Number: 16808 Gerrit-PatchSet: 11 Gerrit-Owner: lynxis lazus <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <[email protected]> Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-Reviewer: neels <[email protected]> Gerrit-CC: keith <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: neels <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Attention: daniel <[email protected]> Gerrit-Attention: lynxis lazus <[email protected]> Gerrit-Comment-Date: Thu, 19 Jan 2023 16:52:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: keith <[email protected]> Gerrit-MessageType: comment
