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 external IP to instance template #14

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

Conversation

rail
Copy link

@rail rail commented Jun 5, 2024

Previously, instances, created by the instance template, were unable to download the launcher binary, because they had no external IP addresses assigned.

This PR sets the network tier to PREMIUM, which assigns an ephemeral IP address to the created instances automatically.

@Apollorion
Copy link
Member

@rail can you bump the version in .spacelift/config.yml after that I will see what we need to do to get this approved.

Previously, instances, created by the instance template, were unable to
download the launcher binary, because they had no external IP addresses
assigned.

This PR sets the network tier to `PREMIUM`, which assigns an ephemeral
IP address to the created instances automatically.
@rail
Copy link
Author

rail commented Jun 5, 2024

done

@michalg9 michalg9 requested a review from a team September 6, 2024 10:07
@sephriot
Copy link

@rail Have you considered using NAT instead directly assigning a public IP to the instance? What would be the trade offs?

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.

3 participants