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

GKE terraform #585

Merged
merged 11 commits into from
Jun 18, 2019
Merged

GKE terraform #585

merged 11 commits into from
Jun 18, 2019

Conversation

gregwebs
Copy link
Contributor

  • use gcloud CLI to automate service account provisioning
  • provide option to use small instances or prod instances
  • local-exec scripts: add set -e and make idempotent
  • use data.google_compute_zones.available.names to prevent bastion re-creation

* use gcloud CLI for service account provisioning
* provide option to use small instances or prod instances
* local-exec scripts: add set -e and make idempotent
@gregwebs gregwebs requested review from jlerche and tennix June 15, 2019 14:42
deploy/gcp/prod.tfvars Outdated Show resolved Hide resolved
}

provisioner "local-exec" {
interpreter = ["bash", "-c"]
command = <<EOS
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use EOT instead of EOS to make Terraform 0.12 format this correctly.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any word will work here.

deploy/gcp/README.md Outdated Show resolved Hide resolved
gregwebs and others added 2 commits June 17, 2019 09:23
jlerche
jlerche previously approved these changes Jun 17, 2019
gregwebs added 2 commits June 17, 2019 19:00
add DEBIAN_FRONTEND=noninteractive
this is still needed even when using -y

add dpkg configure -a which can be needed
tennix
tennix previously approved these changes Jun 18, 2019
Copy link
Member

@tennix tennix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The local volume provisioner startup can take some time
jlerche
jlerche previously approved these changes Jun 18, 2019
@gregwebs gregwebs dismissed stale reviews from jlerche and tennix via c82753e June 18, 2019 18:07
@gregwebs
Copy link
Contributor Author

Also added:

  • change the order of pool creation to prioritize local SSD setup and tiller deployment
  • local volume provisioner shell script improvements
  • don't auto-upgrade/repair
  • set a maitenance window

@gregwebs gregwebs merged commit e533707 into pingcap:master Jun 18, 2019
@gregwebs gregwebs deleted the gcp-terraform-improvements branch June 18, 2019 18:57
yahonda pushed a commit that referenced this pull request Dec 27, 2021
Signed-off-by: Ran <huangran@pingcap.com>
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

Successfully merging this pull request may close these issues.

3 participants