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

Allow projects/ prefix for the project field in project IAM resources #3147

Merged
merged 2 commits into from
Feb 19, 2020

Conversation

rileykarson
Copy link
Member

Upstreams hashicorp/terraform-provider-google#5722

Release Note Template for Downstream PRs (will be copied)

iam: `google_project_iam_member` and `google_project_iam_binding`'s `project` field can be specified with an optional `projects/` prefix

rileykarson and others added 2 commits February 19, 2020 12:34
This patch allows the user to specify the value of the project field for
project IAM resources as `project-id` or `projects/project-id`.

Note that the above was already true for google_project_iam_policy [1].
This patch ensures that the above is also true for other project IAM
resources like google_project_iam_member and google_project_iam_binding.

[1] See commit 2f2f19f
@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 ( 1 file changed, 13 insertions(+), 9 deletions(-))
Terraform Beta: Diff ( 1 file changed, 13 insertions(+), 9 deletions(-))

@rileykarson rileykarson merged commit d0ac14d into GoogleCloudPlatform:master Feb 19, 2020
nathkn pushed a commit to nathkn/magic-modules that referenced this pull request May 18, 2020
…es (GoogleCloudPlatform#3147)

* Make iam_project.go.erb just .go

* Allow `projects/` prefix for the project field in project IAM resources

This patch allows the user to specify the value of the project field for
project IAM resources as `project-id` or `projects/project-id`.

Note that the above was already true for google_project_iam_policy [1].
This patch ensures that the above is also true for other project IAM
resources like google_project_iam_member and google_project_iam_binding.

[1] See commit 2f2f19f

Co-authored-by: Jazel Canseco <jazel.canseco@gmail.com>
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.

5 participants