Paul Ward:

Thank you to all who have participated in this thread.  I've just proposed a 
fix in gerrit.  For those involved thus far, if you could review I would be 
greatly appreciative!

https://review.openstack.org/#/c/68742/1

I wouldn't go so far as to say this verification SHOULDN'T be added, but 
neither would I say it should. From a general use case perspective I don't 
think IPv4 subnets smaller than /29 make sense. A /32 is a commonly used subnet 
length for some use cases (e.g. router loopback interface) but may not have an 
applicable use in a cloud network. I have never seen a /31 network used 
anywhere. Point to point links (e.g. T1/Frame Relay/etc) are often /30 but I've 
never seen a /30 subnet for anything other than connecting two routers.

However, does it really benefit the user to specifically block them from 
entering /32 or block them from entering /30, /31, and /32?

It might not be an equal amount of code, I think a much better effort to help 
the user would be to provide them with a subnet calculator directly in Horizon 
to show them how many usable IPs are in the subnet they're defining. In this 
case, displaying "Usable addresses: 0" right when they enter /32 would be 
helpful and they would figure out for themselves whether they really wanted 
that mask or if they meant something else?


_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to