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 region disk support to attached disk #999

Conversation

chrisst
Copy link
Contributor

@chrisst chrisst commented Dec 6, 2018


[all]

Add region disk support to attached disk

[terraform]

[terraform-beta]

[ansible]

[inspec]

@chrisst
Copy link
Contributor Author

chrisst commented Dec 6, 2018

@modular-magician
Copy link
Collaborator

I am a robot that works on MagicModules PRs!

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

@chrisst
Copy link
Contributor Author

chrisst commented Dec 6, 2018

I didn't have to change the import statement after all because Read currently just iterates through a list of disks that are already attached to an instance and each disk is only referenced by name anyway.

@chrisst chrisst force-pushed the issue-2441-region-attached-disks branch from 6226d2b to 0614406 Compare December 7, 2018 17:37
@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit 00a7be8) have been included in your existing downstream PRs.

@modular-magician modular-magician force-pushed the issue-2441-region-attached-disks branch from 0614406 to 7c76090 Compare December 7, 2018 18:26
@modular-magician modular-magician merged commit 822f057 into GoogleCloudPlatform:master Dec 7, 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.

4 participants