Open displague opened 11 months ago
kubeconfig.admin.yaml (as defined in https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/blob/main/main.tf#L122) should be an output of the module. This example should rely on that output, not take the filename for granted.
https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/blob/main/main.tf#L113-L124 should be modified to ensure that the filename (which could be a filename instead of hardcoded), helpful when the moule is used for multiple workspaces) and the module.path should be used in the paths of the null_resoource provisioner commands.
_Originally posted by @displague in https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/pull/33#discussion_r1355369859_
kubeconfig.admin.yaml (as defined in https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/blob/main/main.tf#L122) should be an output of the module. This example should rely on that output, not take the filename for granted.
https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/blob/main/main.tf#L113-L124 should be modified to ensure that the filename (which could be a filename instead of hardcoded), helpful when the moule is used for multiple workspaces) and the module.path should be used in the paths of the null_resoource provisioner commands.
_Originally posted by @displague in https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/pull/33#discussion_r1355369859_