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

Virtual field for google_cloud_run_service revision names #3306

Merged
merged 2 commits into from
Mar 26, 2020

Conversation

c2thorn
Copy link
Member

@c2thorn c2thorn commented Mar 25, 2020

Fixes hashicorp/terraform-provider-google#5898

Release Note Template for Downstream PRs (will be copied)

cloudrun: added ability to autogenerate revision name

@modular-magician
Copy link
Collaborator

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 2 files changed, 29 insertions(+))
Terraform Beta: Diff ( 2 files changed, 29 insertions(+))
TF Conversion: Diff ( 1 file changed, 17 insertions(+), 1 deletion(-))

@c2thorn c2thorn requested a review from chrisst March 25, 2020 22:22
Copy link
Contributor

@chrisst chrisst left a comment

Choose a reason for hiding this comment

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

I'm pretty sure this allows for both manual revision management and auto and it works pretty well from my local testing, thanks! Can you add autogenerate_revision_name = true to one of the examples?

description: |
If set to `true`, the revision name (template.metadata.name) will be omitted and
autogenerated by Cloud Run. This cannot be set to `true` while `template.metadata.name`
is also set.
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be good to clarify somewhere that if name isn't in the config and this is false it will still be managed server side for legacy support.

@modular-magician
Copy link
Collaborator

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 3 files changed, 37 insertions(+), 2 deletions(-))
Terraform Beta: Diff ( 3 files changed, 37 insertions(+), 2 deletions(-))
TF Conversion: Diff ( 1 file changed, 17 insertions(+), 1 deletion(-))

@c2thorn c2thorn merged commit ef13586 into GoogleCloudPlatform:master Mar 26, 2020
@c2thorn c2thorn deleted the cloud-run-revision-name branch March 26, 2020 17:48
nathkn pushed a commit to nathkn/magic-modules that referenced this pull request May 18, 2020
…oudPlatform#3306)

* Add `autogenerate_revision_name` virtual field to `google_cloud_run_service`

* Add field to examples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cloud Run revision already exists
4 participants