Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provider/openstack: Implement fixed_ip on floating ip allocations #6837

Merged

Conversation

carlpett
Copy link
Contributor

When creating ports with multiple fixed addresses, you cannot assign a floating ip to it unless specifying which fixed address to map to. This PR adds functionality for specifying it.

@carlpett
Copy link
Contributor Author

I added a new required env var to be able to set up a router. That will require some changes in the test environment, I guess?
There might be some room for improvement in the setup of my test case, I had some problems getting a smaller setup to work, but maybe some of the resources can be skipped?

@jtopjian
Copy link
Contributor

@carlpett This is a really nice PR - thanks!

It looks like Travis is failing due to some go fmt issues in the test file. I see some alignment issues with the nested Terraform configuration, too. I run into those all the time, myself...

I'll take a look at this within the next day or so and see if I can find a workaround so the new env variable isn't needed, but if it is, no big deal.

(If you haven't seen it, here's the script that builds out the test environment for acceptance tests.)

@carlpett
Copy link
Contributor Author

@jtopjian Thanks :)
I had a bunch of issues with the formatting parts of the Makefile, due to working on a Windows machine (CRLF-checkout) and building in the Vagrant environment (expects CR only). I'll try to fix it.

@jtopjian
Copy link
Contributor

@carlpett Just a heads up that #6454 was just merged that edits the same file. There will be some merge conflicts that need resolved for this PR to cleanly merge. Sorry about that...

@carlpett carlpett force-pushed the f-openstack-floatingip-fixed-binding branch from e5d3bf5 to 00be33e Compare May 25, 2016 11:50
@carlpett
Copy link
Contributor Author

carlpett commented May 25, 2016

@jtopjian No problem :) I've just rebased and updated. Fixed formatting as well, so hopefully the CI will work this time (go fmt is happy, but it looks a bit strange to me...).

@carlpett
Copy link
Contributor Author

Also, I made a small change - I changed computed to true. I think this is correct, but I could misunderstand how it works?

@jtopjian
Copy link
Contributor

@carlpett This all looks great! Thank you!

@jtopjian jtopjian changed the title Implement fixed_ip on floating ip allocations provider/openstack: Implement fixed_ip on floating ip allocations May 28, 2016
@jtopjian jtopjian merged commit 3aadc19 into hashicorp:master May 28, 2016
@carlpett carlpett deleted the f-openstack-floatingip-fixed-binding branch May 29, 2016 07:16
@ghost
Copy link

ghost commented Apr 25, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants