Uploaded image for project: 'Configuration Persistence Service'
  1. Configuration Persistence Service
  2. CPS-679

Passthrough does not support resourceIdentifier with / tokens

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Medium Medium
    • Istanbul Release
    • Istanbul Release
    • DMI, NCMP

      Currently NCMP does NOT support / tokens in the resource identifier as the OpenAPI definition would regard that as separate REST paths ie. a different rest endpoint

      /v1/ch/{cm-handle}/data/ds/ncmp-datastore:passthrough-operational/{resourceIdentifier}
      /v1/ch/{cm-handle}/data/ds/ncmp-datastore:passthrough-running/{resourceIdentifier}
      

      Currently supported sample URLs

      /v1/ch/node1/data/ds/ncmp-datastore:passthrough-operational/turing-machine:turing-machine?fields=transition-function?depth=3
      

      Wanted  (currently not) supported sample URLs

      /v1/ch/node1/data/ds/ncmp-datastore:passthrough-operational/turing-machine:turing-machine/transition-function?depth=5 
      

      In this sample the resource id is the whole string between the slash after the datastore value and the '?' that indicate the start of the query parameters: "turing-machine:turing-machine/transition-function"

      A possible work-around is required in the OpenApi definition to allow this

      For more details see analysis here: https://wiki.onap.org/pages/viewpage.action?pageId=111120893

      In summary the fix for this bug should:

      1. Update NCMP REST interface to allow Resource Identifier with / tokens
      2. Update DMI Plugin interface to allow Resource Identifier with / tokens
      3. Forward the complete Resource Identifier to SDN-C without intervention/checks in DMI Plugin
        1. This should be demo-ed using full stack possibly using some logging
      4. URLs should be backward compatible with current implementation
        This is no longer possible, see analysis on https://wiki.onap.org/pages/viewpage.action?pageId=111120893

       

            niamhcore niamhcore
            ToineSiebelink Toine Siebelink
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: