Merge "Fix for Submgr crash added - Submgr crashed if it received REST Subscription...
authorAnssi Mannila <anssi.mannila@nokia.com>
Thu, 14 Oct 2021 10:20:55 +0000 (10:20 +0000)
committerGerrit Code Review <gerrit@o-ran-sc.org>
Thu, 14 Oct 2021 10:20:55 +0000 (10:20 +0000)
1  2 
pkg/control/sdl_restSubsDb.go

@@@ -27,8 -27,6 +27,8 @@@ import 
        "gerrit.o-ran-sc.org/r/ric-plt/xapp-frame/pkg/xapp"
  )
  
 +const restSubSdlNs = "submgr_restSubsDb"
 +
  type RESTSubscriptionInfo struct {
        XAppRmrEndPoint  string
        Meid             string
@@@ -40,7 -38,7 +40,7 @@@
  }
  
  func CreateRESTSdl() Sdlnterface {
 -      return sdl.NewSdlInstance("submgr_restSubsDb", sdl.NewDatabase())
 +      return sdl.NewSyncStorage()
  }
  
  func (c *Control) WriteRESTSubscriptionToSdl(restSubId string, restSubs *RESTSubscription) error {
@@@ -59,7 -57,7 +59,7 @@@
                return fmt.Errorf("SDL: WriteSubscriptionToSdl() json.Marshal error: %s", err.Error())
        }
  
 -      if err = c.restSubsDb.Set(restSubId, jsonData); err != nil {
 +      if err = c.restSubsDb.Set(restSubSdlNs, restSubId, jsonData); err != nil {
                c.UpdateCounter(cSDLWriteFailure)
                return fmt.Errorf("SDL: WriteSubscriptionToSdl(): %s", err.Error())
        } else {
@@@ -72,7 -70,7 +72,7 @@@ func (c *Control) ReadRESTSubscriptionF
  
        // This function is now just for testing purpose
        key := restSubId
 -      retMap, err := c.restSubsDb.Get([]string{key})
 +      retMap, err := c.restSubsDb.Get(restSubSdlNs, []string{key})
        if err != nil {
                c.UpdateCounter(cSDLReadFailure)
                return nil, fmt.Errorf("SDL: ReadSubscriptionFromSdl(): %s", err.Error())
@@@ -96,7 -94,7 +96,7 @@@
  
                restSubs = c.CreateRESTSubscription(restSubscriptionInfo, &jsonSubscriptionInfo)
  
-               restDuplicateCtrl.SetMd5sumFromLastOkRequest(restSubId, restSubs.lastReqMd5sum)
+               c.restDuplicateCtrl.SetMd5sumFromLastOkRequest(restSubId, restSubs.lastReqMd5sum)
        }
        return restSubs, nil
  }
@@@ -118,7 -116,7 +118,7 @@@ func (c *Control) CreateRESTSubscriptio
  func (c *Control) RemoveRESTSubscriptionFromSdl(restSubId string) error {
  
        key := restSubId
 -      if err := c.restSubsDb.Remove([]string{key}); err != nil {
 +      if err := c.restSubsDb.Remove(restSubSdlNs, []string{key}); err != nil {
                return fmt.Errorf("SDL: RemoveSubscriptionfromSdl(): %s\n", err.Error())
        } else {
                xapp.Logger.Debug("SDL: Subscription removed from restSubsDb. restSubId = %v", restSubId)
@@@ -130,7 -128,7 +130,7 @@@ func (c *Control) ReadAllRESTSubscripti
  
        retMap := make(map[string]*RESTSubscription)
        // Get all keys
 -      keys, err := c.restSubsDb.GetAll()
 +      keys, err := c.restSubsDb.GetAll(restSubSdlNs)
        if err != nil {
                c.UpdateCounter(cSDLReadFailure)
                return nil, fmt.Errorf("SDL: ReadAllSubscriptionsFromSdl(), GetAll(). Error while reading REST subscriptions keys from DBAAS %s\n", err.Error())
        }
  
        // Get all subscriptionInfos
 -      iRESTSubscriptionMap, err := c.restSubsDb.Get(keys)
 +      iRESTSubscriptionMap, err := c.restSubsDb.Get(restSubSdlNs, keys)
        if err != nil {
                c.UpdateCounter(cSDLReadFailure)
                return nil, fmt.Errorf("SDL: ReadAllSubscriptionsFromSdl(), Get():  Error while reading REST subscriptions from DBAAS %s\n", err.Error())
  
  func (c *Control) RemoveAllRESTSubscriptionsFromSdl() error {
  
 -      if err := c.restSubsDb.RemoveAll(); err != nil {
 +      if err := c.restSubsDb.RemoveAll(restSubSdlNs); err != nil {
                c.UpdateCounter(cSDLRemoveFailure)
                return fmt.Errorf("SDL: RemoveAllSubscriptionsFromSdl(): %s\n", err.Error())
        } else {
 -              xapp.Logger.Debug("SDL: All subscriptions removed from e2SubsDb")
 +              xapp.Logger.Debug("SDL: All subscriptions removed from restSubsDb")
        }
        return nil
  }