Skip to content

Commit

Permalink
Permission for the bucket needs to be the entity name, not the name o…
Browse files Browse the repository at this point in the history
…f the resource
  • Loading branch information
mccare committed Dec 18, 2019
1 parent 1d40aab commit 9cee7a5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion upup/pkg/fi/cloudup/gcetasks/storagebucketacl.go
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ type terraformStorageBucketAcl struct {

func (_ *StorageBucketAcl) RenderTerraform(t *terraform.TerraformTarget, a, e, changes *StorageBucketAcl) error {
var roleEntities []string
roleEntities = append(roleEntities, fi.StringValue(e.Role)+":"+fi.StringValue(e.Name))
roleEntities = append(roleEntities, fi.StringValue(e.Role)+":"+fi.StringValue(e.Entity))
tf := &terraformStorageBucketAcl{
Bucket: fi.StringValue(e.Bucket),
RoleEntity: roleEntities,
Expand Down

0 comments on commit 9cee7a5

Please sign in to comment.