-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Update aws tf #5435
Update aws tf #5435
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bfraz, Miouge1 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* 'master' of https://github.com/kubernetes-sigs/kubespray: Add script for generating binary hashes (kubernetes-sigs#5470) Bump yamllint and ansible-lint versions (kubernetes-sigs#5421) Remove inline shell in YAML for vagrant-validate (kubernetes-sigs#5386) Remove GCE tests files (kubernetes-sigs#5459) [docs] Add CI matrix and script (kubernetes-sigs#5461) Update CNI plugins to v0.8.3 (kubernetes-sigs#5453) Add kube-router configuration to enable metrics exposure (kubernetes-sigs#5416) fix typo (kubernetes-sigs#5452) cri-o: redhat.yml - remove package cri-tools (kubernetes-sigs#5444) Update CI to use v2.12.0 image and update release process (kubernetes-sigs#5448) Bump nodelocaldns version to 1.15.8 (kubernetes-sigs#5447) Add Helm 3.x support (kubernetes-sigs#5441) Update aws tf (kubernetes-sigs#5435)
* update aws tf to function as expected * update tf version * update syntax for tf v0.12 * update tf version in readme * update per tf for v0.12
What type of PR is this?
/kind bug
What this PR does / why we need it:
updates tf for aws provisioning given changing behavior with terraform v0.12. See: hashicorp/terraform#19140
Which issue(s) this PR fixes:
Fixes #5430
Special notes for your reviewer:
There was a change in tf 0.12 that broke previous behavior. This also updates aws tf docs to reflect the change.
For context: here's the changes for tf v0.8.7 to 0.11 #5431
Does this PR introduce a user-facing change?: