-
Notifications
You must be signed in to change notification settings - Fork 29
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-209: PFCP Deletion Timeout Handling #47
Conversation
@@ -941,6 +943,13 @@ func HandlePDUSessionSMContextRelease(smContextRef string, body models.ReleaseSm | |||
Body: nil, | |||
} | |||
|
|||
case smf_context.SessionReleaseTimeout: | |||
smContext.SubCtxLog.Traceln("PDUSessionSMContextRelease, PFCP SessionReleaseTimeout") | |||
smContext.ChangeState(smf_context.Active) |
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.
why do we move to active state here??
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.
these smf states confuse me too.. i coped from other failure cases.. I should understand these flags from Ankur
retest this please |
bfded32
to
033110d
Compare
retest this please |
033110d
to
93b73fb
Compare
retest this please |
1 similar comment
retest this please |
@@ -322,7 +322,7 @@ func SendPfcpSessionModificationResponse(addr *net.UDPAddr) { | |||
} | |||
|
|||
func SendPfcpSessionDeletionRequest(upNodeID pfcpType.NodeID, ctx *smf_context.SMContext) (seqNum uint32) { | |||
pfcpMsg, err := BuildPfcpSessionDeletionRequest() | |||
pfcpMsg, err := BuildPfcpSessionDeletionRequest(upNodeID, ctx) | |||
if err != nil { | |||
ctx.SubPfcpLog.Errorf("Build PFCP Session Deletion Request failed: %v", err) | |||
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.
Returning on build PFCP Deletion req failure might not help. Probably, we can just discuss this once.
No description provided.