cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Rename a confusing variable

Browse source 
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.
This commit is contained in:
Monty Taylor 2016-08-01 08:33:09 -05:00 committed by Matt Clay
parent 4fad93561a
commit f441e2c1a9
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/ansible/modules/cloud/openstack/os_floating_ip.py
View file

@ -164,8 +164,8 @@ def main():
if state == 'present': if state == 'present':
# If f_ip already assigned to server, check that it matches # If f_ip already assigned to server, check that it matches
# requirements. # requirements.
f_ip = cloud.get_server_public_ip(server) public_ip = cloud.get_server_public_ip(server)
f_ip = _get_floating_ip(cloud, f_ip) if f_ip else f_ip f_ip = _get_floating_ip(cloud, public_ip) if public_ip else public_ip
if f_ip: if f_ip:
if network: if network:
network_id = cloud.get_network(name_or_id=network)["id"] network_id = cloud.get_network(name_or_id=network)["id"]