Details
-
Bug
-
Status: Closed
-
Medium
-
Resolution: Done
-
Istanbul Release
Description
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:
- Update NCMP REST interface to allow Resource Identifier with / tokens
- Update DMI Plugin interface to allow Resource Identifier with / tokens
- Forward the complete Resource Identifier to SDN-C without intervention/checks in DMI Plugin
- This should be demo-ed using full stack possibly using some logging
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
Attachments
Issue Links
# | Subject | Branch | Project | Status | CR | V |
---|---|---|---|---|---|---|
124573,4 | Changing resource identifier to a query param | master | cps/ncmp-dmi-plugin | Status: MERGED | +2 | +1 |
124574,9 | Changing resource identifier to a query param | master | cps | Status: MERGED | +2 | +1 |