-
Notifications
You must be signed in to change notification settings - Fork 266
Terraform module cannot deploy into existing Terraform-managed AWS VPC #1966
Comments
Playing around with this some more, it seems like there's problems with count attributes and existing VPCs all over the place:
|
And yet more info - even with literal values the same error is raised:
|
Hitting the same.
|
@alexsomesan - Can you review this issue as well. |
@kerin @thetaris-slack |
@kerin @thetaris-slack - Any update on the tfvars that @alexsomesan asked. Looks like there are no such issues reported in later RCs. Pl confirm that you are not seeing these anymore. Will close this. We are getting close to release 1.7.5 release so want to confirm |
@kerin @thetaris-slack - We are getting close to closing validation cycle to prepare for RC. Can you pl review and report back to take action. Otherwise we will miss this for tectonic 1.7.5 release |
@sudhaponnaganti I'm still seeing this issue I'm afraid. My module instantiation looks like this, against RC5:
The module outputs referenced above are defined as:
On
(obviously I can't provide the literal values from my module outputs, as this error prevents those resources from being created in the first place) |
Thanks @kerin for the input |
Having the same. I tried th workaround from @kerin without success. Any quick fix would be highly appreciated, as I can't scale up.
|
tfvars:
I want to scale up to 3 workers. |
|
Hello, exact same error at my end. Just running a plan as I needed to scale up my cluster. Not done anything other than run the command in the documentation. |
Also running into this
I am getting the following error on a
Mostly at random I picked
https://github.com/coreos/terraform-aws-kubernetes/blob/1.8.4-tectonic.3/main.tf#L75-L77
Our https://github.com/coreos/terraform-aws-kubernetes/blob/master/tectonic.tf#L2
Either way I'm not sure why it thinks it can't compute terraform 0.10.8, though have also tried with 0.11.2 to no avail. |
self-followup: my issue above went away when I did a |
@ssunagari so what was the actual problem? (btw, the language used around the dns zone has been confusing to me too) |
We are still seeing this problem on a fresh default install, even with tectonic-1.8.9 and its terraform v0.10.7 binary. We are not doing any Route53 shenanigans like ssunagari, just a full AWS install and upscale from the vanilla docs with vanilla binaries, no special sauce. Trying anew & failing since 8 months. This issue can be very easily reproduced and we really are in desperate need of a solution. A solution for us would also be a hardcoded instance count in some file if only somebody would finally shed some light on this ticket, but I don't feel comfortable to tinker with the deep-level logic of AWS Terraforming for Kubernetes. As it looks like we will likely move away from Tectonic, b/c this bug is a career destroying bug. |
Apologies, did not see the above comments until now.
So unfortunately I don't have any magic insights to offer -- just a tectonic/k8s n00b fumbling my way through this :-/ |
@ssunagari I have no idea what you are talking about, we have no problems with route53, nor with base domains or cluster names or any zones. Let me kindly suggest you open a new thread for your issue. Our issue is the the same as the Thread Creator's issue which is the count of the workers can't be upscaled. The line in question is this:
We contacted Tectonic on numerous channels on numerous occasions over the last half year, they only pointed to GitHub or to open our wallet. Since an unscalable Kubernetes Cluster is the very opposite of the very rasion d'être for any microservice-oriented archtiecture in the first place, we now are moving to KOPS and wish the Tectonic Project all the best of luck, it looks like they are going to need plenty of it. Overall the experience with Tectonic (Enterprise) and quay (Enterprise), their support and the quality of their products has been a very very bad one and I can only strongly suggest everybody to think twice before choosing them. Very dissapointed! |
What keywords did you search in tectonic-installer issues before filing this one?
terraform
existing vpc
value of 'count' cannot be computed
Is this a BUG REPORT or FEATURE REQUEST?
BUG REPORT
Versions
terraform version
): Terraform v0.10.6What happened?
When attempting to pass in subnet IDs from an existing VPC terraform module's output into the Tectonic module's
tectonic_aws_external_worker_subnet_ids
andtectonic_aws_external_master_subnet_ids
variables,terraform plan
fails with:What you expected to happen?
Tectonic terraform resources should successfully consume computed subnet IDs from other Terraform modules.
How to reproduce it (as minimally and precisely as possible)?
In the
module {}
invocation, reference output from another terraform module:Anything else we need to know?
This seems to be related to these existing upstream Terraform issues (computed values from interpolation functions such as
length()
not handled incount
attributes): hashicorp/terraform#12570 hashicorp/terraform#10462A possible fix for this is to have separate variables for number of subnets in
tectonic-installer/modules/aws/vpc/existing-vpc.tf
instead of usinglength()
:The text was updated successfully, but these errors were encountered: