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

AWS deployment requires terraform >= 0.12 #635

Closed
sokada1221 opened this issue Jul 4, 2019 · 2 comments
Closed

AWS deployment requires terraform >= 0.12 #635

sokada1221 opened this issue Jul 4, 2019 · 2 comments

Comments

@sokada1221
Copy link
Contributor

Bug Report

What version of Kubernetes are you using?
N/A

What version of TiDB Operator are you using?
latest master

What storage classes exist in the Kubernetes cluster and what are used for PD/TiKV pods?
N/A

What's the status of the TiDB cluster pods?
N/A

What did you do?

  1. Checkout latest tidb-operator
  2. terraform apply from deploy/aws

What did you expect to see?
Successful deployment according to the README
terraform version requirement should be documented

What did you see instead?
Parsing error from terraform due to first-class expression syntax. For example, in deploy/aws/bastion.tf, following lines will not work with terraform versions v0.11 and lower:

  vpc_id      = var.create_vpc ? module.vpc.vpc_id : var.vpc_id
  instance_count              = var.create_bastion ? 1 : 0

Looks like this feature was introduced in terraform v0.12.

@sokada1221
Copy link
Contributor Author

sokada1221 commented Jul 4, 2019

@aylei FYI - seems like a new requirement from #616

Edit: Quickly updated the README to reflect the current state. I personally think this new requirement is okay but I'll leave the design decision on whether older terraform versions should be supported or not to PingCAP.

@gregwebs
Copy link
Contributor

gregwebs commented Jul 5, 2019

closed by #636

@gregwebs gregwebs closed this as completed Jul 5, 2019
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

No branches or pull requests

2 participants