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

Add support for floating IP #284

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwakwaversal
Copy link

Automatically assign a floating IP after the droplet is created.

Fixes #278

Automatically assign a floating IP after the droplet is created.
@davidbasalla
Copy link

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,
Copy link

@bbros-dev bbros-dev Feb 18, 2020

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?

Copy link
Author

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.

Copy link
Member

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.

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.

@utkonos
Copy link

utkonos commented Nov 8, 2022

Endpoint path is now reserved_ips.

#278 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability to configure a Floating IP
5 participants