From 0e8ce4ad52560277ca585030b2fd5ea512d1610f Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Mon, 1 Aug 2016 08:33:09 -0500 Subject: [PATCH] Rename a confusing variable I had to read the two lines in question twice. That might just make me dumb, but if I'm dumb, someone else might be too. --- cloud/openstack/os_floating_ip.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cloud/openstack/os_floating_ip.py b/cloud/openstack/os_floating_ip.py index 81c200b2281..f20812a5e57 100644 --- a/cloud/openstack/os_floating_ip.py +++ b/cloud/openstack/os_floating_ip.py @@ -165,8 +165,8 @@ def main(): if state == 'present': # If f_ip already assigned to server, check that it matches # requirements. - f_ip = cloud.get_server_public_ip(server) - f_ip = _get_floating_ip(cloud, f_ip) if f_ip else f_ip + public_ip = cloud.get_server_public_ip(server) + f_ip = _get_floating_ip(cloud, public_ip) if public_ip else public_ip if f_ip: if network: network_id = cloud.get_network(name_or_id=network)["id"]