[NETCONF-520] RPC add netconf device does not work Created: 02/Mar/18 Updated: 08/Mar/18 Resolved: 08/Mar/18 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | netconf |
| Affects Version/s: | Oxygen |
| Fix Version/s: | Oxygen |
| Type: | Bug | Priority: | Medium |
| Reporter: | Luis Gomez | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
This is shown in this test suite: https://jenkins.opendaylight.org/releng/view/netconf/job/netconf-csit-1node-userfeatures-all-oxygen/
I see when this RPC is invoked: restconf/operations/netconf-node-topology:add-netconf-node <input xmlns="urn:opendaylight:netconf-node-topology"> <node-id>netconf-test-device</node-id> <encrypt>true</encrypt> <host>10.30.170.85</host> <port>17830</port> <username>admin</username> <password>topsecret</password> <tcp-only>false</tcp-only> <keepalive-delay>0</keepalive-delay> </input>
Controller responds with:
<errors> xmlns="urn:ietf:params:xml:ns:yang:ietf-restconf"> <error> <error-type>protocol</error-type> <error-tag>invalid-value</error-tag> <error-message>URI has bad format. Possible reasons: 1. "netconf-node-topology:add-netconf-node" was not found in parent data node. 2. "netconf-node-topology:add-netconf-node" is behind mount point. Then it should be in format "/yang-ext:mount/netconf-node-topology:add-netconf-node".</error-message> </error> </errors>
Adding device via topology config works fine and the above RPC works in Nitrogen so unless there is an expected and documented API change in Oxygen, this is a BLOCKER for Oxygen. |
| Comments |
| Comment by Ryan Goulding [ 05/Mar/18 ] |
|
I do believe it is just called "create-device" now... not "add-netconf-device" as of https://git.opendaylight.org/gerrit/#/c/64875/. Given that this API change was never announced to the community, I do question why it was made. However, it looks like just a name change. Can you confirm that this works, and we can then close this one out? |
| Comment by Luis Gomez [ 05/Mar/18 ] |
|
OK, this is the patch that changed the RPC: https://git.opendaylight.org/gerrit/#/c/64718/ Almost nobody in copy and same person pushing and approving.
|
| Comment by Luis Gomez [ 06/Mar/18 ] |
|
FYI test patch using new API works: https://git.opendaylight.org/gerrit/#/c/69101/ So we can reduce the severity for now and close the issue once the above is merged. |
| Comment by Luis Gomez [ 08/Mar/18 ] |
|
Patch is merged, we can close this. |
| Comment by Luis Gomez [ 08/Mar/18 ] |
|
Fix merged and verified. |