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

Make redis alternativeLocationId computed #296

Merged
merged 68 commits into from
Jul 27, 2018

Conversation

emilymye
Copy link
Contributor

@emilymye emilymye commented Jun 21, 2018

Make redis alternativeLocationId computed

Fixes hashicorp/terraform-provider-google#1669


[all]

[terraform]

Make redis alternativeLocationId computed

[puppet]

[puppet-compute]

[puppet-sql]

[puppet-storage]

[chef]

[chef-compute]

[chef-sql]

[chef-storage]

[ansible]

emilymye and others added 30 commits June 21, 2018 15:17
Readability improvements for things I stumbled over the first time going through the philosophy doc.

 * Syntax highlighting
 * Consistent pluralization
 * gendered -> non-gendered pronoun
Added documentation for resource and imports resource fields
Note: This deletes the Git hook that enforced the presubmit script and required
the NO_SQ=1 flag. You may have to fiddle a bit with your Git repo if it still
uses this git hook
danawillow and others added 25 commits July 12, 2018 17:46
…orm#372)

* Write tiny bash script to properly test Terraform.

* Remove locking.
Tracked submodules are build/puppet/bigquery build/puppet/compute build/puppet/sql build/puppet/storage build/puppet/container build/puppet/dns build/puppet/pubsub build/puppet/resourcemanager build/chef/compute build/chef/sql build/chef/storage build/chef/container build/chef/dns build/terraform build/ansible.
@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@modular-magician modular-magician merged commit 4b27fae into GoogleCloudPlatform:master Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating HA google_redis_instance doesn't seem to store auto-generated alternative_location_id
10 participants