pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36444?usp=email )
Change subject: Diameter Gy CCR: Expect RATType AVP in PS-Information instead of MSCC ......................................................................
Diameter Gy CCR: Expect RATType AVP in PS-Information instead of MSCC
open5gs-smfd was recently modified to send RAT-Type inside PS-Information instead of MSCC, since having it in the former is supported since older spec releases and having it in the later creates problems in some real world OCS implementation like PortaOne OCS.
Related: SYS#6837 Related: open5gs.git d0b31177cca360865ebc6ab0b89eee7ee4fc8d1a, Related: open5gs.git 3b5e851f5d1328536052031e66a7b9b03c3057f6 Change-Id: I7ce77d08847a0876291f76e901e5c89c339db27d --- M ggsn_tests/GGSN_Tests.ttcn M pgw/PGW_Tests.ttcn 2 files changed, 32 insertions(+), 15 deletions(-)
Approvals: osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
diff --git a/ggsn_tests/GGSN_Tests.ttcn b/ggsn_tests/GGSN_Tests.ttcn index 13edca5..e2d5fb2 100644 --- a/ggsn_tests/GGSN_Tests.ttcn +++ b/ggsn_tests/GGSN_Tests.ttcn @@ -550,8 +550,7 @@ tr_AVP_MultipleServicesIndicator, tr_AVP_Multiple_Services_Credit_Control(content := superset( tr_AVP_Requested_Service_Unit, - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -567,6 +566,7 @@ tr_AVP_3GPP_NSAPI(nsapi), tr_AVP_3GPP_MS_TimeZone, tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type), tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV), tr_AVP_UserEquipmentInfoValue(imeisv) @@ -596,8 +596,7 @@ tr_AVP_Requested_Service_Unit, tr_AVP_Used_Service_Unit, /* tr_AVP_3GPP_Reporting_Reason, can be sometimes inside UsedServiceUnit */ - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -613,6 +612,7 @@ tr_AVP_3GPP_NSAPI(nsapi), tr_AVP_3GPP_MS_TimeZone, tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type), tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV), tr_AVP_UserEquipmentInfoValue(imeisv) @@ -643,8 +643,7 @@ /* tr_AVP_Requested_Service_Unit, Only in INIT and UPDATE */ tr_AVP_Used_Service_Unit, tr_AVP_3GPP_Reporting_Reason(FINAL), - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -660,6 +659,7 @@ tr_AVP_3GPP_NSAPI(nsapi), tr_AVP_3GPP_MS_TimeZone, tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type), tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV), tr_AVP_UserEquipmentInfoValue(imeisv) diff --git a/pgw/PGW_Tests.ttcn b/pgw/PGW_Tests.ttcn index 3c4bef2..a1850a0 100644 --- a/pgw/PGW_Tests.ttcn +++ b/pgw/PGW_Tests.ttcn @@ -457,8 +457,7 @@ tr_AVP_MultipleServicesIndicator, tr_AVP_Multiple_Services_Credit_Control(content := superset( tr_AVP_Requested_Service_Unit, - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -474,7 +473,8 @@ tr_AVP_3GPP_NSAPI(nsapi), /*We don't yet send MS_Tz in CreateSessionReq: tr_AVP_3GPP_MS_TimeZone,*/ - tr_AVP_3GPP_ULI/*, + tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type)/*, We don't yet send IMEI in CreateSessionReq: tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV), @@ -505,8 +505,7 @@ tr_AVP_Requested_Service_Unit, tr_AVP_Used_Service_Unit, /* tr_AVP_3GPP_Reporting_Reason, can be sometimes inside UsedServiceUnit */ - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -522,7 +521,8 @@ tr_AVP_3GPP_NSAPI(nsapi), /*We don't yet send MS_Tz in CreateSessionReq: tr_AVP_3GPP_MS_TimeZone,*/ - tr_AVP_3GPP_ULI/*, + tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type)/*, We don't yet send IMEI in CreateSessionReq: tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV), @@ -554,8 +554,7 @@ /* tr_AVP_Requested_Service_Unit, Only in INIT and UPDATE */ tr_AVP_Used_Service_Unit, tr_AVP_3GPP_Reporting_Reason(FINAL), - tr_AVP_PCC_3GPP_QoS_Information, - tr_AVP_GI_3GPP_RatType(rat_type) + tr_AVP_PCC_3GPP_QoS_Information )), tr_AVP_3GPP_ServiceInformation(content := superset( tr_AVP_3GPP_PSInformation(content := superset( @@ -571,7 +570,8 @@ tr_AVP_3GPP_NSAPI(nsapi), /*We don't yet send MS_Tz in CreateSessionReq: tr_AVP_3GPP_MS_TimeZone,*/ - tr_AVP_3GPP_ULI/*, + tr_AVP_3GPP_ULI, + tr_AVP_GI_3GPP_RatType(rat_type)/*, We don't yet send IMEI in CreateSessionReq: tr_AVP_UserEquipmentInfo({ tr_AVP_UserEquipmentInfoType(IMEISV),