pespin submitted this change.

View Change


Approvals: osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
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(-)

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),

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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I7ce77d08847a0876291f76e901e5c89c339db27d
Gerrit-Change-Number: 36444
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged