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

Add interconnect attachment state waiting #2999

Merged

Conversation

rileykarson
Copy link
Member

@rileykarson rileykarson commented Jan 21, 2020

We should probably wait for resting states explicitly, but this is the sole
active state and this is the exact code the resource was tested with (since we don't have acceptance tests for it in Terraform)

/cc @jcanseco

Release Note Template for Downstream PRs (will be copied)

compute: added waiting logic to `google_compute_interconnect_attachment` to avoid modifications when the attachment is UNPROVISIONED

@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
Terraform Beta: Diff
TF Conversion: Diff

@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, fe5c99f.

Pull request statuses

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#1664
depends: GoogleCloudPlatform/terraform-google-conversion#327
depends: hashicorp/terraform-provider-google#5459

rileykarson and others added 2 commits January 21, 2020 22:21
Tracked submodules are build/terraform-beta build/terraform-mapper build/terraform build/ansible build/inspec.
@modular-magician modular-magician merged commit 4e9cbb2 into GoogleCloudPlatform:master Jan 21, 2020
@danawillow
Copy link
Contributor

@rileykarson
Copy link
Member Author

Fix in #3004

nathkn pushed a commit to nathkn/magic-modules that referenced this pull request May 18, 2020
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.

5 participants