weizhouapache commented on PR #7746:
URL: https://github.com/apache/cloudstack/pull/7746#issuecomment-1683565479

   > > @Rubueno @wido I gave this another look and it is not going to work. The 
link local addresses are created on createPod and changing the configurations 
will break any SVM created after changing the settings. We'll need a more 
extensive disign. Can you give some input on what a valid scenario for you 
would be?
   > > thanks
   > 
   > We should move to IPv6 Link-Local at all for the SSVM :-) These addresses 
with IPv4 aren't needed at all ;-) Makes life much easier, I promise!
   > 
   > Anyway, this isn't the biggest problem now.
   > 
   > In RFC5549 the IPv4 address 169.254.0.1 is used for routing IPv4 traffic 
via IPv6, something which you will use when using EVPN+BGP+VXLAN.
   > 
   > Therefor you want to change the control CIDR to something else. It's 
'stupid' that we allocate a /16 to this as that's not needed at all, a /20 by 
default should be more then enough.
   > 
   > Any existing installations work just fine. Maybe we should just change the 
default from this /16 to a /20 (169.254.240.0/20) and be done with it for now.
   
   thanks @wido 
   
   @DaanHoogland I think the main problem for this PR is, all link local IPs 
are saved in database table `op_dc_link_local_ip_address_alloc` which is not 
needed at all. 
   We need to save only the allocated IPs in the table and remove all unused 
IPs.
   Some classes need to be updated to support the IP allocation from 
cidr/gateway, instead of picking up IPs from 
`op_dc_link_local_ip_address_alloc` table.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to