X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=pkg%2Fcontrol%2Fregistry.go;h=5e16c0aadf0f894cd8296dec37788068feb34e28;hb=aada64566a3a77cf9a20a98f9ddd7cd6f37529ae;hp=25709605a29c1a56c8d5d333abc19af599eba112;hpb=7348625b9ef03d41dd5a0ca0f6c508376259717e;p=ric-plt%2Fsubmgr.git

diff --git a/pkg/control/registry.go b/pkg/control/registry.go
index 2570960..5e16c0a 100644
--- a/pkg/control/registry.go
+++ b/pkg/control/registry.go
@@ -55,7 +55,7 @@ func (r *Registry) QueryHandler() (models.SubscriptionList, error) {
 	resp := models.SubscriptionList{}
 	for _, subs := range r.register {
 		subs.mutex.Lock()
-		resp = append(resp, &models.SubscriptionData{SubscriptionID: int64(subs.ReqId.Seq), Meid: subs.Meid.RanName, Endpoint: subs.EpList.StringList()})
+		resp = append(resp, &models.SubscriptionData{SubscriptionID: int64(subs.ReqId.InstanceId), Meid: subs.Meid.RanName, Endpoint: subs.EpList.StringList()})
 		subs.mutex.Unlock()
 	}
 	return resp, nil
@@ -63,10 +63,10 @@ func (r *Registry) QueryHandler() (models.SubscriptionList, error) {
 
 func (r *Registry) allocateSubs(trans *TransactionXapp, subReqMsg *e2ap.E2APSubscriptionRequest) (*Subscription, error) {
 	if len(r.subIds) > 0 {
-		sequenceNumber := r.subIds[0]
+		subId := r.subIds[0]
 		r.subIds = r.subIds[1:]
-		if _, ok := r.register[sequenceNumber]; ok == true {
-			r.subIds = append(r.subIds, sequenceNumber)
+		if _, ok := r.register[subId]; ok == true {
+			r.subIds = append(r.subIds, subId)
 			return nil, fmt.Errorf("Registry: Failed to reserve subscription exists")
 		}
 		subs := &Subscription{
@@ -76,10 +76,10 @@ func (r *Registry) allocateSubs(trans *TransactionXapp, subReqMsg *e2ap.E2APSubs
 			valid:     true,
 		}
 		subs.ReqId.Id = 123
-		subs.ReqId.Seq = sequenceNumber
+		subs.ReqId.InstanceId = subId
 
 		if subs.EpList.AddEndpoint(trans.GetEndpoint()) == false {
-			r.subIds = append(r.subIds, subs.ReqId.Seq)
+			r.subIds = append(r.subIds, subs.ReqId.InstanceId)
 			return nil, fmt.Errorf("Registry: Endpoint existing already in subscription")
 		}
 
@@ -169,23 +169,23 @@ func (r *Registry) AssignToSubscription(trans *TransactionXapp, subReqMsg *e2ap.
 	// Subscription route updates
 	//
 	if epamount == 1 {
-		subRouteAction := SubRouteInfo{subs.EpList, uint16(subs.ReqId.Seq)}
+		subRouteAction := SubRouteInfo{subs.EpList, uint16(subs.ReqId.InstanceId)}
 		err = r.rtmgrClient.SubscriptionRequestCreate(subRouteAction)
 	} else {
-		subRouteAction := SubRouteInfo{subs.EpList, uint16(subs.ReqId.Seq)}
+		subRouteAction := SubRouteInfo{subs.EpList, uint16(subs.ReqId.InstanceId)}
 		err = r.rtmgrClient.SubscriptionRequestUpdate(subRouteAction)
 	}
 	r.mutex.Lock()
 
 	if err != nil {
 		if newAlloc {
-			r.subIds = append(r.subIds, subs.ReqId.Seq)
+			r.subIds = append(r.subIds, subs.ReqId.InstanceId)
 		}
 		return nil, err
 	}
 
 	if newAlloc {
-		r.register[subs.ReqId.Seq] = subs
+		r.register[subs.ReqId.InstanceId] = subs
 	}
 	xapp.Logger.Debug("CREATE %s", subs.String())
 	xapp.Logger.Debug("Registry: substable=%v", r.register)
@@ -226,7 +226,7 @@ func (r *Registry) RemoveFromSubscription(subs *Subscription, trans *Transaction
 
 	delStatus := subs.EpList.DelEndpoint(trans.GetEndpoint())
 	epamount := subs.EpList.Size()
-	seqId := subs.ReqId.Seq
+	subId := subs.ReqId.InstanceId
 
 	if delStatus == false {
 		return nil
@@ -247,7 +247,7 @@ func (r *Registry) RemoveFromSubscription(subs *Subscription, trans *Transaction
 			//
 			tmpList := xapptweaks.RmrEndpointList{}
 			tmpList.AddEndpoint(trans.GetEndpoint())
-			subRouteAction := SubRouteInfo{tmpList, uint16(seqId)}
+			subRouteAction := SubRouteInfo{tmpList, uint16(subId)}
 			r.rtmgrClient.SubscriptionRequestDelete(subRouteAction)
 
 			//
@@ -256,18 +256,18 @@ func (r *Registry) RemoveFromSubscription(subs *Subscription, trans *Transaction
 			r.mutex.Lock()
 			defer r.mutex.Unlock()
 
-			if _, ok := r.register[seqId]; ok {
+			if _, ok := r.register[subId]; ok {
 				xapp.Logger.Debug("RELEASE %s", subs.String())
-				delete(r.register, seqId)
+				delete(r.register, subId)
 				xapp.Logger.Debug("Registry: substable=%v", r.register)
 			}
-			r.subIds = append(r.subIds, seqId)
+			r.subIds = append(r.subIds, subId)
 
 		} else if subs.EpList.Size() > 0 {
 			//
 			// Subscription route updates
 			//
-			subRouteAction := SubRouteInfo{subs.EpList, uint16(seqId)}
+			subRouteAction := SubRouteInfo{subs.EpList, uint16(subId)}
 			r.rtmgrClient.SubscriptionRequestUpdate(subRouteAction)
 		}
 
@@ -276,22 +276,22 @@ func (r *Registry) RemoveFromSubscription(subs *Subscription, trans *Transaction
 	return nil
 }
 
-func (r *Registry) GetSubscription(sn uint32) *Subscription {
+func (r *Registry) GetSubscription(subId uint32) *Subscription {
 	r.mutex.Lock()
 	defer r.mutex.Unlock()
-	if _, ok := r.register[sn]; ok {
-		return r.register[sn]
+	if _, ok := r.register[subId]; ok {
+		return r.register[subId]
 	}
 	return nil
 }
 
-func (r *Registry) GetSubscriptionFirstMatch(ids []uint32) (*Subscription, error) {
+func (r *Registry) GetSubscriptionFirstMatch(subIds []uint32) (*Subscription, error) {
 	r.mutex.Lock()
 	defer r.mutex.Unlock()
-	for _, id := range ids {
-		if _, ok := r.register[id]; ok {
-			return r.register[id], nil
+	for _, subId := range subIds {
+		if _, ok := r.register[subId]; ok {
+			return r.register[subId], nil
 		}
 	}
-	return nil, fmt.Errorf("No valid subscription found with ids %v", ids)
+	return nil, fmt.Errorf("No valid subscription found with subIds %v", subIds)
 }