[NETVIRT-1205] Neutron DHCP Port FIB entries are appearing in dual stack network Created: 10/Apr/18 Updated: 30/May/18 Resolved: 24/Apr/18 |
|
| Status: | Verified |
| Project: | netvirt |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | Fluorine |
| Type: | Bug | Priority: | Medium |
| Reporter: | Karthikeyan Krishnan | Assignee: | Karthikeyan Krishnan |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | VPNV6 | ||
| Remaining Estimate: | 0 minutes | ||
| Time Spent: | 1 week | ||
| Original Estimate: | 1 week | ||
| Description |
|
Neutron DHCP Port FIB entries are appearing in dual stack network when openstack q-dhcp service is being used. cookie=0x8000003, duration=1837.161s, table=21, n_packets=0, n_bytes=0, priority=138,icmp6,metadata=0x30d40/0xfffffe,ipv6_dst=2001:db8:1111::1,icmp_type=128,icmp_code=0 actions=move:NXM _OF_ETH_SRC[] Where 10.0.0.2 is the dhcp-port and setup uses neutron dhcp-agent.
-------------------------------------
------------------------------------- [vagrant@centos7_aione_ocata_odl SampleScripts]$ neutron net-list
-------------------------------------
-------------------------------------
-------------------------------------
------------------------------------- |
| Comments |
| Comment by Karthikeyan Krishnan [ 24/Apr/18 ] |
|
Based on the code review comments, dropping this fix. Since it may introduce some regression issue for external FLAT/VLAN based provider networks. Please refer the below link and comments for more details about the discussion. https://git.opendaylight.org/gerrit/#/c/70764/ This is a use-case that is working today and changing it could introduce regressions. Tempest/Neutron functional/scenario tests do validate some use-cases where they try to ping to dhcp-port. I don't know if its within the network or across networks. Either way, there could be users who may want to validate east-west/north-south use-case with dhcp-ports (instead of spawning new VMs). IMHO this is a reasonable expectation and is working today. Worst case, I'm fine if we want to discard this patch and leave the functionality as-is. Its better than introducing some regression. |