Closed michaelbrewer closed 2 months ago
I think I have suggested all the name changes, but I could have missed some.
some of this was there from before and some of it was exactly working code from the code-server implementation. I have accepted all your suggestions, but i will need to retest and fix CI errors.
Thank you, @michaelbrewer ❤️ . You are doing a great job of making modules super helpful.
Let's wait for an approval from @code-asher and I will merge and cut a release.
Thank you, @michaelbrewer ❤️ . You are doing a great job of making modules super helpful.
Let's wait for an approval from @code-asher and I will merge and cut a release.
Would be nice to have official templates is composed of modules. :)
Would be nice to have official templates is composed of modules. :)
I have also thought about this and created an issue. But Coder bundles starter templates within the product and when we add modules that are fetched from registry, this does not work on restricted or air-gapped deployments.
Would be nice to have official templates is composed of modules. :)
I have also thought about this and created an issue. But Coder bundles starter templates within the product and when we add modules that are fetched from registry, this does not work on restricted or air-gapped deployments.
Most of the included examples don't run air-gapped as they would curl to install code-server.
Maybe https://developer.hashicorp.com/terraform/language/providers/requirements#source-addresses to allow for registry.coder.com
?
Scanning through the examples you can see cases of this:
examples/templates/aws-devcontainer/main.tf
- (https://get.docker.com
and https://registry.coder.com/modules/code-server
etc..)examples/templates/aws-linux/main.tf
- (https://code-server.dev/install.sh
)examples/templates/docker/main.tf
- (https://code-server.dev/install.sh
)
etc..