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
SDCORE-842: Adding Support for UE Requested PDU Session Release Procedure #43
Conversation
gajjarvini
commented
Mar 17, 2022
•
edited
edited
common/procedures.go
Outdated
AN_RELEASE_PROCEDURE: "AN-RELEASE-PROCEDURE", | ||
UE_TRIGGERED_SERVICE_REQUEST_PROCEDURE: "UE-TRIGGERED-SERVICE-REQUEST-PROCEDURE", | ||
NW_TRIGGERED_UE_DEREGISTRATION_PROCEDURE: "NW-TRIGGERED-UE-DEREGISTRATION-PROCEDURE", | ||
AMF_RELEASE_PROCEDURE: "AMF_RELEASE_PROCEDURE", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please update "AMF_RELEASE_PROCEDURE" as well ? instead of _ , use - ?
common/procedures.go
Outdated
} | ||
|
||
func (id ProcedureType) String() string { | ||
procStr, ok := procStrMap[id] | ||
if !ok { | ||
logger.AppLog.Fatalln("Invalid Procedure ID:", id) | ||
logger.AppLog.Fatalf("Invaid Procedure ID: %#v", id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spell of Invalid.
common/events.go
Outdated
} | ||
|
||
func (id EventType) String() string { | ||
evtStr, ok := evtStrMap[id] | ||
if !ok { | ||
logger.AppLog.Fatalln("Invaid Event ID:", id) | ||
logger.AppLog.Fatalf("Invaid Event ID: %#v", id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spell of invalid.
@@ -164,7 +164,7 @@ func (cpTprt *GnbCpTransport) ReceiveFromPeer(peer transportcommon.TransportPeer | |||
} | |||
} | |||
|
|||
cpTprt.Log.Infof("Read %v bytes from %v\n", n, conn) | |||
cpTprt.Log.Infof("Read %v bytes from %v\n", n, amf.GetIpAddr()) | |||
//TODO Post to gnbamfworker channel | |||
gnbamfworker.HandleMessage(cpTprt.GnbInstance, amf, recvMsg[:n]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's understand this more. If no task created for this then we should create one.
return | ||
} | ||
if gnb == nil { | ||
amf.Log.Errorln("GNodeB Message is nil") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
message is nil ? or context is nil ?
amf.Log.Errorln("RANUENGAPID is nil") | ||
return | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we break if you find one occurrence of RANUENGAPID ?
…dure 1) Generating and processing NGAP/NAS message for the procedure 2) Terminating PDU Session Contexts in Real UE and gNodeB component
149a534
to
6dd23ca
Compare