-
Notifications
You must be signed in to change notification settings - Fork 36
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
NfDown Notification Handling #93
NfDown Notification Handling #93
Conversation
vthiruveedula
commented
Jan 9, 2023
- refer to the latest util/drsm 1.0.9.
- NfDownNotification handling
e984bb4
to
6178ead
Compare
oam/routers.go
Outdated
@@ -100,4 +102,10 @@ var routes = Routes{ | |||
"/active-ues", | |||
HTTPGetActiveUes, | |||
}, | |||
{ | |||
"Amf Instance Down Notificatication", |
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.
typo : *Notification
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.
done
@@ -41,7 +43,9 @@ func InitAmfContext(context *context.AMFContext) { | |||
config := factory.AmfConfig | |||
logger.UtilLog.Infof("amfconfig Info: Version[%s] Description[%s]", config.Info.Version, config.Info.Description) | |||
configuration := config.Configuration | |||
context.NfId = uuid.New().String() | |||
if context.NfId == "" { |
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.
what is the case when NfId wont be null ??
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.
Network Instance Id will be empty when the amf comes up first, so first-time NfId should be generated, later we should use the same NfId.
logger.ConsumerLog.Errorf("Remove NF Subscription Failed Problem[%+v]", problemDetails) | ||
} else if err != nil { | ||
logger.ConsumerLog.Errorf("Remove NF Subscription Error[%+v]", err) | ||
if notificationData.Event == models.NotificationEventType_DEREGISTERED { |
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 need this check. will the amf instance down reach here with some other event type ?
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.
there are two events currently, PROFILE_Changed or DEREGISTRED, so entry from the cache must be deleted only in case of DEREGISTERED
a9f7b59
to
2138c7b
Compare
2138c7b
to
9c9d6ef
Compare
retest please |
1. Handling NF Down Notification from NRF 2. refer new util version 1.0.9(nfinstanceid added in keepalive and chunk table) 3. removing subscription from NRF when the target NF instance is down
9c9d6ef
to
9fe3b94
Compare
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.
+1