Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SMF] Gy: Check Multiple-Services-Credit-Control Result-Code in CCA-I #3111

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

pespin
Copy link
Contributor

@pespin pespin commented Apr 5, 2024

This is the continuation of commit
12158ee, which only checked the code in CCA[Update], but not in CCA[Initial].

The handling in CCA[Initial] is a bit more complex since depending on the outcome, we may end up with a Result-Code != SUCCESS in MSCC but the session may still be created at the OCS because the message Result-Code = SUCCESS. In that scenario, we want to abort setting up the PDN session but we still need to make sure we terminate the Gy session that was just created.

This is the continuation of commit
12158ee, which only checked the code in
CCA[Update], but not in CCA[Initial].

The handling in CCA[Initial] is a bit more complex since depending on
the outcome, we may end up with a Result-Code != SUCCESS in MSCC but the
session may still be created at the OCS because the message Result-Code
= SUCCESS. In that scenario, we want to abort setting up the PDN session
but we still need to make sure we terminate the Gy session that was just
created.
@acetcom acetcom merged commit 990bfe9 into open5gs:main Apr 5, 2024
5 checks passed
@pespin pespin deleted the pespin/gy-sys6838 branch April 5, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants