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

google_storage_bucket_acl tries to delete owner access #428

Closed
selmanj opened this issue Sep 13, 2017 · 3 comments
Closed

google_storage_bucket_acl tries to delete owner access #428

selmanj opened this issue Sep 13, 2017 · 3 comments
Assignees
Labels

Comments

@selmanj
Copy link
Contributor

selmanj commented Sep 13, 2017

Not clear why this is happening yet, but it appears that when trying to manage acls with field role_entity on resource google_storage_bucket_acl, they are applied one at a time and accidentally attempting to remove owners access, causing an error.

Terraform Version

Terraform v0.10.0-dev (be900e808506efb018bd6adbe457cfcbaf6281a9)
(master of terraform-provider-google)

Affected Resource(s)

  • google_storage_bucket_acl

Terraform Configuration Files

resource "google_storage_bucket" "log-bucket" {
    name     = "my-unique-bucket"
}

resource "google_storage_bucket_acl" "log-bucket-acl" {
  bucket = "${google_storage_bucket.log-bucket.name}"

  role_entity = [
    "OWNER:project-owners-943217423282",
    "OWNER:project-editors-943217423282",
    "READER:project-viewers-943217423282",
    "OWNER:[email protected]"
  ]
}

Expected Behavior

Should have applied acl list.

Actual Behavior

Received this error:

Error applying plan:

1 error(s) occurred:

* google_storage_bucket_acl.log-bucket-acl: 1 error(s) occurred:

* google_storage_bucket_acl.log-bucket-acl: Error updating ACL for bucket my-unique-bucket: googleapi: Error 403: The owner of the resource is required to have OWNER access., forbidden

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

Steps to Reproduce

  1. terraform apply
@paddycarver
Copy link
Contributor

This should be addressed by #439.

@selmanj
Copy link
Contributor Author

selmanj commented Oct 2, 2017

👍

@selmanj selmanj closed this as completed Oct 2, 2017
luis-silva pushed a commit to luis-silva/terraform-provider-google that referenced this issue May 21, 2019
@ghost
Copy link

ghost commented Mar 30, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants