Skip to content

samples/guestbook/gcp: don't manage project APIs by default #121

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

Merged
merged 3 commits into from
Jun 21, 2018
Merged

samples/guestbook/gcp: don't manage project APIs by default #121

merged 3 commits into from
Jun 21, 2018

Conversation

zombiezen
Copy link
Contributor

Once the fix for hashicorp/terraform-provider-google#1579 is released, then we can kick out this variable. In the short term, this may cause the user to need to enable services manually and re-run terraform apply.

Also set GKE cluster zone.

Updates #108

Once the fix for hashicorp/terraform-provider-google#1579 is
released, then we can kick out this variable. In the short term, this
may cause the user to need to enable services manually and re-run
`terraform apply`.

Also set GKE cluster zone.

Updates #108
@zombiezen zombiezen added this to the Sprint 13 milestone Jun 21, 2018
@zombiezen zombiezen requested a review from enocom June 21, 2018 17:54
Copy link
Contributor

@enocom enocom left a comment

Choose a reason for hiding this comment

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

This is an improvement as the terraform apply step now makes it almost to the end.

There's a MySQL connection error, but I'll file that as a separate issue, as it's not related to the work here.

@enocom enocom merged commit 6375ec1 into google:master Jun 21, 2018
@enocom enocom deleted the guestbook-gcpfixes branch June 21, 2018 22:57
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.

2 participants