stackforge/fuel-web — pull request #5 by TorstenS73 (2014-09-18 13:24:17+00:00)
Title: fxing minor case of "don't repeat yourself"

I think the original purpose of the replaced code was to ensure that the IP address of the node is in the admin subnet. It would therefore make sense IMHO to use the existing function is_ip_belongs_to_admin_subnet as we may decide later that is_ip_belongs_to_admin_subnet may use other criteria than just checking if the IP is within the range of one given CIDR. Think multiple disjunct admin network segments for example or think IKv6 (see also https://blueprints.launchpad.net/fuel/+spec/add-ipv6-support-to-fuel; not that there is too much action yet on that one ;-) ).