[NEUTRON-113] Router Interfaces of Routers remain unfilled even though subnets are added to Router via Openstack Neutron Created: 17/Feb/16  Updated: 30/Oct/17  Resolved: 10/Mar/16

Status: Resolved
Project: neutron
Component/s: transcriber
Affects Version/s: Multiple
Fix Version/s: None

Type: Bug
Reporter: Vivekanandan Narasimhan Assignee: Vishal Thapar
Resolution: Won't Do Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

External issue ID: 5369
Priority: High

 Description   

Router Interfaces field in Neutron Northbound aren't populated on router interface additions made in Openstack Neutron.

As a result, all ODL services that bank on NeutronRouter to recover all information and change notifications fail.

 Comments   
Comment by Vishal Thapar [ 17/Feb/16 ]

Bug is in https://github.com/opendaylight/neutron/blob/master/transcriber/src/main/java/org/opendaylight/neutron/transcriber/NeutronRouterInterface.java#L156

toMd() has this block of code:

if (router.getInterfaces() != null) {
Map<String, NeutronRouter_Interface> mapInterfaces = new HashMap<String, NeutronRouter_Interface>();
List<Interfaces> interfaces = new ArrayList<Interfaces>();
for (Entry<String, NeutronRouter_Interface> entry : mapInterfaces.entrySet())

{ interfaces.add((Interfaces) entry.getValue()); }

routerBuilder.setInterfaces(interfaces);
}

mapInterfaces is initialized to empty list but never populated with actual interfaces received from Northbound. So for loop always exists without adding to interfaces and setInterfaces always sets empty list.

Comment by Vishal Thapar [ 17/Feb/16 ]

Gerrit patch: https://git.opendaylight.org/gerrit/34818

Comment by Vishal Thapar [ 10/Mar/16 ]

Decision is to drop interfaces from router in Boron and leave it as is for Beryllium. Providers should use Port ownership to determine if it is router interface and handle it.

Generated at Wed Feb 07 20:25:35 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.