Add support for SDM Subscription and Unsubscription #95

saileshvvr · 2024-02-04T15:37:44Z

Add support for SDM Subscription during UE 1st PDU Session Establishment
Add support for SDM UnSubscription during release of last UE PDU Session
Code refactor to handle all SDM processing (SDM_Get and Subscription/Unsubscription) under consumer package

andy89923 · 2024-03-06T05:53:05Z

Hi!
Thanks for the PR. We tested and found out that there is unexpected behavior occurred. I'm finding out the error, so I am marking this PR as a draft.
The error occurred when using UERANSIM to test the whole 5gc functionalities.

saileshvvr · 2024-03-06T05:58:56Z

Hi,
Thanks for sharing. Sure please.
Request to share the test scenario or logs or elaborate the details, it helps me too for finding the issue.
Earlier while adding the support for UECM registration i encountered crash with UDM due to missing pointer deference. I fixed the UDM issue separately while SMF PR initiated.

andy89923 · 2024-03-06T06:10:42Z

Hi,
I created a subscriber and tried to establish PDU sessions uses UERANSIM, the errors as follow:

2024-03-06T05:44:58.678704422Z [ERRO][SMF][GIN] panic: runtime error: invalid memory address or nil pointer dereference
goroutine 93 [running]:
runtime/debug.Stack()
        /usr/local/go/src/runtime/debug/stack.go:24 +0x65
github.com/free5gc/util/logger.ginRecover.func1.1()
        /home/ctfang/go/pkg/mod/github.com/free5gc/[email protected]/logger/logger.go:284 +0x1a5
panic({0xb53480, 0x11c3170})
        /usr/local/go/src/runtime/panic.go:838 +0x207
github.com/free5gc/smf/internal/sbi/producer.updateAnUpfPfcpSession(0xc0005b2000, {0xc0000564d0, 0x2, 0x2}, {0xc0000564e0, 0x2, 0x2}, {0xc0000bd390, 0x0, 0x0}, ...)
        /home/ctfang/free5gc-dev/NFs/smf/internal/sbi/producer/datapath.go:368 +0xd9
github.com/free5gc/smf/internal/sbi/producer.HandlePDUSessionSMContextUpdate({0xc00068e380, 0x2d}, {0xc000782680, {0x0, 0x0, 0x0}, {0xc00067a800, 0xf, 0x3e8}})
        /home/ctfang/free5gc-dev/NFs/smf/internal/sbi/producer/pdu_session.go:768 +0x36fa
github.com/free5gc/smf/internal/sbi/pdusession.HTTPUpdateSmContext(0xc000162000)
        /home/ctfang/free5gc-dev/NFs/smf/internal/sbi/pdusession/api_individual_sm_context.go:83 +0x3bb
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/ctfang/go/pkg/mod/github.com/gin-gonic/[email protected]/context.go:174
github.com/free5gc/util/logger.ginRecover.func1(0xc000162000)
        /home/ctfang/go/pkg/mod/github.com/free5gc/[email protected]/logger/logger.go:316 +0x7b
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/ctfang/go/pkg/mod/github.com/gin-gonic/[email protected]/context.go:174
github.com/free5gc/util/logger.ginToLogrus.func1(0xc000162000)
        /home/ctfang/go/pkg/mod/github.com/free5gc/[email protected]/logger/logger.go:251 +0x87
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/ctfang/go/pkg/mod/github.com/gin-gonic/[email protected]/context.go:174
github.com/gin-gonic/gin.(*Engine).handleHTTPRequest(0xc000498ea0, 0xc000162000)
        /home/ctfang/go/pkg/mod/github.com/gin-gonic/[email protected]/gin.go:620 +0x671
github.com/gin-gonic/gin.(*Engine).ServeHTTP(0xc000498ea0, {0xda3370?, 0xc00068c020}, 0xc0006a2400)
        /home/ctfang/go/pkg/mod/github.com/gin-gonic/[email protected]/gin.go:576 +0x1dd
golang.org/x/net/http2.(*serverConn).runHandler(0xc000056370?, 0xc00049c7d0?, 0x900666?, 0xc000083ba0?)
        /home/ctfang/go/pkg/mod/golang.org/x/[email protected]/http2/server.go:2368 +0xc2
created by golang.org/x/net/http2.(*serverConn).scheduleHandler
        /home/ctfang/go/pkg/mod/golang.org/x/[email protected]/http2/server.go:2303 +0x23b
2024-03-06T05:44:58.678955069Z [INFO][SMF][GIN] | 500 |       127.0.0.1 | POST    | /nsmf-pdusession/v1/sm-contexts/urn:uuid:7bc2f7cd-37e7-412b-8696-2fb115514b78/modify |

saileshvvr · 2024-03-06T06:14:01Z

Thank you for sharing. I shall also check to analyze the issue. Thanks again :)

saileshvvr · 2024-04-06T13:39:32Z

Hi,

The merge issue was resolved in the latest patch. Remove duplicated code that is causing the crash.

Thanks
Sailesh

saileshvvr · 2024-04-06T13:40:29Z

Sorry, wrongly clicked close with commit button. Reopening PR. Thanks!

andy89923 · 2024-04-06T14:48:37Z

Thanks, I will try this PR next week.

ming-hsien · 2024-04-13T15:40:35Z

Looks good for me !

saileshvvr · 2024-04-15T05:14:21Z

Thanks @ming-hsien .
Hi @andy89923 , please let me know in case if anything missed/needed to be checked from my end. Thanks!

andy89923 · 2024-04-17T07:45:55Z

internal/sbi/producer/sm_common.go

@@ -16,6 +16,26 @@ func RemoveSMContextFromAllNF(smContext *smf_context.SMContext, sendNotification
 		}
 	}

+	problemDetails, err := consumer.SDMUnSubscribe(smContext)


This was also called in some HandleFunction in pdu_session.go, leading to re-UnSubscribe.

I used PacketRusher termination to send Deregistration and found the result.

andy89923 · 2024-04-17T07:50:53Z

Also, I found out that there are some problems in UDM when sending RegistrationSmfRegistrations.
I checked the code and found out that the implementation of the parsing request in UDM may not comply with openapi.

saileshvvr · 2024-04-24T16:06:09Z

Can the UDM complying with openapi be handled seperately

andy89923 · 2024-04-25T14:19:58Z

Can the UDM complying with openapi be handled seperately

Sure.

saileshvvr · 2024-05-01T17:19:45Z

Could you please share the steps used with Packet rusher termination for recreating the issue.

andy89923 · 2024-05-02T02:55:44Z

Could you please share the steps used with Packet rusher termination for recreating the issue.

This article shows how to use PacketRusher. When using CRTL+c to terminate UE, packet rusher would send dereg.
https://free5gc.org/blog/20240110/20240110/

saileshvvr · 2024-06-08T07:25:04Z

Hi @andy89923 , sorry for the delay as i was occupied in other work. I looked the issue today and identified the problem.

The issue is due to the return statement in line #162 in subscriber_data_management.go. Due to this the DeleteUe() is not triggered.

I will fix the issue and upload latest patch.

Thanks!

andy89923 · 2024-06-08T07:57:24Z

Hi, thanks for your contribution.

Some informations to let you know:

The commit message should follow Conventional commit message (you can use git rebase and reword to modify commit message)
SMF is under refactoring, so this PR would be review after the refactoring, you could see refactor: Refactor SMF consumer processor service #106.

saileshvvr · 2024-06-08T10:33:25Z

could you please share the commands so that i shall not make any mistake. Thanks!

saileshvvr · 2024-06-08T11:44:22Z

I did rebase and followed ""Conventional commit message" in latest commit. However it looks like due to old commit message the commit check failure encountered.
Thanks!

andy89923 · 2024-06-08T12:45:24Z

could you please share the commands so that i shall not make any mistake. Thanks!

git rebase -i

And change "pick" -> "reword", you can revise the commit message.

fix lint error fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import

Bumps google.golang.org/protobuf from 1.30.0 to 1.33.0. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

saileshvvr · 2024-06-08T13:42:55Z

could you please share the commands so that i shall not make any mistake. Thanks!
git rebase -i 
And change "pick" -> "reword", you can revise the commit message.

I followed the recommended. However encountering issue due to multiple old commits. Thanks!

andy89923 · 2024-06-08T13:49:48Z

If the PR is ready, just let me know. I can help you fix the commit message.

saileshvvr · 2024-06-08T13:53:36Z

Hi PR is ready. issues encountered due to rebase and commit messages. Thanks!

andy89923 · 2024-06-08T14:09:17Z

Ok!
After the refactor PR is merged, I think this PR would be more conflicts to fix.
We could discuss and fix then.

andy89923 · 2024-09-12T04:23:46Z

Hi, @saileshvvr

This PR need rebase and drop not necessary commits since it had conflict with main branch.
Another suggestion is using cherry pick to pick the commit you want and create another pull request.
Thanks.

saileshvvr · 2024-09-12T06:39:51Z

Hi @andy89923 .
Thanks for the update.
I'll do it during this weekend.

Thanks
Sailesh

andy89923 · 2024-09-12T06:43:01Z

Hi @andy89923 . Thanks for the update. I'll do it during this weekend.

Thanks Sailesh

Thanks for your contribution~
Take you time.

saileshvvr · 2024-09-14T05:57:37Z

Hi @andy89923 ,
It looks like free5gc.org is down and not accessible from yesterday.
Could you please share/attach if any document available having the steps for VM creation, configurations, builds, unit tests etc.
Thanks

andy89923 · 2024-09-14T06:11:20Z

Hi @andy89923 ,

It looks like free5gc.org is down and not accessible from yesterday.

Could you please share/attach if any document available having the steps for VM creation, configurations, builds, unit tests etc.

Thanks

@saileshvvr
You can view all the documents here.

Thanks for reporting; I am looking for where is the problem.

saileshvvr · 2024-09-14T16:43:03Z

Hi @andy89923 ,
I added the code changes in the new PR #123.

Thanks
Sailesh

andy89923 · 2024-09-17T13:55:00Z

@saileshvvr
Thanks for creating a new PR!
Move to #123

andy89923 force-pushed the sdm_subscription branch from c382a7e to 14c5885 Compare February 20, 2024 06:15

andy89923 force-pushed the sdm_subscription branch from 14c5885 to 2a2ec51 Compare March 6, 2024 05:39

andy89923 marked this pull request as draft March 6, 2024 05:47

saileshvvr closed this Apr 6, 2024

saileshvvr reopened this Apr 6, 2024

andy89923 self-requested a review April 6, 2024 14:48

andy89923 marked this pull request as ready for review April 17, 2024 05:54

andy89923 reviewed Apr 17, 2024

View reviewed changes

andy89923 marked this pull request as draft May 2, 2024 02:52

saileshvvr force-pushed the sdm_subscription branch 2 times, most recently from c63b92c to ad3979f Compare June 8, 2024 13:16

saileshvvr and others added 13 commits June 8, 2024 18:51

Fix: Fixed ci-lint error

81e8754

fix lint error fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import fix golangci-lint error for package import

Fix: add OAuth in UDM service request

de06064

Fix: fix linter error

bf66461

Fix: Add CHF ConvergeCharging with OAuth2 Token

b602628

update go.sum

8d38f12

mv dl/up teid generator to smCtx (free5gc#98)

a725ba9

update pkg hash

751c64b

Fix: merge conflict issue. Removed duplicated code

f29bef7

Fix: Addressed the issue during SMF UDMUnSubscription for a PDU Session

255b48e

Fix: SDM Unsubscribe issue

4ec88b2

Fix: Addressed the issue during SMF UDMUnSubscription for a PDU Session

ebee94c

Fix: rebased latest repo

216e322

saileshvvr force-pushed the sdm_subscription branch from ad3979f to 216e322 Compare June 8, 2024 13:27

Merge branch 'main' into sdm_subscription

17836af

saileshvvr added 2 commits June 8, 2024 19:20

Fix: Update sm_context.go

7621131

Fix: resolve merge conflict

5418c70

saileshvvr force-pushed the sdm_subscription branch from d8a8c3a to 5418c70 Compare June 8, 2024 13:51

andy89923 closed this Sep 17, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add support for SDM Subscription and Unsubscription #95

Add support for SDM Subscription and Unsubscription #95

saileshvvr commented Feb 4, 2024

andy89923 commented Mar 6, 2024

saileshvvr commented Mar 6, 2024

andy89923 commented Mar 6, 2024

saileshvvr commented Mar 6, 2024

saileshvvr commented Apr 6, 2024

saileshvvr commented Apr 6, 2024

andy89923 commented Apr 6, 2024

ming-hsien commented Apr 13, 2024

saileshvvr commented Apr 15, 2024

andy89923 Apr 17, 2024

andy89923 Apr 17, 2024

andy89923 commented Apr 17, 2024

saileshvvr commented Apr 24, 2024

andy89923 commented Apr 25, 2024

saileshvvr commented May 1, 2024

andy89923 commented May 2, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

andy89923 commented Sep 12, 2024 •

edited

Loading

saileshvvr commented Sep 12, 2024

andy89923 commented Sep 12, 2024

saileshvvr commented Sep 14, 2024 •

edited

Loading

andy89923 commented Sep 14, 2024

saileshvvr commented Sep 14, 2024

andy89923 commented Sep 17, 2024

Add support for SDM Subscription and Unsubscription #95

Add support for SDM Subscription and Unsubscription #95

Conversation

saileshvvr commented Feb 4, 2024

andy89923 commented Mar 6, 2024

saileshvvr commented Mar 6, 2024

andy89923 commented Mar 6, 2024

saileshvvr commented Mar 6, 2024

saileshvvr commented Apr 6, 2024

saileshvvr commented Apr 6, 2024

andy89923 commented Apr 6, 2024

ming-hsien commented Apr 13, 2024

saileshvvr commented Apr 15, 2024

andy89923 Apr 17, 2024

Choose a reason for hiding this comment

andy89923 Apr 17, 2024

Choose a reason for hiding this comment

andy89923 commented Apr 17, 2024

saileshvvr commented Apr 24, 2024

andy89923 commented Apr 25, 2024

saileshvvr commented May 1, 2024

andy89923 commented May 2, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

saileshvvr commented Jun 8, 2024

andy89923 commented Jun 8, 2024

andy89923 commented Sep 12, 2024 • edited Loading

saileshvvr commented Sep 12, 2024

andy89923 commented Sep 12, 2024

saileshvvr commented Sep 14, 2024 • edited Loading

andy89923 commented Sep 14, 2024

saileshvvr commented Sep 14, 2024

andy89923 commented Sep 17, 2024

andy89923 commented Sep 12, 2024 •

edited

Loading

saileshvvr commented Sep 14, 2024 •

edited

Loading