- Notifications
You must be signed in to change notification settings - Fork928
feat: Add examples/templates/do-linux for Digital Ocean Droplets#1749
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
1888f81 toaa51940Compare| @@ -0,0 +1,46 @@ | |||
| #cloud-config | |||
| users: | |||
| - name: coder | |||
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 could even make the username their Coder username 👀
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.
Oh I love that!
aa51940 to8d73582CompareSuper small nit, but |
@coadler My motivation for using |
This is true for the other clouds too though. Maybe we should have subfolders for each cloud? Don't mean to derail this at all, just thinking out loud! |
Good point. How about I make the change here to standard naming scheme, then we can consolidate later? At the risk of bike-shedding, I do kind of like that the name explains exactly what it is since the name is often inherited from the folder name. So we'd have |
Works for me! |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
examples/do-linux/main.tf Outdated
| description = "Which Droplet image would you like to use for your workspace?" | ||
| default = "ubuntu-22-04-x64" | ||
| validation { | ||
| condition = contains(["ubuntu-22-04-x64", "ubuntu-20-04-x64", "fedora-36-x64", "fedora-35-x64", "debian-11-x64", "debian-10-x64", "centos-stream-9-x64", "centos-stream-8-x64", "rockylinux-8-x64", "rockylinux-8-4-x64"], var.droplet_image) |
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 wanted to conditionally include Fedora images based on if SSH key is set, but it's not possible, unfortunately :/hashicorp/terraform#25609
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.
Looking great!
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Cian Johnston <cian@coder.com>
Right now we leave the SSH server running on the Droplet. I think this is OK, if someone doesn't want that, they can edit the With the last commit (08483d8) we could in theory use remote-exec instead of cloud-config (when SSH key is provided). But perhaps that could be left for an alternative implementation of this template. |
Co-authored-by: Cian Johnston <cian@coder.com>
Uh oh!
There was an error while loading.Please reload this page.
This PR adds an example that provisions workspaces as Digital Ocean Droplets and mounts a persistent volume for the users (
coder) home directory.I wanted to provision the workspaces in a specified Digital Ocean Project, but due to terraform detecting this as two workspace volumes, there's a side-effect of not being able to SSH into the workspace.
For now, the project specification is commented out:
See#1750 for details on the issue.