equinix-labs / terraform-equinix-metal-k3s

Manage K3s (k3s.io) region clusters on Equinix Metal
https://registry.terraform.io/modules/equinix/k3s/metal/latest?tab=readme
Apache License 2.0
46 stars 15 forks source link

Removes External Provider Dependency #13

Closed jmarhee closed 5 years ago

jmarhee commented 5 years ago

Because of apparent incompatibility with Terraform 0.12, I removed the provider script, and the dependency of checking the node-token into Terraform state. This also improves spin-up time, but introduces a local-exec (so same behavior in terms of client connection) that SCPs (rather than pulling down the data and registering with TF) the node-token from the controller to cluster nodes.

This also adds auto-assign: false to the MetalLB base config. This will enable #5 and #10 while not wasting a public IP address.

Adds note on clusters.tf intended use in in-line comment, to supplement the README's description of using the Makefile to add clusters.