fixes #289

CHANGELOG.txt

@@ -1,3 +1,5 @@
+2.14.4:
+    - By default, don't allow using an existing floating IP if it had already been allocated.
 2.14.3:
     - Fix check subnet cidr for fixed ip.
 2.14.2:

openstack_plugin_common/floatingip.py

@@ -14,6 +14,7 @@
 #  * limitations under the License.
 
 from cloudify import ctx
+from cloudify.exceptions import RecoverableError
 from openstack_plugin_common import (
     delete_resource_and_runtime_properties,
     use_external_resource,
@@ -22,7 +23,6 @@
     OPENSTACK_ID_PROPERTY,
     OPENSTACK_TYPE_PROPERTY)
 
-
 FLOATINGIP_OPENSTACK_TYPE = 'floatingip'
 
 # Runtime properties
@@ -35,6 +35,12 @@ def use_external_floatingip(client, ip_field_name, ext_fip_ip_extractor):
     external_fip = use_external_resource(
         ctx, client, FLOATINGIP_OPENSTACK_TYPE, ip_field_name)
     if external_fip:
+        if ctx.node.properties['allow_reallocation'] \
+                and external_fip['status'] == 'ACTIVE':
+            raise RecoverableError(
+                    'Floating IP address {0} is already associated'.format(
+                        external_fip['floating_ip_address'])
+            )
         ctx.instance.runtime_properties[IP_ADDRESS_PROPERTY] = \
             ext_fip_ip_extractor(external_fip)
         return True

plugin.yaml

@@ -883,6 +883,14 @@ node_types:
         description: >
           a boolean for setting whether to create the resource or use an existing one.
           See the using existing resources section.
+      allow_reallocation:
+        type: boolean
+        default: false
+        description: >
+          (Applicable only when use_external_resource is true)
+          If true, then allow using this floating IP even if it has already been allocated to
+          another instance. If false, and the floating IP is already allocated (that is, it is
+          in 'ACTIVE' state), a recoverable error is raised.
       create_if_missing:
         type: boolean
         default: false