-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
provider/openstack: Implement fixed_ip on floating ip allocations #6837
Conversation
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? |
@carlpett This is a really nice PR - thanks! It looks like Travis is failing due to some 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.) |
@jtopjian Thanks :) |
e5d3bf5
to
00be33e
Compare
@jtopjian No problem :) I've just rebased and updated. Fixed formatting as well, so hopefully the CI will work this time ( |
Also, I made a small change - I changed |
@carlpett This all looks great! Thank you! |
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. |
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.