Description
NETCONF is currently abusing DOMMountPoint and especially DOMRpcService to funnel unparsed anyxml nodes through DOM interfaces.
This relies on a historic leniency of DOMRpcResult of allowing other nodes than containers to carry the result.
This is completely wrong, as NETCONF mount points which do not have a schema should not expose a DOMRpcService, but some completely different service, which does not play pretend it conforms to DOMRpcService specification.
Most notable example is that while normal mount points can be interacted with via Binding components, as we can make sense of the payload. schemaless mounts cannot – yet they pretend their services can in fact be a source of normalized data.
Attachments
Issue Links
- blocks
-
MDSAL-541 Switch DOMRpc{Implementation,Result} to work on ContainerNode
-
- Resolved
-
- relates to
-
MDSAL-542 Drop the notion that a DOMMountPoint automatically has a SchemaContext
-
- Resolved
-
- split to
-
NETCONF-925 Rework optional reconnect logic
-
- Confirmed
-
Gerrit Reviews
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 103210,19 | Create RemoteDeviceServices.{Actions,Rpcs} | master | netconf | Status: MERGED | +2 | +1 |
| 103212,4 | Specialize RemoteDevice to NetconfMessage | master | netconf | Status: MERGED | +2 | +1 |
| 103213,3 | Specialize MessageTransformer to NetconfMessage | master | netconf | Status: MERGED | +2 | +1 |
| 103214,3 | Ditch NetconfNode from SchemaResourceManager | master | netconf | Status: MERGED | +2 | +1 |
| 103216,4 | Specialize RemoteDevice to NetconfSessionPreferences | master | netconf | Status: MERGED | +2 | +1 |
| 103218,3 | Specialize RemoteDeviceHandler to NetconfSessionPreferences | master | netconf | Status: MERGED | +2 | +1 |
| 103609,16 | Make sure NetconfSessionPreferences are immutable | master | netconf | Status: MERGED | +2 | +1 |
| 103612,32 | Disconnect NetconfDeviceCapabilities and NetconfSessionPreferences | master | netconf | Status: MERGED | +2 | +1 |
| 103617,13 | Move DeviceSources registration | master | netconf | Status: MERGED | +2 | +1 |
| 103618,8 | Clean up NetconfTopology methods | master | netconf | Status: MERGED | +2 | +1 |
| 103619,14 | Complete future outside of lock | master | netconf | Status: MERGED | +2 | +1 |
| 103628,6 | NetconfSessionPreferences is a record | master | netconf | Status: MERGED | +2 | +1 |
| 103629,7 | Turn UserPreferences into a record | master | netconf | Status: MERGED | +2 | +1 |
| 103652,1 | Fixup NetconfDeviceCommunicator | master | netconf | Status: MERGED | +2 | +1 |
| 103653,10 | Add RemoteDeviceServices | master | netconf | Status: MERGED | +2 | +1 |
| 103691,1 | Rework KeepaliveTask lifecycle | master | netconf | Status: MERGED | +2 | +1 |