OpenDaylight JIRA https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 [NEUTRON-31] On a router_interface_add, the Northbound code assumed (incorrectly) the port has null device_id and device_owner https://jira.opendaylight.org/browse/NEUTRON-31 neutron <p>This was found while integrating with the router_interface_add/remove calls from OpenStack.</p> <p>Operating System: All<br/> Platform: All</p> NEUTRON-31 On a router_interface_add, the Northbound code assumed (incorrectly) the port has null device_id and device_owner Bug Resolved Done Rajesh Jaiswal Kyle Mestery Fri, 17 Apr 2015 14:54:17 +0000 Thu, 11 Jun 2015 16:08:44 +0000 Thu, 11 Jun 2015 16:08:44 +0000 unspecified General 0 3 <p>As per northbound code the device_Id and device_Owner on router_interface_add does not looks that it assumes (incorrectly) the port has null device_id and device_owner .Please look at the following code .. </p> <p> if (targetPort.getDeviceID() != null &amp;&amp; !targetPort.getDeviceID().equals(routerUUID))<br/> throw new ResourceConflictException("Target Port already allocated to a different device id");<br/> if (targetPort.getDeviceOwner() != null &amp;&amp;<br/> !targetPort.getDeviceOwner().equalsIgnoreCase(ROUTER_INTERFACE_STR))<br/> throw new ResourceConflictException("Target Port already allocated to non-router interface");</p> <p>/////////////////////////</p> <p> //mark the port device id and device owner fields<br/> if (targetPort.getDeviceOwner() == null || targetPort.getDeviceOwner().isEmpty())<br/> targetPort.setDeviceOwner(ROUTER_INTERFACE_STR);<br/> targetPort.setDeviceID(routerUUID);</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/22220" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/22220</a> adds tests to address this defect in master branch</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/22222" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/22222</a> does the same for the stable/lithium branch</p> <p>Merge to Helium via this gerrit:</p> <p><a href="https://git.opendaylight.org/gerrit/22227" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22227</a></p> Development External issue ID 3014 External issue URL Rank 0|i02ua7: