'terraform get' can't download modules as expected about terraform-aws-kubernetes HOT 3 CLOSED

@aholbreich I just copied your example verbatim into a main.tf in an empty directory and I was unable to reproduce the issue. Running terraform get works as expected:

Get: https://api.github.com/repos/coreos/terraform-aws-kubernetes/tarball/1.7.5-tectonic.1-rc.1?archive=tar.gz
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1
Get: git::https://github.com/coreos/tectonic-installer.git?ref=1.7.5-tectonic.1-rc.1

This could have been a network issue on your host. Can you please try running this again to help find the reproduction steps?

A couple notes:

• please use an explicit version of the module in your Terraform configuration. Navigating to https://registry.terraform.io/modules/coreos/kubernetes/aws with show you the documentation for the latest release (currently 1.7.5-tectonic.1.rc.5), however it looks like terraform get downloaded rc.1 for you. Being explicit about these versions is likely to help avoid future issues with API changes.
• tectonic_admin_password is not a valid variable for rc.1, but it is for rc.5; the former corresponding variable was tectonic_admin_password_hash

Please let me know if you continue to have issues.

from terraform-aws-kubernetes.

@aholbreich we need to streamline the vanilla_k8s option; currently all five variables (tectonic_admin_email, tectonic_admin_password_hash, tectonic_aws_ssh_key, tectonic_base_domain, and tectonic_cluster_name) are required even if many of them will end-up unused. We need to give these variables defaults so that they can empty in the case of a vanilla k8s installation. I will close this issue since the original problem was solved and track this new feature request internally. Thanks for pointing it out!

from terraform-aws-kubernetes.

Thank you for fast answer.
After retrying several times it seem to work now. And indeed i run in to the issue with unknown field
module kubernetes: tectonic_admin_password is not a valid parameter

Are these needed at all if i use tectonic_vanilla_k8s = true option?

tried pinning with:
source = "git::ssh://[email protected]/coreos/terraform-aws-kubernetes.git?ref=master"
terraform get worked...

from terraform-aws-kubernetes.