Attention is currently required from: laforge.

lynxis lazus has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/32512 )

Change subject: Add support for multiple APN profiles for subscriber data
......................................................................


Patch Set 5:

(1 comment)

File src/hlr_vty.c:

https://gerrit.osmocom.org/c/osmo-hlr/+/32512/comment/867e78dd_cf23be1f
PS5, Line 133: Define a PDP profile set.\n"
             :       "Unique identifier for this PDP profile set.\n")
> I think the description strings don't fit the default profile enabled flag, 
> but they were copied fro […]
I wasn't sure what a complex scenario could be.
In some future we want to have subscriber specific profiles. On the other end 
most of the subscriber will use a common pdp-profile. The idea here is to allow 
multiple common pdp-profiles (called pdp profile sets) in the future.

Do you have an idea how to improve the help?



--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/32512
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I540132ee5dcfd09f4816e02e702927e1074ca50f
Gerrit-Change-Number: 32512
Gerrit-PatchSet: 5
Gerrit-Owner: lynxis lazus <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Wed, 10 May 2023 07:58:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment

Reply via email to