-
Notifications
You must be signed in to change notification settings - Fork 183
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
Add support for floating IP #284
base: master
Are you sure you want to change the base?
Add support for floating IP #284
Conversation
Automatically assign a floating IP after the droplet is created.
This looks very useful, is there an update on this PR? |
if @machine.provider_config.floating_ip | ||
result = @client.post("/v2/floating_ips/#{@machine.provider_config.floating_ip}/actions", { | ||
:type => 'assign', | ||
:droplet_id => @machine.id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a hanging ,
here that likely is preventing this from running. @kwakwaversal?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't code in Ruby (I quickly knocked this up to serve a purpose), but my understanding is that a trailing comma is purely stylistic and doesn't affect the functionality of the code.
I have added a trailing comma out of habit (to reduce line noise in MRs) but I'm using this same code on my box to provision digital ocean containers without a problem.
I think this repo might be stale. If I had feedback to tell me to drop the comma I will, but until then it will continue to knock at the PR door.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're still here! We've been working on partnering with DigitalOcean to get this plugin up-to-date with their product suite. Until then, there will be some delay on features like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack. I couldn't get this fix to work for me, so likely needs some closer scrutiny.
What did work was a curl script in the provisioning stage - but without DO's input it'll be painful working out a robust solution.
Endpoint path is now |
Automatically assign a floating IP after the droplet is created.
Fixes #278