Rename add_floating_ip arg to floatingip_addr 33/25833/1
authorRomanos Skiadas <rski@intracom-telecom.com>
Tue, 13 Dec 2016 08:28:01 +0000 (10:28 +0200)
committerRomanos Skiadas <rski@intracom-telecom.com>
Tue, 13 Dec 2016 08:29:37 +0000 (10:29 +0200)
nova_client.servers.add_floating_ip takes a Floating IP address as a second
argument, not an id. Naming the argument floatingip_id is misleading.

Change-Id: I39919fa305a46b58878c8b4f83e40c78390cfc07
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
functest/utils/openstack_utils.py

index 10aaf50..c304ae3 100755 (executable)
@@ -393,13 +393,13 @@ def create_floating_ip(neutron_client):
     return {'fip_addr': fip_addr, 'fip_id': fip_id}
 
 
-def add_floating_ip(nova_client, server_id, floatingip_id):
+def add_floating_ip(nova_client, server_id, floatingip_addr):
     try:
-        nova_client.servers.add_floating_ip(server_id, floatingip_id)
+        nova_client.servers.add_floating_ip(server_id, floatingip_addr)
         return True
     except Exception, e:
         logger.error("Error [add_floating_ip(nova_client, '%s', '%s')]: %s"
-                     % (server_id, floatingip_id, e))
+                     % (server_id, floatingip_addr, e))
         return False