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 it clear that we don't need to ForceSend nested fields in ComputeInstanceFromTemplate #2543

Conversation

danawillow
Copy link
Contributor

We would only need to do this in order to send false values in nested fields, and those work already, as proven by this test. Fixes hashicorp/terraform-provider-google#4505.

Release Note Template for Downstream PRs (will be copied)


@modular-magician
Copy link
Collaborator

Hi! I'm the modular magician, I work on Magic Modules.
I see that this PR has already had some downstream PRs generated. Any open downstreams are already updated to your most recent commit, ecffbbf.

Pull request statuses

No diff detected in terraform-google-conversion.
No diff detected in Ansible.
No diff detected in Inspec.

New Pull Requests

I built this PR into one or more new PRs on other repositories, and when those are closed, this PR will also be merged and closed.
depends: hashicorp/terraform-provider-google-beta#1308
depends: hashicorp/terraform-provider-google#4762

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.

nit: maybe change title to be more generic, otherwise it like the commit updating the tests wasn't intentional.

@danawillow danawillow changed the title remove TODO about ForceSending nested fields Make it clear that we don't need to ForceSend nested fields in ComputeInstanceFromTemplate Oct 25, 2019
danawillow and others added 3 commits October 25, 2019 23:22
Tracked submodules are build/terraform-beta build/terraform-mapper build/terraform build/ansible build/inspec.
@modular-magician modular-magician force-pushed the tf-4505-instance-from-template-nested branch from ecffbbf to 76c3b95 Compare October 25, 2019 23:22
@modular-magician modular-magician merged commit 109ee4f into GoogleCloudPlatform:master Oct 25, 2019
@danawillow danawillow deleted the tf-4505-instance-from-template-nested branch October 25, 2019 23:24
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.

Decide whether to send all levels or just top levels of compute instance from template
4 participants