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

use metal_ variables instead of equinix_metal_ #85

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

displague
Copy link
Member

@displague displague commented Jul 11, 2022

Reverts the equinix_metal_ variable names back to metal_.

metal_ is the current convention: https://github.com/equinix-labs/terraform-equinix-template/blob/main/variables.tf


Previously this PR was focused on the conversion to the Equinix provider, which was merged in #88

The old description follows:

Updates the Terraform provider from the Metal provider, which has been deprecated, to the Equinix provider.

If updating from a previously deployed environment, the following guide should help: https://registry.terraform.io/providers/equinix/equinix/latest/docs/guides/migration_guide_equinix_metal

terraform apply builds the HA Anthos 1.8.7 environment successfully.

$ terraform init
...
$ terraform validate
...
$ terraform apply
...
Apply complete! Resources: 58 added, 0 changed, 0 destroyed.

Outputs:

Control_Plane_Public_IPs = [
  "147.28.148.43",
  "147.28.148.137",
  "147.28.148.111",
]
Control_Plane_VIP = "145.40.81.128"
Equinix_Metal_Project_ID = "24822e74-bf8c-4637-bcb9-71ef8fd88eed"
Ingress_VIP = "147.28.154.43"
Kubeconfig_location = "/Users/marques/src/terraform-metal-anthos-on-baremetal/eqnx-metal-gke-tgll8-kubeconfig"
Worker_Public_IPs = [
  "147.28.148.61",
  "147.28.148.63",
  "147.28.148.37",
]
ssh_key_location = "/Users/marques/.ssh/anthos-eqnx-metal-gke-tgll8"
$ KUBECONFIG=/Users/marques/src/terraform-metal-anthos-on-baremetal/eqnx-metal-gke-tgll8-kubeconfig kubectl get nodes 
NAME                             STATUS   ROLES                  AGE   VERSION
eqnx-metal-gke-tgll8-cp-01       Ready    control-plane,master   16m   v1.20.9-gke.102
eqnx-metal-gke-tgll8-cp-02       Ready    control-plane,master   10m   v1.20.9-gke.102
eqnx-metal-gke-tgll8-cp-03       Ready    control-plane,master   10m   v1.20.9-gke.102
eqnx-metal-gke-tgll8-worker-01   Ready    <none>                 14m   v1.20.9-gke.102
eqnx-metal-gke-tgll8-worker-02   Ready    <none>                 14m   v1.20.9-gke.102
eqnx-metal-gke-tgll8-worker-03   Ready    <none>                 14m   v1.20.9-gke.102

@displague
Copy link
Member Author

I enabled HA and #83 appears to not be an issue.

@displague displague changed the title update TF metal provider to equinix use metal_ variables instead of equinix_metal_ May 17, 2023
displague added 2 commits May 17, 2023 18:00
equinix_metal_ as a prefix is overly verbose and doesn't follow the
practices of other Equinix Terraform modules.

Signed-off-by: Marques Johansson <[email protected]>
Signed-off-by: Marques Johansson <[email protected]>
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.

2 participants