Fix creation of floating IP to use network ID instead of name

[dannytrigo]

Also fix deletion to ensure its a floating IP we try to delete

[dannytrigo]

The previous changes to allocate and destroy a floating IP do not work well as Fog expects a network ID rather than a name. This should fix it.

[coveralls]

Coverage increased (+0.02%) to 99.296% when pulling 8a4c2ef on dannytrigo:feature/allocateip into f4646cd on test-kitchen:master.

[dannytrigo]

Tested better this time:

-----> Creating <serverspec-openstack>...
       OpenStack instance with ID of <4bcac94b-44eb-49de-b057-3aea62a65aec> is ready.
       Attaching floating IP from <external> pool
       Created floating IP <1.2.3.4> from <external> pool
       Attaching floating IP <1.2.3.4>
       Waiting for server to be ready...
       Waiting for SSH service on 1.2.3.4:22, retrying in 3 seconds

-----> Destroying <serverspec-openstack>...
       Retrieve the floating IP
       Retrieve the ID of floating IP <1.2.3.4>
       OpenStack Floating IP <1.2.3.4> released.
       OpenStack instance <4bcac94b-44eb-49de-b057-3aea62a65aec> destroyed.
       Finished destroying <serverspec-openstack> (0m20.84s).

[dannytrigo]

Any chance to merge this? The implementation with the new flag is completely broken without it

[dannytrigo]

Thanks!